test: add utility and env test coverage (Tier 1) by gabitoesmiapodo · Pull Request #415 · BootNodeDev/dAppBooster

gabitoesmiapodo · 2026-03-23T21:17:42Z

Summary

Adds tests for strings.ts, getExplorerLink.ts, address.ts utilities
Adds env.ts Zod schema validation tests
Creates src/test-utils.tsx with shared test helpers

Test plan

pnpm test passes on this branch
All new utility tests cover the public API surface used by wagmi/viem-dependent code

Part of test safety net before dependency updates. See plan for full context.

vercel · 2026-03-23T21:17:47Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
components.dappbooster	Ready	Preview, Comment	Mar 25, 2026 6:50pm
demo.dappbooster	Ready	Preview	Mar 25, 2026 6:50pm
docs.dappbooster	Ready	Preview, Comment	Mar 25, 2026 6:50pm

Copilot

Pull request overview

Adds a Tier 1 testing safety net around several utility modules and environment parsing so upcoming dependency updates have better regression coverage.

Changes:

Added Vitest unit tests for strings, getExplorerLink, and address utilities.
Added Vitest coverage for env.ts Zod/@t3-oss/env-core validation behavior using a committed .env.test.
Introduced src/test-utils.tsx with shared helpers (providers wrapper + chain/web3 mocks).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/utils/strings.test.ts	Adds unit tests for string truncation helpers.
src/utils/getExplorerLink.test.ts	Adds unit tests for explorer URL generation logic.
src/utils/address.test.ts	Adds unit tests for native-token address detection (env-driven).
src/test-utils.tsx	Adds shared test helpers (render wrapper + mocks).
src/env.test.ts	Adds unit tests asserting env schema output under Vitest mode.
.env.test	Adds test-mode env vars required by the env schema.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/env.test.ts

src/test-utils.tsx

src/utils/getExplorerLink.test.ts

Copilot · 2026-03-23T21:21:38Z

src/utils/getExplorerLink.test.ts

+  it('returns tx URL using chain block explorer for a hash', () => {
+    const url = getExplorerLink({ chain, hashOrAddress: txHash })
+    expect(url).toBe(`${chain.blockExplorers?.default.url}/tx/${txHash}`)
+  })


Same issue as the address-case: the assertion uses chain.blockExplorers?.default.url, which would still produce a matching (but broken) undefined/tx/... string if the chain mock lacks a block explorer. Using the explicit mock URL will make this test meaningfully fail on regressions.

Fixed: same as above.

from this comment I think that best is to throw when chain does not exist, so we prevent the undefined/<tx|address> invalid route in runtime and throw.

I mean, this surfaced an issue with the util's implementation.

If !explorerUrl && !chain.blockExplorers?.default then throw. I remember when implementing this I adopted the naïve approach of reviewing the different definitions here: https://github.com/wevm/viem/blob/main/src/chains/index.ts to see if all chains have "default" explorer configured.

That will mean adding a new test for that particular case.

Fixing the missing-explorer behavior (returning undefined/... vs throwing) is a valid improvement but requires changing getExplorerLink.ts — outside the scope of this test PR. The test at line 42 already documents the safe path when explorerUrl is explicitly provided. A separate fix PR for the implementation is the right place for this.

@fernandomg Fixed in #426.

src/env.test.ts

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/utils/getExplorerLink.test.ts

src/env.test.ts

src/utils/address.test.ts

Add test safety net before dependency updates. New files: - src/test-utils.tsx: renderWithProviders, createMockWeb3Status, createMockChain - src/utils/strings.test.ts: truncateStringInTheMiddle and getTruncatedHash (13 tests) - src/utils/getExplorerLink.test.ts: getExplorerLink (6 tests) - src/utils/address.test.ts: isNativeToken (5 tests) - src/env.test.ts: Zod-validated env schema (9 tests) - .env.test: Vitest environment variables for required PUBLIC_ fields

