Fix false positively suspicious google urls (#428)

* Fix false positively suspicious google urls

* Add an integration test

Author: yshym

modules/web-discovery-project/sources/web-discovery-project.es

@@ -1732,6 +1732,8 @@ const WebDiscoveryProject = {
         sanitizeUrl(activeURL, {
           testMode: WebDiscoveryProject.testMode,
         }).result !== "safe"
+        // For search engine URLs we check if the query is suspicious
+        && !WebDiscoveryProject.isSearchEngineUrl(activeURL)
       ) {
         logger.debug("[onLocationChange] isSuspiciousURL", activeURL);
         return;

modules/web-discovery-project/tests/integration/web-discovery-project-test.es

@@ -104,6 +104,22 @@ export default function () {
       });
     });
 
+    describe("web-discovery-project.searchEngineUrl", function () {
+      const testSearchEngineUrl = "https://google.com/search?q=apple+pie&sca_esv=3204631f57a9e94fc9b15156c2a60e5a&ei=6e19f02b9aade4603fe3bc81768743f6";
+
+      it(`${testPrivateUrl} is private`, function () {
+        WebDiscoveryProject.isAlreadyMarkedPrivate(testPrivateUrl, (e) => {
+          expect(e.private).to.equal(1);
+        });
+      });
+
+      it(`${testSearchEngineUrl} is public`, function () {
+        WebDiscoveryProject.isAlreadyMarkedPrivate(testSearchEngineUrl, (e) => {
+          expect(e.private).to.equal(0);
+        });
+      });
+    });
+
     describe("web-discovery-project.storage", function () {
       it("storage test", function (done) {
         WebDiscoveryProject.db.saveRecordTelemetry("unit-test", "test", () => {