Skip to content

Commit ec553eb

Browse files
authored
Merge branch 'main' into fix/hardware-wallet-eip-7702-issue
2 parents b7364d4 + e8e693d commit ec553eb

209 files changed

Lines changed: 8619 additions & 14711 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/CODEOWNERS

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,8 +121,10 @@ app/components/Views/AccountSelector @MetaMask/accounts-e
121121
**/multichainAccounts/** @MetaMask/accounts-engineers
122122

123123
# Swaps Team
124-
app/components/UI/Swaps @MetaMask/swaps-engineers
125-
app/components/UI/Bridge @MetaMask/swaps-engineers
124+
app/components/UI/Swaps @MetaMask/swaps-engineers
125+
app/components/UI/Bridge @MetaMask/swaps-engineers
126+
app/core/Engine/controllers/bridge-controller @MetaMask/swaps-engineers
127+
app/core/Engine/controllers/bridge-status-controller @MetaMask/swaps-engineers
126128

127129
# Notifications Team
128130
app/components/Views/Notifications @MetaMask/notifications

.github/scripts/collect-qa-stats.mjs

Lines changed: 253 additions & 34 deletions
Large diffs are not rendered by default.

.github/workflows/build.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ jobs:
115115
build_name: ${{ inputs.build_name }}
116116
use-tarball: true
117117
upload-artifact: true
118-
artifact-name: node-modules-${{ matrix.platform }}
118+
artifact-name: node-modules-${{ inputs.build_name }}-${{ matrix.platform }}
119119
artifact-retention-days: 1
120120

121121
# Build
@@ -165,7 +165,7 @@ jobs:
165165
- name: Download node_modules tarball
166166
uses: actions/download-artifact@v4
167167
with:
168-
name: node-modules-${{ matrix.platform }}
168+
name: node-modules-${{ inputs.build_name }}-${{ matrix.platform }}
169169

170170
- name: Extract tarball (preserves symlinks)
171171
run: |

.github/workflows/ci.yml

Lines changed: 30 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,6 +190,7 @@ jobs:
190190
with:
191191
name: ios-bundle
192192
path: ios/main.jsbundle
193+
retention-days: 7
193194

194195
ship-js-bundle-size-check:
195196
runs-on: ubuntu-latest
@@ -262,6 +263,7 @@ jobs:
262263
name: coverage-unit-${{ matrix.shard }}
263264
path: ./tests/coverage/
264265
if-no-files-found: error
266+
retention-days: 7
265267
- name: Require clean working directory
266268
shell: bash
267269
run: |
@@ -272,6 +274,8 @@ jobs:
272274
echo "No changes detected"
273275
fi
274276
277+
# We need to merge both unit and component view tests into a single coverage report so the PR coverage
278+
# threshold calculation is accurate.
275279
merge-unit-and-component-view-tests:
276280
runs-on: ubuntu-latest
277281
needs: [unit-tests, component-view-tests]
@@ -321,6 +325,11 @@ jobs:
321325
[ -f "$file" ] && cp "$file" ./tests/coverage-cv-merged/
322326
done
323327
328+
mkdir -p tests/coverage-unit-merged
329+
for file in ./tests/coverage/coverage-unit-*/coverage-unit-*.json; do
330+
[ -f "$file" ] && cp "$file" ./tests/coverage-unit-merged/
331+
done
332+
324333
find ./tests/coverage/coverage-* -name 'coverage-*.json' -exec mv {} ./tests/coverage/ \;
325334
- run: yarn test:merge-coverage
326335
- run: yarn test:validate-coverage
@@ -329,23 +338,41 @@ jobs:
329338
name: lcov.info
330339
path: ./tests/merged-coverage/lcov.info
331340
if-no-files-found: error
341+
retention-days: 7
332342
- uses: actions/upload-artifact@v4
333343
with:
334344
name: cv-test-stats
335345
path: ./cv-test-stats.json
336346
if-no-files-found: error
347+
retention-days: 7
337348
- uses: actions/upload-artifact@v4
338349
with:
339350
name: unit-test-stats
340351
path: ./unit-test-stats.json
341352
if-no-files-found: error
342-
- name: Generate CV test HTML coverage report
343-
run: yarn nyc report --temp-dir ./tests/coverage-cv-merged --report-dir ./tests/coverage-cv-lcov --reporter html
353+
retention-days: 7
354+
- name: Generate CV test coverage report
355+
run: yarn nyc report --temp-dir ./tests/coverage-cv-merged --report-dir ./tests/coverage-cv-lcov --reporter html --reporter json-summary
344356
- uses: actions/upload-artifact@v4
345357
with:
346358
name: cv-test-coverage-html
347359
path: ./tests/coverage-cv-lcov/
348360
if-no-files-found: error
361+
retention-days: 7
362+
- uses: actions/upload-artifact@v4
363+
with:
364+
name: cv-test-coverage-summary
365+
path: ./tests/coverage-cv-lcov/coverage-summary.json
366+
if-no-files-found: error
367+
retention-days: 7
368+
- name: Generate unit test coverage summary
369+
run: yarn nyc report --temp-dir ./tests/coverage-unit-merged --report-dir ./tests/coverage-unit-lcov --reporter json-summary
370+
- uses: actions/upload-artifact@v4
371+
with:
372+
name: unit-test-coverage-summary
373+
path: ./tests/coverage-unit-lcov/coverage-summary.json
374+
if-no-files-found: error
375+
retention-days: 7
349376
- name: Require clean working directory
350377
shell: bash
351378
run: |
@@ -397,6 +424,7 @@ jobs:
397424
name: coverage-cv-${{ matrix.shard }}
398425
path: ./tests/coverage/
399426
if-no-files-found: error
427+
retention-days: 7
400428

