Skip to content
This repository has been archived by the owner on Jun 16, 2022. It is now read-only.

Commit

Permalink
v2.33.1
Browse files Browse the repository at this point in the history
  • Loading branch information
gre committed Sep 13, 2021
1 parent f35401d commit bbd6f08
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"name": "ledger-live-desktop",
"productName": "Ledger Live",
"description": "Ledger Live - Desktop",
"version": "2.33.0",
"version": "2.33.1",
"author": "Ledger Live Team <[email protected]>",
"repository": "https://github.com/LedgerHQ/ledger-live-desktop",
"license": "MIT",
Expand Down

1 comment on commit bbd6f08

@ledger-live-comment-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @gre

Lint outputs ✅

Tests outputs ✅

PASS src/generate-cryptoassets-md.test.js

Test Suites: 1 skipped, 1 passed, 1 of 2 total
Tests: 5 skipped, 1 passed, 6 total
Snapshots: 0 total
Time: 1.617 s
Ran all test suites matching /src/i.

Tests outputs ❌

PASS tests/specs/swap.spec.js (76.767 s)
PASS tests/specs/firmwareupdate.spec.js (85 s)
FAIL tests/specs/global.spec.js (127.654 s)
● Global › go to buy / sell cryto

Expected image to be the same size as the snapshot (1024x1264), but was different (1024x1180).
See diff for details: /home/runner/work/ledger-live-desktop/ledger-live-desktop/tests/specs/__image_snapshots__/__diff_output__/global-experimental-features-diff.png

  32 |     const experimentalButton = await portfolioPage.drawerExperimentalButton();
  33 |     await experimentalButton.click();
> 34 |     expect(await app.client.screenshot()).toMatchImageSnapshot({
     |                                           ^
  35 |       customSnapshotIdentifier: "global-experimental-features",
  36 |     });
  37 |   });

  at Object.<anonymous> (tests/specs/global.spec.js:34:43)

› 1 snapshot failed.
PASS tests/specs/updater.spec.js (109.53 s)
PASS tests/specs/settings/passwordlock.spec.js (121.832 s)
PASS tests/specs/walletconnect.spec.js (90.14 s)
PASS tests/specs/accounts/subaccount.spec.js (219.564 s)
PASS tests/specs/manager.spec.js (73.429 s)
FAIL tests/specs/settings/settings.spec.js (57.044 s)
● Settings › go to settings -> experimental

Expected image to be the same size as the snapshot (1024x1264), but was different (1024x1180).
See diff for details: /home/runner/work/ledger-live-desktop/ledger-live-desktop/tests/specs/__image_snapshots__/__diff_output__/settings-experimental-diff.png

  37 |   it("go to settings -> experimental", async () => {
  38 |     await settingsPage.goToExperimentalTab();
> 39 |     expect(await app.client.screenshot()).toMatchImageSnapshot({
     |                                           ^
  40 |       customSnapshotIdentifier: "settings-experimental",
  41 |     });
  42 |   });

  at Object.<anonymous> (tests/specs/settings/settings.spec.js:39:43)

› 1 snapshot failed.
PASS tests/specs/accounts/ethereum.spec.js (300.987 s)
PASS tests/specs/settings/developermode.spec.js (314.659 s)
PASS tests/specs/accounts/bitcoin.spec.js (286.991 s)
PASS tests/specs/accounts/cosmos.spec.js (292.152 s)
PASS tests/specs/accounts/global.spec.js (244.51 s)
PASS tests/specs/accounts/tezos.spec.js (295.018 s)
PASS tests/specs/accounts/xrp.spec.js (295.94 s)
PASS tests/specs/onboarding/nanoS.spec.js (253.999 s)
PASS tests/specs/onboarding/nanoX.spec.js (201.867 s)
PASS tests/specs/onboarding/blue.spec.js (198.638 s)

Snapshot Summary
› 2 snapshots failed from 2 test suites. Inspect your code changes or run yarn run spectron -u to update them.

Test Suites: 2 failed, 17 passed, 19 total
Tests: 2 failed, 393 passed, 395 total
Snapshots: 2 failed, 309 passed, 311 total
Time: 1016.353 s
Ran all test suites matching /tests/specs/.*.spec.js/i.
error Command failed with exit code 1.

Diff output ❌

global-experimental-features

settings-experimental

Suggested snapshots to update

Please sign in to comment.