Back to pull request #30515

fix(30509): remove 'fake-metrics-id' in e2e to avoid controller error #59466

Sign in to view logs

Triggered via pull request February 24, 2025 08:20

DDDDDanica

synchronize #30515

fix/30509-e2e-metrics-id

Status Success

Total duration 44m 49s

Artifacts 15

main.yml

on: pull_request

Wait for CircleCI workflow status / Wait for CircleCI workflow status

Build storybook / Build storybook

Build beta / Build beta

Matrix: Run tests / Unit tests

Matrix: Validate lavamoat policy webapp / Validate lavamoat policy webapp

Lint workflows / Lint workflows

Test lint shellcheck / Test lint shellcheck

Test lint / Test lint

Test lint changelog / Test lint changelog

Test lint lockfile / Test lint lockfile

Test yarn dedupe / Test yarn dedupe

Test deps depcheck / Test deps depcheck

Test storybook / Test storybook

Validate lavamoat allow scripts / Validate lavamoat allow scripts

Validate lavamoat policy build / Validate lavamoat policy build

prep-build-test-browserify

prep-build-test-webpack

Run tests / Webpack tests

Run tests / Integration tests

Test circular deps

Test deps audit / Test deps audit

Runway / Runway

Publish prerelease / Publish prerelease

Matrix: run-benchmarks / benchmarks

Run tests / Report coverage

All jobs completed

Log merge group failure / Log merge group failure

Annotations

10 warnings

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1265

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1286

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1306

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1390

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1406

Some tests seem to be commented

Test lint / Test lint: app/scripts/controllers/swaps/swaps.test.ts#L1429

Some tests seem to be commented

Test lint / Test lint: test/e2e/flask/solana/common-solana.ts#L91

'#fb9f18' Hex color values are not allowed. Consider using design tokens instead. For support reach out to the design system team #metamask-design-system on Slack

Test lint / Test lint: ui/components/app/alert-system/alert-modal/alert-modal.tsx#L209

React Hook "useI18nContext" is called conditionally. React Hooks must be called in the exact same order in every component render. Did you accidentally call a React Hook after an early return?

Test lint / Test lint: ui/components/app/assets/asset-list/asset-list-control-bar/asset-list-control-bar.tsx#L83

React Hook useMemo has an unnecessary dependency: 'TEST_CHAINS'. Either exclude it or remove the dependency array. Outer scope values like 'TEST_CHAINS' aren't valid dependencies because mutating them doesn't re-render the component

Test lint / Test lint: ui/components/app/assets/asset-list/asset-list-control-bar/asset-list-control-bar.tsx#L109

React Hook useEffect has missing dependencies: 'allOpts', 'currentNetwork.chainId', 'dispatch', and 'tokenNetworkFilter'. Either include them or remove the dependency array

Artifacts

Produced during runtime

Name	Size
benchmark-browserify-pageload	819 Bytes
benchmark-browserify-userActions	197 Bytes
benchmark-webpack-pageload	820 Bytes
benchmark-webpack-userActions	197 Bytes
coverage-integration	804 KB
coverage-unit-1	1.08 MB
coverage-unit-2	1.07 MB
coverage-unit-3	1.08 MB
coverage-unit-4	1.09 MB
coverage-unit-5	1.07 MB
coverage-unit-6	1.08 MB
coverage-webpack	14.6 KB
lcov.info	333 KB
prep-build-test-browserify	58.5 MB
prep-build-test-webpack	27.5 MB