401429
needs_e2e_build:
402430
uses: ./.github/workflows/needs-e2e-build.yml

.github/workflows/qa-stats.yml

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,13 @@
11
name: QA Stats
22

33
on:
4-
workflow_run:
5-
workflows:
6-
- ci
7-
types:
8-
- completed
9-
branches:
10-
- main
4+
schedule:
5+
- cron: '0 4 * * *'
116

127
jobs:
138
collect-qa-stats:
149
name: Collect QA Stats
1510
runs-on: ubuntu-latest
16-
if: ${{ github.event.workflow_run.conclusion == 'success' && github.event.workflow_run.event == 'push' && github.event.workflow_run.head_repository.full_name == github.repository }}
1711

1812
steps:
1913
- uses: actions/checkout@v6
@@ -26,7 +20,7 @@ jobs:
2620
run: node .github/scripts/collect-qa-stats.mjs
2721
env:
2822
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
29-
WORKFLOW_RUN_ID: ${{ github.event.workflow_run.id }}
23+
GITHUB_REPOSITORY: ${{ github.repository }}
3024

3125
- name: Upload QA stats artifact
3226
uses: actions/upload-artifact@v6

.github/workflows/run-e2e-smoke-tests-android-flask.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,3 +205,4 @@ jobs:
205205
with:
206206
name: e2e-smoke-android-flask-all-test-artifacts
207207
path: all-test-artifacts/
208+
retention-days: 7

.github/workflows/run-e2e-smoke-tests-android.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,7 @@ jobs:
259259
with:
260260
name: e2e-smoke-android-all-test-artifacts
261261
path: all-test-artifacts/
262+
retention-days: 7
262263

263264
- name: Create mobile JSON test report
264265
id: create-json-report
@@ -290,3 +291,4 @@ jobs:
290291
with:
291292
name: test-e2e-android-json-report
292293
path: test/test-results/test-runs-android.json
294+
retention-days: 7

.github/workflows/run-e2e-smoke-tests-ios-flask.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,3 +181,4 @@ jobs:
181181
with:
182182
name: e2e-smoke-ios-flask-all-test-artifacts
183183
path: all-test-artifacts/
184+
retention-days: 7

.github/workflows/run-e2e-smoke-tests-ios.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -283,6 +283,7 @@ jobs:
283283
with:
284284
name: e2e-smoke-ios-all-test-artifacts
285285
path: all-test-artifacts/
286+
retention-days: 7
286287

287288
- name: Create mobile JSON test report
288289
id: create-json-report
@@ -314,3 +315,4 @@ jobs:
314315
with:
315316
name: test-e2e-ios-json-report
316317
path: test/test-results/test-runs-ios.json
318+
retention-days: 7

.js.env.example

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,4 +208,7 @@ export MM_PREDICT_ENABLED="true"
208208
export MM_CARD_BAANX_API_CLIENT_KEY_DEV=""
209209

210210
## PNA25 (Privacy Notice)
211-
export MM_EXTENSION_UX_PNA25=""
211+
export MM_EXTENSION_UX_PNA25=""
212+
213+
## Metro
214+
export METRO_RESET_CACHE="true"

0 commit comments

Comments
 (0)