fix: evm fee drawer correcly updates tx after changing fee strat (aligned with BTC) #7755
β Success
Perform end to end and unit tests, type checks and run the linter on the Ledger Live Desktop application.
π Workflow run
TypeCheck
Typechecks are fine
- β
Type checks ended with status
success
Unit Tests (Jest)
Unit tests are fine
- β
Unit tests ended with status
success
Screenshot Tests (Playwright)
π§ Linux |
---|
β (success) |
Screenshots: β
There are no changes in the screenshots for this PR. If this is expected, you are good to go.
Details
π‘ Basics Tips
To share some piece of advice and help the community, please do not hesitate to edit this file and submit a pull request. Thank you! π
If you're troubleshooting a build failure, here are a few tips to keep in mind.
Note: you should always navigate to the workflow run page linked above to get additional information about the build failure.
β οΈ Read the logs β οΈ
Start by carefully reading the logs. In most cases (99% of the time), this will be enough to understand the cause of the failure.
Determine if the failure is os-specific
LLD is tested on Linux, macOS, and Windows, and some errors may only occur on a single platform.
Identify the specific step where the error occurred
The LLD test process consists of the following actions that are performed (in a nutshell):
-
Unit testing
- Checkout the repository. βΉοΈ For pull requests, it does a checkout of the branch targeted by the pull request, merges the pull request branch into it and then runs the workflow from that merge commit. It does NOT run at the HEAD commit of the pull request branch. So if you have different results locally make sure to checkout, fetch and pull the target branch and rebase your branch onto it.
- Install the system dependencies
- Install the javascript dependencies
- Build the dependencies of LLD from source
- Runs the
test:jest
script
-
End to End testing (For linux, macOS and windows - in parallel)
- Checkout the repository
- Install the system dependencies
- Install the javascript dependencies
- Build the dependencies of LLD from source
- Bundles the LLD javascript (
build:testing
script) and the SDK dummy app - Tests with playwright (
test:playwright
script) - Uploads the playwright results as artifacts
Look for for the source of the error in the logs.
Use the keyboard shortcut Cmd + F on macOS or Ctrl + F on Windows and Linux to search for the relevant error message.
The issue might be unrelated to the LLD codebase, but rather related to a dependency like live-common
or ledgerjs
for instance.
The error message should be enough to understand what went wrong. If it's not the case, then you should try to reproduce the error locally.
Screenshots mismatch
Most of the time the CI fails because playwright detected a change while performing Visual Comparisons.
The check run summary should contain a report (including snapshot diff images) to help understanding what happened.
If you are sure that the changes are expected, you can update the snapshots by commenting /generate-screenshots
on the pull request or clicking the Regen. Screenshots
button at the top of this page.
It will trigger a workflow that will update the snapshots on all operating systems and push the changes to the pull request after completion.
To run playwright locally you can use the pnpm desktop test:playwright
command - but you will need to bundle the app using pnpm desktop build
(or pnpm desktop build:testing
) beforehand.
Unit tests issue
You can run jest locally by using the pnpm desktop test:jest
command.
Check other runs of the same workflow
Check other runs of the same workflow. If other branches are experiencing the same issue, it's likely that the error is not specific to your code.
Review the workflow code.
The workflow YAML file provides information about the commands that are run on the continuous integration (CI) environment.
You should be able to reproduce locally by simply running the commands in the same order.
Infrastucture issues
Consider the possibility of infrastructure issues.
If the error appears to be related to a slow network or a physical machine, it's possible that the problem is not related to the code. In this case, wait a bit and restart the job, or file an issue if the problem persists.
Annotations
Check warning on line 113 in apps/ledger-live-desktop/tests/component/layout.component.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 84 in apps/ledger-live-desktop/tests/fixtures/common.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 18 in apps/ledger-live-desktop/tests/misc/reporters/step.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 23 in apps/ledger-live-desktop/tests/misc/reporters/step.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 5 in apps/ledger-live-desktop/tests/utils/customJsonReporter.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 6 in apps/ledger-live-desktop/tests/utils/customJsonReporter.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 6 in apps/ledger-live-desktop/tests/utils/fileUtils.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 113 in apps/ledger-live-desktop/tests/component/layout.component.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 84 in apps/ledger-live-desktop/tests/fixtures/common.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 18 in apps/ledger-live-desktop/tests/misc/reporters/step.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 23 in apps/ledger-live-desktop/tests/misc/reporters/step.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 5 in apps/ledger-live-desktop/tests/utils/customJsonReporter.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 6 in apps/ledger-live-desktop/tests/utils/customJsonReporter.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.
Check warning on line 6 in apps/ledger-live-desktop/tests/utils/fileUtils.ts
live-github-bot / @Desktop β’ Test App
@typescript-eslint/no-explicit-any
Unexpected any. Specify a different type.