Bug 1872295 - Enable the 'browser.search.log' pref by default across all search unit tests. r=jteow

Differential Revision: https://phabricator.services.mozilla.com/D197390
This commit is contained in:
Stephanie Cunnane 2023-12-28 23:02:11 +00:00
Родитель 4303f386e4
Коммит 2540cc093a
7 изменённых файлов: 1 добавлений и 8 удалений

Просмотреть файл

@ -95,7 +95,6 @@ const TEST_DOMAIN_TO_CATEGORIES_MAP_RANK_PENALIZATION_3 = {
};
add_setup(async () => {
Services.prefs.setBoolPref("browser.search.log", true);
Services.prefs.setBoolPref(
"browser.search.serpEventTelemetryCategorization.enabled",
true

Просмотреть файл

@ -51,7 +51,6 @@ const TESTS = [
];
add_setup(async function () {
Services.prefs.setBoolPref(SearchUtils.BROWSER_SEARCH_PREF + "log", true);
Services.prefs.setBoolPref(
SearchUtils.BROWSER_SEARCH_PREF + "serpEventTelemetry.enabled",
true

Просмотреть файл

@ -92,7 +92,6 @@ const RECORDS = {
};
add_setup(async () => {
Services.prefs.setBoolPref("browser.search.log", true);
Services.prefs.setBoolPref(
"browser.search.serpEventTelemetryCategorization.enabled",
true

Просмотреть файл

@ -8,7 +8,6 @@
ChromeUtils.defineESModuleGetters(this, {
SearchSERPTelemetry: "resource:///modules/SearchSERPTelemetry.sys.mjs",
SearchUtils: "resource://gre/modules/SearchUtils.sys.mjs",
});
const TESTS = [
@ -165,7 +164,6 @@ const TESTS = [
];
add_setup(async function () {
Services.prefs.setBoolPref(SearchUtils.BROWSER_SEARCH_PREF + "log", true);
await SearchSERPTelemetry.init();
});

Просмотреть файл

@ -6,7 +6,6 @@ ChromeUtils.defineESModuleGetters(this, {
NetUtil: "resource://gre/modules/NetUtil.sys.mjs",
PrivateBrowsingUtils: "resource://gre/modules/PrivateBrowsingUtils.sys.mjs",
SearchSERPTelemetry: "resource:///modules/SearchSERPTelemetry.sys.mjs",
SearchUtils: "resource://gre/modules/SearchUtils.sys.mjs",
TelemetryTestUtils: "resource://testing-common/TelemetryTestUtils.sys.mjs",
sinon: "resource://testing-common/Sinon.sys.mjs",
});
@ -263,7 +262,6 @@ async function testAdUrlClicked(serpUrl, adUrl, expectedAdKey) {
do_get_profile();
add_task(async function setup() {
Services.prefs.setBoolPref(SearchUtils.BROWSER_SEARCH_PREF + "log", true);
await SearchSERPTelemetry.init();
sinon.stub(BrowserSearchTelemetry, "shouldRecordSearchCount").returns(true);
// There is no concept of browsing in unit tests, so assume in tests that we

Просмотреть файл

@ -259,7 +259,6 @@ async function testAdUrlClicked(serpUrl, adUrl, expectedAdKey) {
do_get_profile();
add_task(async function setup() {
Services.prefs.setBoolPref(SearchUtils.BROWSER_SEARCH_PREF + "log", true);
Services.prefs.setBoolPref(
SearchUtils.BROWSER_SEARCH_PREF + "serpEventTelemetry.enabled",
true

Просмотреть файл

@ -1,5 +1,6 @@
[DEFAULT]
support-files = ["../../../../../services/settings/dumps/main/search-config-v2.json"]
prefs = ["browser.search.log=true"]
skip-if = ["toolkit == 'android'"] # bug 1730213
firefox-appdir = "browser"