Main

test: Add basic tests for SnapUIRenderer #57323

Sign in to view logs

Triggered via pull request February 5, 2025 13:49

FrederikBolding

synchronize #30099

fb/snap-ui-renderer-tests

Status Failure

Total duration 36m 59s

Artifacts 6

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

Run tests / Webpack tests

Run tests / Integration tests

Test circular deps / Test circular dependencies

Test deps audit / Test deps audit

Runway / Runway

Publish prerelease / Publish prerelease

Run tests / Report coverage

All jobs completed

Log merge group failure / Log merge group failure

Annotations

7 errors and 10 warnings

Test lint / Test lint: ui/components/app/snaps/snap-ui-renderer/snap-ui-renderer.test.js#L366

Replace `·label:·'Checkbox',·children:·Checkbox({·name:·'checkbox'·})` with `⏎··············label:·'Checkbox',⏎··············children:·Checkbox({·name:·'checkbox'·}),⏎···········`

Test lint / Test lint: ui/components/app/snaps/snap-ui-renderer/snap-ui-renderer.test.js#L424

Replace `·name:·'checkbox',·type:·'InputChangeEvent',·value:·true` with `⏎················name:·'checkbox',⏎················type:·'InputChangeEvent',⏎················value:·true,⏎·············`

Test lint / Test lint

Process completed with exit code 1.

Run tests / Unit tests (5)

Process completed with exit code 1.

Run tests / Unit tests (2)

The job was canceled because "_5" failed.

Run tests / Unit tests (2)

The operation was canceled.

All jobs pass

Process completed with exit code 1.

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/detected-token/detected-token.js#L117

React Hook useEffect has a missing dependency: 'tokensListDetected'. Either include it or remove the dependency array

Test lint / Test lint: ui/components/app/name/name.stories.tsx#L129

Named exports should not use the name annotation if it is redundant to the name that would be generated by the export name

Test lint / Test lint: ui/components/app/snaps/snap-home-page/snap-home-renderer.js#L49

React Hook useEffect has a missing dependency: 'dispatch'. Either include it or remove the dependency array

Artifacts

Produced during runtime

Name	Size
coverage-integration	795 KB
coverage-unit-1	1.07 MB
coverage-unit-3	1.07 MB
coverage-unit-4	1.08 MB
coverage-unit-6	1.07 MB
coverage-webpack	14.2 KB