Update broken test after rebase

duckduckgo · Mar 3, 2025 · 2209e93 · 2209e93
1 parent b4b7010
commit 2209e93
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/src/main/java/com/duckduckgo/app/browser/BrowserTabViewModel.kt b/app/src/main/java/com/duckduckgo/app/browser/BrowserTabViewModel.kt
@@ -3303,7 +3303,10 @@ class BrowserTabViewModel @Inject constructor(
         )
 
         if (!exempted) {
-            if (currentBrowserViewState().maliciousSiteBlocked && previousSite?.url == url.toString()) return
+            if (currentBrowserViewState().maliciousSiteBlocked && previousSite?.url == url.toString()) {
+                Timber.tag("Cris").d("maliciousSiteBlocked already shown for $url, previousSite: ${previousSite.url}")
+                return
+            }
             Timber.d("Received MaliciousSiteWarning for $url, feed: $feed, exempted: false, clientSideHit: $clientSideHit")
             val params = mapOf(CATEGORY_KEY to feed.name.lowercase(), CLIENT_SIDE_HIT_KEY to clientSideHit.toString())
             pixel.fire(AppPixelName.MALICIOUS_SITE_PROTECTION_ERROR_SHOWN, params)

diff --git a/...java/com/duckduckgo/app/browser/webview/RealMaliciousSiteBlockerWebViewIntegrationTest.kt b/...java/com/duckduckgo/app/browser/webview/RealMaliciousSiteBlockerWebViewIntegrationTest.kt
@@ -144,7 +144,7 @@ class RealMaliciousSiteBlockerWebViewIntegrationTest {
 
         val result = testee.shouldIntercept(request, maliciousUri) {}
 
-        assertEquals(Safe, result)
+        assertEquals(Safe(true), result)
     }
 
     @Test
-Original file line number
+Diff line change
@@ Expand Up / @@ -144,7 +144,7 @@ class RealMaliciousSiteBlockerWebViewIntegrationTest { @@
             val result = testee.shouldIntercept(request, maliciousUri) {}
-            assertEquals(Safe, result)
+            assertEquals(Safe(true), result)
         }
         @Test
@@ Expand Down @@