- Fix env.test.ts test names that incorrectly claimed to test defaults while .env.test was setting those vars explicitly - Assert exact value for PUBLIC_WALLETCONNECT_PROJECT_ID (was typeof check) - Fix createMockWeb3Status shape to match actual Web3Status interface (was using isConnected/chainId/publicClient; hook uses isWalletConnected/ walletChainId/readOnlyClient) - Hardcode expected explorer URL in getExplorerLink assertions instead of deriving from mock chain (prevents false positives if mock misconfigured)

BigNumberInput: add 9 tests covering placeholder, disabled state, onChange with parsed bigint, onError for min/max violations, decimal precision enforcement, and input clearing. HashInput: add 7 tests covering placeholder, typed value reflection, initial value, clear to null, debounced search callback, onLoading lifecycle, and custom renderInput.

- Rename misleading test names in BigNumberInput (onError absence test, maxUint256 constraint test) - Add beforeEach mock clear in HashInput to prevent cross-test contamination from unconsumed mockResolvedValueOnce calls - Add debounceTime: 0 to clear test to avoid real-timer dependency - Rename debounce test to accurately describe what it covers; fake-timer debounce testing conflicts with waitFor + userEvent in jsdom

- Use @/src/env alias import in env.test.ts (matches repo convention) - Rename 'checksummed zero address' test to 'zero address string literal' (no checksum casing difference in this value; name was misleading)

- useWeb3Status.test.ts: 8 tests covering disconnected/connected state, isWalletSynced, switchingChain, disconnect, switchChain, appChainId - useWeb3StatusConnected.test.ts: 2 tests (co-located in same file) covering throw-on-disconnect and pass-through when connected - useErc20Balance.test.ts: 5 tests covering missing address/token, native token skip, successful balance fetch, and error handling - useTokenLists.test.ts: 4 tests covering return shape, deduplication, native token injection, and schema validation filtering

- Add beforeEach mock clearing in useWeb3Status to prevent shared mock call history leaking between tests - Switch mockReturnValue to mockReturnValueOnce for the connected test in useWeb3StatusConnected (hook calls useWeb3Status twice internally) - Add beforeEach mockReadContract.mockClear() in useErc20Balance to prevent cross-test false negatives on not.toHaveBeenCalled assertions

New tests for shared components and utilities: - TokenLogo: img src/alt/size, IPFS URL conversion, placeholder on error - TransactionButton: wallet states, pending label, onMined callback - SwitchNetwork: network display, disabled state, menu item rendering - WalletStatusVerifier: connect fallback, wrong chain, synced renders children - withWalletStatusVerifier HOC: fallback and pass-through behavior - withSuspense/withSuspenseAndRetry: Suspense fallback, error message, retry Also adds: - ResizeObserver mock to setupTests.ts (required by @floating-ui in jsdom) - .env.test and src/test-utils.tsx (shared test utilities from tier 1)

- setupTests.ts: replace vi.fn() ResizeObserver with a real class guarded by a conditional check — vi.restoreAllMocks() can no longer clear it - suspenseWrapper.test.tsx: restore only the console.error spy in afterEach instead of vi.restoreAllMocks() which would wipe the ResizeObserver polyfill - TransactionButton.test.tsx: make useWaitForTransactionReceipt mock hash-aware so it only returns the receipt when called with the expected hash

Smoke tests for all demo page components to verify they render without crashing. Mocks external Web3 deps at module level. New test files: - home/index.test.tsx: Home renders Welcome + Examples sections - NotFound404.test.tsx: 404 page with mocked useNavigate - demos/ConnectWallet/index.test.tsx: mocked ConnectWalletButton - demos/EnsName/index.test.tsx: mocked useEnsName - demos/HashHandling/index.test.tsx: mocked useWeb3Status + detectHash - demos/SignMessage/index.test.tsx: shows fallback when wallet absent - demos/SwitchNetwork/index.test.tsx: shows fallback when wallet absent - demos/TransactionButton/index.test.tsx: shows fallback when wallet absent - demos/TokenDropdown/index.test.tsx: mocked BaseTokenDropdown - demos/TokenInput/index.test.tsx: mocked useTokenLists + useTokenSearch Also adds .env.test and src/test-utils.tsx (shared test utilities).

The module exports detectHash as a default export; the mock only provided a named export so HashInput would receive undefined and throw on invocation. Added both default and named exports to the mock factory.

- home/index.test.tsx: use renderWithProviders from test-utils - TransactionButton demo: use createMockWeb3Status for complete hook shape - TokenInput demo: fix useTokenInput mock to match the real hook return (amount/setAmount/amountError/balance/isLoadingBalance/selectedToken/setTokenSelected) and use renderWithProviders + createMockWeb3Status helpers

fernandomg

In regard to address.test.ts I would chose another value for the PUBLIC_NATIVE_TOKEN_ADDRESS so it makes the different tests cases valid, or I would just remove those unecessary

fernandomg · 2026-03-25T15:06:54Z

src/utils/address.test.ts

+  it('returns true for the zero address in lowercase', () => {
+    expect(isNativeToken(zeroAddress.toLowerCase())).toBe(true)
+  })


this doesn't make much sense

Fixed in a412c4f. Removed the test — zeroAddress.toLowerCase() equals zeroAddress so it was identical to the first case.

src/utils/address.test.ts

fernandomg · 2026-03-25T17:51:40Z

src/utils/getExplorerLink.test.ts

+  it('returns tx URL using chain block explorer for a hash', () => {
+    const url = getExplorerLink({ chain, hashOrAddress: txHash })
+    expect(url).toBe(`${chain.blockExplorers?.default.url}/tx/${txHash}`)
+  })


from this comment I think that best is to throw when chain does not exist, so we prevent the undefined/<tx|address> invalid route in runtime and throw.

I mean, this surfaced an issue with the util's implementation.

If !explorerUrl && !chain.blockExplorers?.default then throw. I remember when implementing this I adopted the naïve approach of reviewing the different definitions here: https://github.com/wevm/viem/blob/main/src/chains/index.ts to see if all chains have "default" explorer configured.

That will mean adding a new test for that particular case.

fernandomg · 2026-03-25T17:55:22Z

.env.test

gabitoesmiapodo · 2026-03-25T18:27:51Z

The missing throw case for getExplorerLink (no block explorer + no explorerUrl) is covered in #426 — both the implementation fix and the test. When both PRs land there will be a conflict in src/utils/getExplorerLink.test.ts that will need to be resolved by merging the two test files.

test: enhance BigNumberInput and HashInput coverage (Tier 2)

test: add hook test coverage (Tier 3)

test: add component test coverage (Tier 4)

test: add demo page smoke tests (Tier 5)

Copilot AI review requested due to automatic review settings March 23, 2026 21:17

Copilot started reviewing on behalf of gabitoesmiapodo March 23, 2026 21:18 View session

vercel bot deployed to Preview – components.dappbooster March 23, 2026 21:18 View deployment

vercel bot deployed to Preview – docs.dappbooster March 23, 2026 21:18 View deployment

gabitoesmiapodo mentioned this pull request Mar 23, 2026

feat: AI integration baseline #420

Open

5 tasks

vercel bot deployed to Preview – demo.dappbooster March 23, 2026 21:18 View deployment

Copilot AI reviewed Mar 23, 2026

View reviewed changes

vercel bot deployed to Preview – components.dappbooster March 23, 2026 22:04 View deployment

vercel bot deployed to Preview – docs.dappbooster March 23, 2026 22:04 View deployment

vercel bot deployed to Preview – demo.dappbooster March 23, 2026 22:05 View deployment

gabitoesmiapodo force-pushed the test/utils branch from d6f454e to cf0fdcb Compare March 23, 2026 22:16

vercel bot deployed to Preview – docs.dappbooster March 23, 2026 22:17 View deployment

vercel bot deployed to Preview – components.dappbooster March 23, 2026 22:17 View deployment

vercel bot deployed to Preview – demo.dappbooster March 23, 2026 22:17 View deployment

gabitoesmiapodo requested a review from Copilot March 23, 2026 22:18

Copilot started reviewing on behalf of gabitoesmiapodo March 23, 2026 22:18 View session

Copilot AI reviewed Mar 23, 2026

View reviewed changes

src/utils/getExplorerLink.test.ts Show resolved Hide resolved

src/env.test.ts Outdated Show resolved Hide resolved

src/utils/address.test.ts Outdated Show resolved Hide resolved

