-
Notifications
You must be signed in to change notification settings - Fork 349
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(aptos): improve aptos unit tests #8781
base: feat/integrate-aptos
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
3 Skipped Deployments
|
Desktop Bundle ChecksMobile Bundle Checks
|
libs/ledger-live-common/src/families/aptos/getFeesForTransaction.test.ts
Show resolved
Hide resolved
libs/ledger-live-common/src/families/aptos/getFeesForTransaction.test.ts
Show resolved
Hide resolved
libs/ledger-live-common/src/families/aptos/getFeesForTransaction.test.ts
Show resolved
Hide resolved
}); | ||
|
||
describe("when key is in cache", () => { | ||
it("should return cached fee", async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here we need to change mock value on the second time, so we make sure we return the cached value and not current value
eg. we call the first time, then we change the mock value, we call it the second time, it should return the first mock value and not the second
β Checklist
npx changeset
was attached.π Description
Adds improvements to aptos unit tests
Coverage for the current test
β Context
π§ Checklist for the PR Reviewers