From a99503969221363261f22b359255ef7e89ba05ac Mon Sep 17 00:00:00 2001
From: Mark Banner <standard8@mozilla.com>
Date: Mon, 17 Oct 2022 12:56:04 +0000
Subject: [PATCH] Bug 1445134 - Remove user/policy engine tests from
 test_webextensions_valid.js as they are already covered elsewhere. r=mcheang

These are covered by test_userEngine.js and test_policyEngine.js

Differential Revision: https://phabricator.services.mozilla.com/D159032
---
 .../xpcshell/test_webextensions_valid.js      | 34 -------------------
 1 file changed, 34 deletions(-)

diff --git a/toolkit/components/search/tests/xpcshell/test_webextensions_valid.js b/toolkit/components/search/tests/xpcshell/test_webextensions_valid.js
index 684f5b203510..ad89aa3f4dc1 100644
--- a/toolkit/components/search/tests/xpcshell/test_webextensions_valid.js
+++ b/toolkit/components/search/tests/xpcshell/test_webextensions_valid.js
@@ -160,37 +160,3 @@ add_task(async function test_missing_extension() {
 
   await oldRemoveEngineFunc(Services.search.getEngineByName("Example"));
 });
-
-add_task(async function test_user_engine() {
-  Services.telemetry.clearScalars();
-
-  await Services.search.addUserEngine("test", "https://example.com/", "fake");
-
-  await Services.search.runBackgroundChecks();
-  let scalars = TelemetryTestUtils.getProcessScalars("parent", true, true);
-
-  Assert.deepEqual(
-    scalars,
-    {},
-    "Should not have recorded any issues for a user-defined engine"
-  );
-});
-
-add_task(async function test_policy_engine() {
-  Services.telemetry.clearScalars();
-
-  await Services.search.addPolicyEngine({
-    description: "test policy engine",
-    name: "test_policy_engine",
-    search_url: "https://www.example.org/?search={searchTerms}",
-  });
-
-  await Services.search.runBackgroundChecks();
-  let scalars = TelemetryTestUtils.getProcessScalars("parent", true, true);
-
-  Assert.deepEqual(
-    scalars,
-    {},
-    "Should not have recorded any issues for a policy defined engine"
-  );
-});