Skip to content

Conversation

Aniket-Engg
Copy link
Collaborator

@Aniket-Engg Aniket-Engg commented Sep 22, 2025

From @bunsenstraat :

Previously we were relying on an old version of chrome to allow the extension which is built for an older version to load properly. However this is quite unmaintainable ofc. In the long run. So we see it failing.

The latest version of chrome doesn't allow extension loading so I used the 'chrome for testing' version. That one does allow loading extensions, but you can't find the tab opened for the extension besides using the generated extension ID, which you then have to hardcode.
Previously we could switch tabs to the extension but that doesn't work anymore. Chrome prevents access to that tab in the window handles.

In previous version We used a special metamask build that would allow e2e to interact with the extension like filling in fields. In the new extension which I build from src using the gh action I made for that, that doesn't work anymore too. You can't control it anymore. No more JS filling in fields.

  • if we stay with an older version of metamask the e2e is pointless, same for the chrome browser
  • continuously dealing with these complicated extension updates and security is a huge burden
  • maybe after all this it's better just to give it a manual test during QA, just by default.

ALSO, fixed the AI e2e

Copy link

netlify bot commented Sep 22, 2025

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit 2a67d46
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/68d10c904c227f0008338061
😎 Deploy Preview https://deploy-preview-6395--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented Sep 22, 2025

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit e93a347
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/68d115721bfbe50008771bfc
😎 Deploy Preview https://deploy-preview-6395--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Aniket-Engg Aniket-Engg added the browser-tests e2e tests for remix-ide label Sep 22, 2025
@Aniket-Engg Aniket-Engg merged commit a7dff5e into master Sep 22, 2025
38 checks passed
@Aniket-Engg Aniket-Engg deleted the metamask_disable branch September 22, 2025 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-tests e2e tests for remix-ide
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants