Skip to content

Commit

Permalink
Add default tests for some boolean parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
GuiltyDolphin committed Nov 13, 2023
1 parent 26d520c commit eaf49f0
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 0 deletions.
4 changes: 4 additions & 0 deletions broken-site-reporting/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,10 @@ Test suite specific fields:
- `remoteConfigVersion` - string - string representation of remote configuration version (note, this is the numeric version found in the remote config (e.g, `1680178584671`, not `v1` or `v2`))
- `providedDescription` - string - user-provided breakage description
- `protectionsEnabled` - boolean - if protections are enabled (true) or disabled (false) as visible to the user on the dashboard.
- `consentManaged` - bool - if the browser handled a cookie pop-up on the site
- `consentOptoutFailed` - bool - if the opt-out process returned failed
- `consentSelftestFailed` - bool - if the rule's self-test failed
- `urlParametersRemoved` - bool - if URL parameters were removed

All of these custom fields are supported by the `reports` array objects within the multiple_report_tests.json file.

Expand Down
25 changes: 25 additions & 0 deletions broken-site-reporting/tests.json
Original file line number Diff line number Diff line change
Expand Up @@ -421,6 +421,31 @@
"safari-extension",
"windows-browser"
]
},
{
"name": "Default sending of consent* parameters",
"siteURL": "http://example.com/",
"expectReportURLParams": [
{"name": "consentManaged", "value": "false"},
{"name": "consentOptoutFailed", "value": "false"},
{"name": "consentSelftestFailed", "value": "false"}
],
"exceptPlatforms": [
"ios-browser",
"safari-extension",
"web-extension"
]
},
{
"name": "Default sending of urlParametersRemoved",
"siteURL": "http://example.com/",
"expectReportURLParams": [
{"name": "urlParametersRemoved", "value": "false"}
],
"exceptPlatforms": [
"safari-extension",
"windows-browser"
]
}
]
}
Expand Down

0 comments on commit eaf49f0

Please sign in to comment.