Skip to content

Commit a3acc21

Browse files
shakyShaneShane Osbourne
andauthored
added 'protectionsState' to breakage report (#112)
Co-authored-by: Shane Osbourne <sosbourne@duckduckgo.com>
1 parent 7519c3d commit a3acc21

File tree

3 files changed

+110
-17
lines changed

3 files changed

+110
-17
lines changed

broken-site-reporting/README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ Test suite specific fields:
3434
- `remoteConfigEtag` - string - string representation of remote configuration etag
3535
- `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`))
3636
- `providedDescription` - string - user-provided breakage description
37+
- `protectionsEnabled` - boolean - if protections are enabled (true) or disabled (false) as visible to the user on the dashboard.
3738

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

@@ -136,3 +137,6 @@ the reference tests is provided below:
136137
- truncation tests:
137138
- the following platforms do not currently implement the optional `noActionRequests`, `adAttributionRequests`, `ignoredByUserRequests`, or `ignoreRequests` truncatable parameters: `android-browser`, `ios-browser`, `macos-browser`, `safari-extension`, `windows-browser`
138139
- see https://app.asana.com/0/0/1204271046995906/f for more information about truncation and implementation requirements
140+
- param `protectionsState`
141+
- The Extension is the only platform that supports a 'denylist' - this allows a user to override remote configurations and forcibly enable protections even when we've tried to disable them.
142+
- For this case, an extension-only case was added in `broken-site-reporting/tests.json`

broken-site-reporting/multiple_report_tests.json

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
"blocklistVersion": "abc123",
1616
"remoteConfigEtag": "abd142",
1717
"remoteConfigVersion": "1234",
18+
"protectionsEnabled": true,
1819
"expectReportURLPrefix": "https://improving.duckduckgo.com/t/epbf",
1920
"expectReportURLParams": [
2021
{"name": "category", "value": "content"},
@@ -23,7 +24,8 @@
2324
{"name": "tds", "value": "abc123"},
2425
{"name": "blockedTrackers", "value": "bad.tracker.test,tracking.test"},
2526
{"name": "surrogates", "value": "surrogate.domain.test,domain2.test"},
26-
{"name": "lastSentDay", "present": false}
27+
{"name": "lastSentDay", "present": false},
28+
{"name": "protectionsState", "value": "true"}
2729
]
2830
},
2931
{
@@ -36,6 +38,7 @@
3638
"blocklistVersion": "abc123",
3739
"remoteConfigEtag": "abd142",
3840
"remoteConfigVersion": "1234",
41+
"protectionsEnabled": true,
3942
"expectReportURLPrefix": "https://improving.duckduckgo.com/t/epbf",
4043
"expectReportURLParams": [
4144
{"name": "category", "value": "content"},
@@ -46,7 +49,8 @@
4649
{"name": "remoteConfigVersion", "value": "1234"},
4750
{"name": "blockedTrackers", "value": "bad.tracker.test,tracking.test"},
4851
{"name": "surrogates", "value": "surrogate.domain.test,domain2.test"},
49-
{"name": "lastSentDay", "present": true, "matchesCurrentDay": true, "matches": "^[0-9]{4}-[0-9]{2}-[0-9]{2}$"}
52+
{"name": "lastSentDay", "present": true, "matchesCurrentDay": true, "matches": "^[0-9]{4}-[0-9]{2}-[0-9]{2}$"},
53+
{"name": "protectionsState", "value": "true"}
5054
]
5155
},
5256
{
@@ -59,6 +63,7 @@
5963
"blocklistVersion": "abc123",
6064
"remoteConfigEtag": "abd142",
6165
"remoteConfigVersion": "1234",
66+
"protectionsEnabled": true,
6267
"expectReportURLPrefix": "https://improving.duckduckgo.com/t/epbf",
6368
"expectReportURLParams": [
6469
{"name": "category", "value": "content"},
@@ -69,7 +74,8 @@
6974
{"name": "remoteConfigVersion", "value": "1234"},
7075
{"name": "blockedTrackers", "value": "bad.tracker.test,tracking.test"},
7176
{"name": "surrogates", "value": "surrogate.domain.test,domain2.test"},
72-
{"name": "lastSentDay", "present": false}
77+
{"name": "lastSentDay", "present": false},
78+
{"name": "protectionsState", "value": "true"}
7379
]
7480
},
7581
{
@@ -82,6 +88,7 @@
8288
"blocklistVersion": "abc123",
8389
"remoteConfigEtag": "abd142",
8490
"remoteConfigVersion": "1234",
91+
"protectionsEnabled": true,
8592
"expectReportURLPrefix": "https://improving.duckduckgo.com/t/epbf",
8693
"expectReportURLParams": [
8794
{"name": "category", "value": "content"},
@@ -92,7 +99,8 @@
9299
{"name": "remoteConfigVersion", "value": "1234"},
93100
{"name": "blockedTrackers", "value": "bad.tracker.test,tracking.test"},
94101
{"name": "surrogates", "value": "surrogate.domain.test,domain2.test"},
95-
{"name": "lastSentDay", "present": true, "matchesCurrentDay": true, "matches": "^[0-9]{4}-[0-9]{2}-[0-9]{2}$"}
102+
{"name": "lastSentDay", "present": true, "matchesCurrentDay": true, "matches": "^[0-9]{4}-[0-9]{2}-[0-9]{2}$"},
103+
{"name": "protectionsState", "value": "true"}
96104
]
97105
},
98106
{
@@ -105,6 +113,7 @@
105113
"blocklistVersion": "abc123",
106114
"remoteConfigEtag": "abd142",
107115
"remoteConfigVersion": "1234",
116+
"protectionsEnabled": true,
108117
"expectReportURLPrefix": "https://improving.duckduckgo.com/t/epbf",
109118
"expectReportURLParams": [
110119
{"name": "category", "value": "content"},
@@ -115,7 +124,8 @@
115124
{"name": "remoteConfigVersion", "value": "1234"},
116125
{"name": "blockedTrackers", "value": "bad.tracker.test,tracking.test"},
117126
{"name": "surrogates", "value": "surrogate.domain.test,domain2.test"},
118-
{"name": "lastSentDay", "present": false}
127+
{"name": "lastSentDay", "present": false},
128+
{"name": "protectionsState", "value": "true"}
119129
]
120130
}
121131
],

0 commit comments

Comments
 (0)