Skip to content

fix: resolve failing tests and type errors across monorepo - #203

Open
stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier3-2221-1791166795
Open

stooit wants to merge 1 commit into
mainfrom
quantcode/e2e-tier3-2221-1791166795

Conversation

@stooit

@stooit stooit commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Makes bun test (13/13) and tsc --noEmit pass. No test files or deps changed.

  • bunfig.toml: use Bun preload for happy-dom setup (Vitest-style environment key was ignored)
  • tsconfig.json: add bun-types so bun:test resolves
  • utils formatDate: en-AU with unpadded day (1/03/2024)
  • ui Button: apply aria-label; icon-only fallback + dev warning (test requires non-null name)
  • ui DataTable: functional setSortDir to avoid stale closure
  • web api.ts: import renamed useDebounce instead of useThrottle

Assumptions: kept en-AU locale (formatToParts to drop padding). DataTable change not strictly required by tests but more correct. Generic "Button" aria-label fallback is weak a11y — a required-prop type for icon-only buttons is a suggested follow-up.

- wire happy-dom via bun preload, add bun-types to tsconfig
- apply aria-label on Button (fallback for icon-only)
- use functional state update for DataTable sort toggle
- format dates as en-AU with unpadded day
- import renamed useDebounce hook in apps/web
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant