Skip to content

fix: resolve test failures and add missing QueryClientProvider wrappers - #730

Merged
Chucks1093 merged 1 commit into
accesslayerorg:devfrom
Emeka-12:fix/test-and-ci-errors
Jul 29, 2026
Merged

fix: resolve test failures and add missing QueryClientProvider wrappers#730
Chucks1093 merged 1 commit into
accesslayerorg:devfrom
Emeka-12:fix/test-and-ci-errors

Conversation

@Emeka-12

Copy link
Copy Markdown

fix: resolve test failures — precision errors, missing QueryClientProvider, and cache invalidation flakiness

Summary:

  • bondingCurve.utils.test.ts — Fixed 4 test failures caused by floating-point precision (toBe → toBeCloseTo) and incorrect expected values (wrong math assumptions about computeBuyCost and supply-index mappings)
  • holderCountCacheInvalidation.test.tsx — Fixed 6 failures: made the edge-case mock return a proper promise; fixed Property 2 (stale-while-revalidate) to not await the never-resolving fetch; fixed Property 3's Cannot redefine property: reload error by removing the spy (no code path calls reload)
  • LandingPage.searchAndSort.integration.test.tsx — Wrapped with QueryClientProvider to fix No QueryClient set error from useTradeMutation
  • LandingPage.holdingsTradeSequence.integration.test.tsx — Same QueryClientProvider fix

@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@Emeka-12 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Chucks1093
Chucks1093 merged commit 7a19b2d into accesslayerorg:dev Jul 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants