Skip to content

Commit

Permalink
fix: remove aptos unit test for function assertions (#8839)
Browse files Browse the repository at this point in the history
* fix: aptos e2e test with speculos

* fix: aptos bridge integration test

* fix: remove tobe(function) assertions

* fix: remove tobe(function) assertions from ledger account tests

* chore: rollback changes

* chore: rollback changes

* test: add unit tests for currencyBridge methods

* fix: bridge test

* fix: bridge test
  • Loading branch information
semeano authored and joaoccmartins committed Jan 14, 2025
1 parent 9ee65ec commit a5bb1fc
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion libs/ledger-live-common/src/families/aptos/bridge/js.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ describe("Aptos bridge interface ", () => {
});
});

describe("accountBridge ", () => {
describe("accountBridge", () => {
it("should contain all methods", () => {
expect(bridge.accountBridge.estimateMaxSpendable).toBeDefined();
expect(typeof bridge.accountBridge.estimateMaxSpendable).toBe("function");
Expand All @@ -53,8 +53,15 @@ describe("Aptos bridge interface ", () => {
expect(typeof bridge.accountBridge.signOperation).toBe("function");
expect(bridge.accountBridge.broadcast).toBeDefined();
expect(typeof bridge.accountBridge.broadcast).toBe("function");
expect(bridge.currencyBridge.hydrate).toBeDefined();
expect(typeof bridge.currencyBridge.hydrate).toBe("function");
expect(bridge.currencyBridge.preload).toBeDefined();
expect(typeof bridge.currencyBridge.preload).toBe("function");
expect(bridge.currencyBridge.scanAccounts).toBeDefined();
expect(typeof bridge.currencyBridge.scanAccounts).toBe("function");
});
});

describe("updateTransaction", () => {
it("should update the transaction with the given patch", () => {
const initialTransaction = {
Expand Down

0 comments on commit a5bb1fc

Please sign in to comment.