Move flask E2E tests to separate folder

[matthewwalsh0]

Description

Currently all flask specific E2E tests are ran by the test-e2e-chrome-snaps-flask and test-e2e-firefox-snaps-flask pipeline jobs.

To achieve this they are placed in the test/e2e/snaps folder, however as all snaps directories are owned by the @MetaMask/snaps-devs, this creates a dependency where an approval from the snaps team is required for any unrelated change to these flask E2E tests.

Pending a larger refactor to organise the E2E pipeline jobs by build type, this PR simply moves the flask specific E2E tests into a separate directory and updates the codeowners so both extension and snaps developers own flask folders since the flask build type is now used by any team wishing to trial new functionality outside of the main release.

Pre-merge author checklist

• I’ve followed MetaMask Coding Standards.
• I've clearly explained:
  • What problem this PR is solving.
  • How this problem was solved.
  • How reviewers can test my changes.
• I’ve indicated what issue this PR is linked to: Fixes #???
• I’ve included tests if applicable.
• I’ve documented any added code.
• I’ve applied the right labels on the PR (see labeling guidelines).
• I’ve properly set the pull request status:
  • In case it's not yet "ready for review", I've set it to "draft".
  • In case it's "ready for review", I've changed it from "draft" to "non-draft".

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (12c9d97) 68.49% compared to head (ac9a6af) 68.49%.
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #21111   +/-   ##
========================================
  Coverage    68.49%   68.49%           
========================================
  Files         1010     1010           
  Lines        40398    40398           
  Branches     10788    10788           
========================================
  Hits         27669    27669           
  Misses       12729    12729           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [ac9a6af]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4
  • common: 0, 1, 2, 3, 4, 5, 6
  • content-script: 0
  • contentscript: contentscript
  • disable-console: disable-console
  • inpage: inpage
  • policy-load: policy-load
  • sentry-install: sentry-install
  • ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (835 ± 391 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	88	116	98	8	4
		domContentLoaded	70	136	94	17	8
		load	82	1901	835	814	391
		domInteractive	70	135	94	17	8

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 0 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[danjm]

Makes sense 👍