Skip to content

Conversation

@jtomaszewski
Copy link

Summary

  • Add comprehensive test suite for the MikroORM driver using Vitest and real PostgreSQL database
  • Add GitHub Actions workflow for CI testing with PostgreSQL service
  • Fix integer overflow for timestamp fields by using bigint column type

Changes

  • Set up Vitest with SWC for decorator metadata support
  • Add test utilities for spinning up isolated NestJS apps with fresh PostgreSQL databases
  • Add unit tests for MikroOrmInboxOutboxTransportEvent entity (10 tests)
  • Add unit tests for MikroORMDatabaseDriver (11 tests)
  • Add unit tests for MikroORMDatabaseDriverFactory (4 tests)
  • Add integration tests for TransactionalEventEmitter with real database (11 tests)
  • Add test-mikroorm-driver.yml GitHub Actions workflow

Test plan

  • All 36 tests pass locally
  • GitHub Actions workflow passes with PostgreSQL service

🤖 Generated with Claude Code

jtomaszewski and others added 3 commits December 3, 2025 00:54
- Set up Vitest with SWC for decorator metadata support
- Add test utilities for spinning up isolated NestJS apps with fresh PostgreSQL databases
- Add unit tests for MikroOrmInboxOutboxTransportEvent entity
- Add unit tests for MikroORMDatabaseDriver (persist, remove, flush, findAndExtendReadyToRetryEvents)
- Add unit tests for MikroORMDatabaseDriverFactory (forked entity managers, isolation)
- Add integration tests for TransactionalEventEmitter with real database
- Fix integer overflow for timestamps by using bigint column type

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add test-mikroorm-driver.yml workflow with PostgreSQL service
- Revert @mikro-orm versions to ^6.3.9 as requested

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jtomaszewski jtomaszewski marked this pull request as ready for review December 3, 2025 00:13
… tests

Import and use the TransactionalEventEmitterOperations enum instead of
string literals ('persist' as const, 'remove' as const) in integration tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
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