gabitoesmiapodo added 2 commits March 23, 2026 20:23

gabitoesmiapodo force-pushed the test/utils branch from cf0fdcb to 7edb47f Compare March 23, 2026 23:24

gabitoesmiapodo added 2 commits March 23, 2026 20:24

vercel bot deployed to Preview – components.dappbooster March 23, 2026 23:24 View deployment

vercel bot deployed to Preview – docs.dappbooster March 23, 2026 23:24 View deployment

vercel bot deployed to Preview – demo.dappbooster March 23, 2026 23:25 View deployment

test: address review feedback on Tier 1 test suite (round 2)

40e06d3

- Use @/src/env alias import in env.test.ts (matches repo convention) - Rename 'checksummed zero address' test to 'zero address string literal' (no checksum casing difference in this value; name was misleading)

vercel bot deployed to Preview – components.dappbooster March 23, 2026 23:54 View deployment

vercel bot deployed to Preview – docs.dappbooster March 23, 2026 23:54 View deployment

vercel bot deployed to Preview – demo.dappbooster March 23, 2026 23:54 View deployment

gabitoesmiapodo added 7 commits March 23, 2026 21:02

test: fix detectHash mock in HashHandling smoke test

4fe5f28

The module exports detectHash as a default export; the mock only provided a named export so HashInput would receive undefined and throw on invocation. Added both default and named exports to the mock factory.

fernandomg requested changes Mar 25, 2026

View reviewed changes

test: fix redundant cases in isNativeToken test suite

a412c4f

vercel bot deployed to Preview – components.dappbooster March 25, 2026 18:16 View deployment

vercel bot deployed to Preview – docs.dappbooster March 25, 2026 18:16 View deployment

vercel bot deployed to Preview – demo.dappbooster March 25, 2026 18:16 View deployment

chore: ignore .worktrees directory

990a656

gabitoesmiapodo requested a review from fernandomg March 25, 2026 18:28

vercel bot deployed to Preview – components.dappbooster March 25, 2026 18:31 View deployment

vercel bot deployed to Preview – docs.dappbooster March 25, 2026 18:31 View deployment

vercel bot deployed to Preview – demo.dappbooster March 25, 2026 18:31 View deployment

Merge pull request #416 from BootNodeDev/test/enhance-existing

ff358c9

test: enhance BigNumberInput and HashInput coverage (Tier 2)

vercel bot deployed to Preview – components.dappbooster March 25, 2026 18:36 View deployment

vercel bot deployed to Preview – docs.dappbooster March 25, 2026 18:36 View deployment

vercel bot deployed to Preview – demo.dappbooster March 25, 2026 18:36 View deployment

gabitoesmiapodo added 4 commits March 25, 2026 15:45

refactor(tests): use Address type from viem instead of template literal

5861213

Merge pull request #417 from BootNodeDev/test/hooks

11fb2a4

test: add hook test coverage (Tier 3)

Merge pull request #418 from BootNodeDev/test/components

9d1aff7

test: add component test coverage (Tier 4)

Merge pull request #419 from BootNodeDev/test/demo-smoke

cb38783

test: add demo page smoke tests (Tier 5)

vercel bot deployed to Preview – components.dappbooster March 25, 2026 18:49 View deployment

vercel bot deployed to Preview – docs.dappbooster March 25, 2026 18:50 View deployment

vercel bot deployed to Preview – demo.dappbooster March 25, 2026 18:50 View deployment

fernandomg approved these changes Mar 26, 2026

View reviewed changes

Conversation

gabitoesmiapodo commented Mar 23, 2026

Summary

Test plan

Uh oh!

vercel bot commented Mar 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

gabitoesmiapodo Mar 23, 2026

Choose a reason for hiding this comment

Uh oh!

fernandomg Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

gabitoesmiapodo Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

gabitoesmiapodo Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

fernandomg left a comment

Choose a reason for hiding this comment

Uh oh!

fernandomg Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

gabitoesmiapodo Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

fernandomg Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

fernandomg Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

gabitoesmiapodo commented Mar 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

vercel bot commented Mar 23, 2026 •

edited

Loading

gabitoesmiapodo Mar 25, 2026 •

edited

Loading