Skip to content

Add integration test for email-to-map database pipeline#582

Merged
claude-ERA merged 1 commit intomainfrom
test/map-pipeline-integration
Mar 1, 2026
Merged

Add integration test for email-to-map database pipeline#582
claude-ERA merged 1 commit intomainfrom
test/map-pipeline-integration

Conversation

@jonschull
Copy link
Owner

Summary

  • Fixes Add integration test for email-to-map database pipeline #577
  • Tests get_next_id() and insert_bigmap_entry() against the real database
  • Catches the RealDictCursor + placeholder syntax issues that broke production in Phase 5
  • Works on both SQLite (local) and PostgreSQL (Render) — isinstance(row, dict) pattern explicitly verified
  • Also validates API endpoint count matches DB row count

Test plan

  • 8/8 tests passing against local SQLite
  • Run with ERA2_DATABASE_URL set on Render to verify PostgreSQL path

🤖 Generated with Claude Code

Tests get_next_id() and insert_bigmap_entry() against the real database
to catch RealDictCursor and placeholder syntax issues that broke
production during Phase 5. Works on both SQLite (local) and PostgreSQL
(Render) — the isinstance(row, dict) pattern is explicitly verified.

8/8 tests passing on local SQLite.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jonschull jonschull added the auto-merge Non-strategy change: auto-merge after review PASS label Mar 1, 2026
@claude-ERA claude-ERA merged commit df4f6a1 into main Mar 1, 2026
1 check passed
@claude-ERA claude-ERA deleted the test/map-pipeline-integration branch March 1, 2026 23:40
@claude-ERA
Copy link
Collaborator

Auto-merged

PR #582 merged and deployed.

Smoke test: WARNING — see output

Verify: https://era-website.onrender.com

Test output
Traceback (most recent call last):
  File "/Users/admin/era2c/era2/website/tests/smoke_test.py", line 12, in <module>
    from playwright.sync_api import sync_playwright
ModuleNotFoundError: No module named 'playwright'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Non-strategy change: auto-merge after review PASS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add integration test for email-to-map database pipeline

2 participants