refactor: extract API route test harness #381

Merged

barrettruth merged 1 commit from issue-366-api-route-test-harness into main 2026-05-12 13:05:36 +00:00

Conversation 0 Commits 1 Files changed 5 +209 -191

barrettruth commented 2026-05-11 21:35:01 +00:00

Owner

Copy link

Problem

• API route tests repeated DB/auth/encryption/request JSON setup.

Solution

• Added tests/api/helpers.ts for shared API route DB/auth/encryption setup and request/response helpers.
• Migrated geocode, NLP parse recurrence, settings NLP, and settings integrations route tests.
• Kept route-specific assertions in the individual tests.

Verification

• nix develop --command pnpm vitest run tests/api/geocode.test.ts tests/api/nlp-parse-recurrence.test.ts tests/api/settings-nlp.test.ts tests/api/settings-integrations.test.ts --reporter=verbose
• nix develop --command pnpm vitest run tests/api --reporter=verbose
• nix develop --command just lint
• nix develop --command just test
• nix develop --command just build

Closes #366

Problem - API route tests repeated DB/auth/encryption/request JSON setup. Solution - Added tests/api/helpers.ts for shared API route DB/auth/encryption setup and request/response helpers. - Migrated geocode, NLP parse recurrence, settings NLP, and settings integrations route tests. - Kept route-specific assertions in the individual tests. Verification - nix develop --command pnpm vitest run tests/api/geocode.test.ts tests/api/nlp-parse-recurrence.test.ts tests/api/settings-nlp.test.ts tests/api/settings-integrations.test.ts --reporter=verbose - nix develop --command pnpm vitest run tests/api --reporter=verbose - nix develop --command just lint - nix develop --command just test - nix develop --command just build Closes #366

barrettruth added 1 commit 2026-05-11 21:35:01 +00:00

refactor: extract API route test harness 105867c85f

barrettruth force-pushed issue-366-api-route-test-harness from 105867c85f to 9288fa2570 2026-05-12 12:57:47 +00:00 Compare

barrettruth force-pushed issue-366-api-route-test-harness from 105867c85f to 9288fa2570 2026-05-12 12:58:42 +00:00 Compare

barrettruth merged commit 237fe6a705 into main 2026-05-12 13:05:36 +00:00

barrettruth deleted branch issue-366-api-route-test-harness 2026-05-12 13:05:36 +00:00

barrettruth referenced this pull request from a commit 2026-05-12 13:05:36 +00:00

refactor: extract API route test harness (#381)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

bug

Something isn't working

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

enhancement

New feature or request

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

question

Further information is requested

  

status:blocked

Blocked on a decision, credential, or external dependency

  

track:api

API layer track

  

track:auto

Automation track

  

track:core

Core engine track

  

track:deploy

Deployment track

  

track:infra

Infrastructure track

  

track:ui

Web UI track

  

type:cleanup

Small cleanup, polish, or consolidation work

  

type:docs

Documentation and operator guidance

  

type:epic

Umbrella issue that coordinates related work

  

type:release

Release, packaging, and versioning work

  

type:research

Decision or investigation work before implementation

  

wontfix

This will not be worked on

No labels

bug

documentation

duplicate

enhancement

good first issue

help wanted

invalid

question

status:blocked

track:api

track:auto

track:core

track:deploy

track:infra

track:ui

type:cleanup

type:docs

type:epic

type:release

type:research

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

barrettruth/delta!381

No description provided.