Promote dev → uat: test fixes (CAR-1006) #33

Merged

Savannah Savings merged 4 commits from dev into uat 2026-05-23 23:43:08 +00:00

Conversation 0 Commits 4 Files Changed 4

+1365 -7

Savannah Savings commented 2026-05-23 23:42:59 +00:00

Member

Copy Link

Copy Source

Promotion: dev → uat

Content: Test failure fixes merged to dev (PR #29)

What's included

• PR #29 — Fix test failures: email_inbound_token server_default for SQLite
  • Adds email_inbound_token to test user INSERT statements
  • Generates unique token values to avoid UNIQUE constraint violations
  • Wrapped SQL strings to pass ruff lint

Target

For CAR-1006 — verify CI registry migration on UAT

## Promotion: dev → uat **Content**: Test failure fixes merged to dev (PR #29) ### What's included - PR #29 — Fix test failures: email_inbound_token server_default for SQLite - Adds email_inbound_token to test user INSERT statements - Generates unique token values to avoid UNIQUE constraint violations - Wrapped SQL strings to pass ruff lint ### Target For CAR-1006 — verify CI registry migration on UAT

Savannah Savings added 4 commits 2026-05-23 23:43:00 +00:00

fix: resolve email_inbound_token conflict in test fixtures c9fd066c31

fix: resolve email_inbound_token conflict in test fixtures ... cc6ca5982c

Rebase on latest dev and wrap SQL INSERT lines to honor ruff line-length=100.

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

Fix: strip PostgreSQL server_default from email_inbound_token for SQLite ... 0e3c9fb52e

The email_inbound_token column uses a PostgreSQL-only server_default
(gen_random_bytes/encode/trim) that SQLite cannot parse.
Strip the server_default before metadata.create_all() in both the
sync engine and async db_engine fixtures so tests run against SQLite.

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

Merge pull request 'Fix test failures: email_inbound_token server_default for SQLite' (#29) from betty/fix-email-inbound-token-tests into dev ... 280882f515

Fix test failures: email_inbound_token server_default for SQLite (#29)

Strip PostgreSQL-only server_default from email_inbound_token before SQLite create_all(). Add email_inbound_token to test user INSERT statements.

Reviewed-by: Savannah Savings (CTO)
Approved-by: Checkout Charlie (QA)

Savannah Savings merged commit 0c0cc63d59 into uat 2026-05-23 23:43:08 +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) cs_betty (Barcode Betty) cs_charlie (Checkout Charlie) cpfarhood (Chris Farhood) ci (Continuous Integration [bot]) cs_carl (Coupon Carl) cs_dottie (Deal Dottie) flux (Flux CD) admin (Gitea Admin) cs_martha (Markdown Martha) renovate (Mend Renovate) cs_savannah (Savannah Savings) cs_steve (Stockboy Steve)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: cartsnitch/api#33