GRO-1949: add behavioral and skin medicalAlertPool types, deterministic seeding for TestCooper/TestRocky #109

Merged

Scrubs McBarkley merged 6 commits from fleaflicker/gro-medical-alert-types-behavioral-skin into dev 2026-05-30 04:12:06 +00:00

Conversation 0 Commits 6 Files Changed 4

+270 -108

Scrubs McBarkley commented 2026-05-30 04:11:07 +00:00

Owner

Copy Link

Copy Source

Extends medicalAlertPool with behavioral and skin alert types; makes TestCooper get deterministic behavioral alert and TestRocky get deterministic skin alert per GRO-1937 ACs.

cc @cpfarhood

Extends medicalAlertPool with behavioral and skin alert types; makes TestCooper get deterministic behavioral alert and TestRocky get deterministic skin alert per GRO-1937 ACs. cc @cpfarhood

Scrubs McBarkley added 5 commits 2026-05-30 04:11:07 +00:00

fix(seed): use typeof on enum.enumValues for db build ... 4df7d96020

TS2749: enumValues is a value, not a type — wrap with typeof before
indexing.

Also extends Lint & Typecheck CI job to run pnpm --filter @groombook/db
typecheck so this class of error is caught at lint time rather than
Docker build time.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

fix(test): mock db to handle sql count(*) queries and async iteration ... bf064b3ada

The petProfileSummary mock's sql tag returned a plain string instead of
a proper Drizzle SQL object, so count(*) queries via .as("count") failed.
Also added Symbol.asyncIterator support for for-await-of patterns used
in the pets router.

Fixes: GRO-1917

Co-Authored-By: Paperclip <noreply@paperclip.ing>

fix(seed): add uat-customer client record for SSO bridge UAT (GRO-1935) ... 4a0dd5ed2a

- Add UAT Customer client row (id: c0000001-0000-0000-0000-000000000001)
  with email uat-customer@groombook.dev in seedKnownUsers()
- Add two UAT Customer pets (UAT Pup Alpha, UAT Pup Beta) with stable IDs
- Add test case covering 201 response with correct clientId/clientName
  for uat-customer SSO bridge flow
- Explicit comment clarifying uat-groomer/uat-super are staff, not clients

Co-Authored-By: Paperclip <noreply@paperclip.ing>

GRO-1921: seedUatStaffAccounts() shared fn — full UAT seed honors numeric OIDC subs ... 473868579b

Extract UAT staff account seeding into a shared async function so it
runs in both seedKnownUsers() and the full seed() UAT branch.

Before this change the full seed() UAT path never created the
deterministic UAT staff (UAT Super/Staff/Groomer) with their numeric
oidcSub values from SEED_UAT_*_OIDC_SUB env vars — seedKnownUsers()
had that logic but was bypassed by SEED_KNOWN_USERS_ONLY=true in the
UAT reset CronJob.

seedUatStaffAccounts() handles:
- UAT Super Staff (SEED_UAT_SUPER_OIDC_SUB)
- UAT Staff Groomer (SEED_UAT_STAFF_OIDC_SUB)
- UAT Groomer Personas (SEED_UAT_GROOMER_EMAILS + _NAMES)
- Better Auth email+password credentials (SEED_UAT_*_PASSWORD)
- UAT Customer client + 2 pets

Co-Authored-By: Paperclip <noreply@paperclip.ing>

GRO-1949: add behavioral and skin medicalAlertPool types, deterministic seeding for TestCooper/TestRocky ... cc63299415

- Extend medicalAlertPool with 4 new entries: 2 behavioral, 2 skin
- Make TestCooper (uat-charlie) get a deterministic behavioral alert
- Make TestRocky (uat-delta) get a deterministic skin alert
- Other UAT pets retain random alert assignment

Co-Authored-By: Paperclip <noreply@paperclip.ing>

The Dogfather added 1 commit 2026-05-30 04:11:29 +00:00

fix: correct behavioral alert severity to low (matches TC-API-3.23 expectation) ... 558c51b357

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Scrubs McBarkley merged commit 1891b9c523 into dev 2026-05-30 04:12:06 +00:00

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

wontfix

This will not be worked on

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

ai-review (AI Review) gb_barkley (Barkley Trimsworth) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) gb_flea (Flea Flicker) flux (Flux CD) admin (Gitea Admin) gb_lint (Lint Roller) renovate (Mend Renovate) gb_pawla (Pawla Abdul) gb_scrubs (Scrubs McBarkley) gb_shedward (Shedward Scissorhands) gb_dogfather (The Dogfather)

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: groombook/api#109