remove fixture fallback config for phishing controller and default st…

…ate of it since now it is just an empty array
MetaMask · Mar 5, 2024 · c4026fe · c4026fe
1 parent 9c0de9f
commit c4026fe
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 15 deletions.
diff --git a/app/core/Engine.test.js b/app/core/Engine.test.js
@@ -42,10 +42,6 @@ describe('Engine', () => {
   it('matches initial state fixture', () => {
     const engine = Engine.init({});
     let backgroundState = engine.datamodel.state;
-    // Replace phishing controller fallback config, as it bloats the test fixture too much
-    backgroundState.PhishingController.phishingLists[0].allowlist = [];
-    backgroundState.PhishingController.phishingLists[0].blocklist = [];
-    backgroundState.PhishingController.phishingLists[0].fuzzylist = [];
 
     // deleting lastVisited from chainStatus, since its timestamp it makes the test case fail
     const { chainId, versionInfo } =

diff --git a/app/util/test/initial-background-state.json b/app/util/test/initial-background-state.json
@@ -75,17 +75,7 @@
     "networkStatus": "unknown"
   },
   "PhishingController": {
-    "phishingLists": [
-      {
-        "allowlist": [],
-        "blocklist": [],
-        "fuzzylist": [],
-        "tolerance": 2,
-        "version": 2,
-        "name": "MetaMask",
-        "lastUpdated": 0
-      }
-    ],
+    "phishingLists": [],
     "whitelist": [],
     "hotlistLastFetched": 0,
     "stalelistLastFetched": 0