Fix e2es

MetaMask · Sep 20, 2023 · ff04005 · ff04005
1 parent 135ea69
commit ff04005
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/app/scripts/lib/ens-ipfs/setup.js b/app/scripts/lib/ens-ipfs/setup.js
@@ -60,8 +60,7 @@ export default function setupEnsIpfsResolver({
 
     const ensSiteUrl = `https://app.ens.domains/name/${name}`;
 
-    // This is the only case where we can show the loading indicator
-    // https://github.com/MetaMask/metamask-extension/issues/20910#issuecomment-1723962822
+    // We cannot show this if useAddressBarEnsResolution is off...
     if (useAddressBarEnsResolution && ipfsGateway) {
       browser.tabs.update(tabId, { url: 'loading.html' });
     }
@@ -129,6 +128,8 @@ export default function setupEnsIpfsResolver({
     } catch (err) {
       console.warn(err);
     } finally {
+      // Only forward to destination URL if a URL exists and
+      // useAddressBarEnsResolution is properly
       if (
         url &&
         (useAddressBarEnsResolution ||

diff --git a/test/e2e/tests/ipfs-ens-resolution.spec.js b/test/e2e/tests/ipfs-ens-resolution.spec.js
@@ -31,7 +31,7 @@ describe('Settings', function () {
     await driver.quit();
   });
 
-  it('Does not lookup IPFS data for ENS Domain when switched off', async function () {
+  it('Does not ENS data for ENS Domain when switched off', async function () {
     let server;
 
     await withFixtures(
@@ -54,7 +54,10 @@ describe('Settings', function () {
         await driver.clickElement({ text: 'Settings', tag: 'div' });
         await driver.clickElement({ text: 'Security & privacy', tag: 'div' });
 
-        // turns off IPFS domain resolution
+        // turns off IPFS setting
+        await driver.clickElement('[data-testid="ipfsToggle"] .toggle-button');
+
+        // turns off ENS domain resolution
         await driver.clickElement(
           '[data-testid="ipfs-gateway-resolution-container"] .toggle-button',
         );