Skip to content

Commit dc5ab3c

Browse files
authored
update data/ test lists (#518)
1 parent 082f2d6 commit dc5ab3c

File tree

9 files changed

+112561
-92228
lines changed

9 files changed

+112561
-92228
lines changed

data/brave/brave-main-list.txt

Lines changed: 90119 additions & 80075 deletions
Large diffs are not rendered by default.

data/brave/brave-resources.json

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

data/easylist.to/easylist/easylist.txt

Lines changed: 21775 additions & 11937 deletions
Large diffs are not rendered by default.

data/easylist.to/easylist/easyprivacy.txt

Lines changed: 541 additions & 136 deletions
Large diffs are not rendered by default.

data/easylist.to/easylistgermany/easylistgermany.txt

Lines changed: 91 additions & 61 deletions
Large diffs are not rendered by default.

data/update-lists.js

Lines changed: 24 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,11 @@ const args = process.argv.slice(2);
1010

1111
if (args.length < 2) {
1212
console.error(
13-
"Usage: BRAVE_SERVICE_KEY=<key> node update-lists.js <brave_list_version> <resource_list_version>\n" +
14-
"The component names are 'Brave Default Adblock Filters' and 'Brave Ad Block Resources Library'"
13+
"Usage: BRAVE_SERVICE_KEY=<key> node update-lists.js <defalt_adblock_filters_version> <defalt_privacy_filters_version> <resource_list_version>\n" +
14+
"The component names are: \n" +
15+
"Brave Default Adblock Filters\n" +
16+
"Brave Default Privacy Filters\n" +
17+
"Brave Ad Block Resources Library\n"
1518
);
1619
process.exit(1);
1720
}
@@ -21,10 +24,12 @@ if (!apiKey) {
2124
console.error("Error: BRAVE_SERVICE_KEY is not set");
2225
process.exit(1);
2326
}
24-
const braveVersionNumber = args[0].replace(/\./g, "_");
25-
const resourceVersionNumber = args[1].replace(/\./g, "_");
27+
const braveDefaultAdblockFiltersVersionNumber = args[0].replace(/\./g, "_");
28+
const braveDefaultPrivacyFiltersVersionNumber = args[1].replace(/\./g, "_");
29+
const resourceVersionNumber = args[2].replace(/\./g, "_");
2630

27-
const braveMainListId = "iodkpdagapdfkphljnddpjlldadblomo";
31+
const braveDefaultAdblockFiltersId = "iodkpdagapdfkphljnddpjlldadblomo";
32+
const braveDefaultPrivacyFiltersId = "kihnoaefogbkmblfimmibknnmkllbhlf";
2833
const braveResourceListId = "mfddibmblmbccpadfndgakiopmmhebop";
2934

3035
execSync(
@@ -48,8 +53,13 @@ try {
4853
process.chdir(tempDir);
4954

5055
execSync(
51-
`curl -o main_list.zip -H "BraveServiceKey: ${apiKey}" ` +
52-
`https://brave-core-ext.s3.brave.com/release/${braveMainListId}/extension_${braveVersionNumber}.crx`
56+
`curl -o brave-default-adblock-filters.zip -H "BraveServiceKey: ${apiKey}" ` +
57+
`https://brave-core-ext.s3.brave.com/release/${braveDefaultAdblockFiltersId}/extension_${braveDefaultAdblockFiltersVersionNumber}.crx`
58+
);
59+
60+
execSync(
61+
`curl -o brave-default-privacy-filters.zip -H "BraveServiceKey: ${apiKey}" ` +
62+
`https://brave-core-ext.s3.brave.com/release/${braveDefaultPrivacyFiltersId}/extension_${braveDefaultPrivacyFiltersVersionNumber}.crx`
5363
);
5464

5565
execSync(
@@ -70,7 +80,13 @@ try {
7080
fs.renameSync(fileName, path.join(rootDir, "data/brave", outputFileName));
7181
}
7282

73-
takeFile("main_list.zip", "list.txt", "brave-main-list.txt");
83+
// Merge the two default lists into one
84+
takeFile("brave-default-adblock-filters.zip", "list.txt", "default-1.txt");
85+
takeFile("brave-default-privacy-filters.zip", "list.txt", "default-2.txt");
86+
const defaultContent1 = fs.readFileSync(path.join(rootDir, "data/brave/default-1.txt"), { encoding: "utf-8" });
87+
const defaultContent2 = fs.readFileSync(path.join(rootDir, "data/brave/default-2.txt"), { encoding: "utf-8" });
88+
fs.writeFileSync(path.join(rootDir, "data/brave/brave-main-list.txt"), defaultContent1 + "\n" + defaultContent2);
89+
7490
takeFile("resources.zip", "resources.json", "brave-resources.json");
7591

7692
} finally {

tests/matching.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,5 +201,5 @@ fn check_rule_matching_browserlike() {
201201
let (blocked, passes) = bench_rule_matching_browserlike(&engine, &requests);
202202
let msg = "The number of blocked/passed requests has changed. ".to_string()
203203
+ "If this is expected, update the expected values in the test.";
204-
assert_eq!((blocked, passes), (101756, 141189), "{}", msg);
204+
assert_eq!((blocked, passes), (101638, 141307), "{}", msg);
205205
}

tests/unit/blocker.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1375,20 +1375,20 @@ mod legacy_rule_parsing_tests {
13751375
// * not handling document/subdocument options;
13761376
// * the optimizer that merges multiple rules into one;
13771377
const EASY_LIST: ListCounts = ListCounts {
1378-
filters: 35043,
1378+
filters: 45878 - 697,
13791379
cosmetic_filters: if cfg!(feature = "css-validation") {
1380-
23668
1380+
23639
13811381
} else {
1382-
23682
1382+
23655
13831383
},
1384-
exceptions: 706,
1384+
exceptions: 697,
13851385
duplicates: 0,
13861386
};
13871387
// differences in counts explained by hashset size underreporting as detailed in the next two cases
13881388
const EASY_PRIVACY: ListCounts = ListCounts {
1389-
filters: 53029, // total - exceptions
1390-
cosmetic_filters: 28,
1391-
exceptions: 737,
1389+
filters: 54156 - 756, // total - exceptions
1390+
cosmetic_filters: 29,
1391+
exceptions: 756,
13921392
duplicates: 2,
13931393
};
13941394
// ublockUnbreak = { 4, 8, 0, 94 };

tests/unit/engine.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,9 +219,9 @@ mod tests {
219219
let data = engine.serialize().unwrap();
220220

221221
let expected_hash = if cfg!(feature = "css-validation") {
222-
6556399856993017327
222+
14270179196454678892
223223
} else {
224-
4130628479730907288
224+
8787095256768277705
225225
};
226226

227227
assert_eq!(hash(&data), expected_hash, "{}", HASH_MISMATCH_MSG);

0 commit comments

Comments
 (0)