Skip to content

Conversation

@jtomaszewski
Copy link

Summary

  • Fix typo in field name delivedToListenersdeliveredToListeners throughout codebase
  • Add database migrations for both MikroORM and TypeORM drivers to rename the column
  • Update all tests to use the corrected field name

Breaking Change

Existing databases require running the new migration to rename the column from delived_to_listeners to delivered_to_listeners.

Files Changed

  • packages/core/src/model/inbox-outbox-transport-event.interface.ts
  • packages/core/src/poller/retryable-inbox-outbox-event.poller.ts
  • packages/core/src/processor/inbox-outbox-event.processor.ts
  • packages/mikroorm-driver/src/model/mikroorm-inbox-outbox-transport-event.model.ts
  • packages/typeorm-driver/src/model/typeorm-inbox-outbox-transport-event.model.ts
  • New migration files for both drivers

Test plan

  • Run existing test suite to verify no regressions
  • Test migration on existing database with old column name

🤖 Generated with Claude Code

jtomaszewski and others added 2 commits December 3, 2025 01:12
* test(mikroorm-driver): add comprehensive test suite for MikroORM driver

- 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>

* ci: add GitHub workflow for MikroORM driver tests

- 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>

* fix(ci): build core package before running MikroORM driver tests

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

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

---------

Co-authored-by: Claude <noreply@anthropic.com>
Fix typo in field name throughout codebase and add database migrations
to rename the column from delived_to_listeners to delivered_to_listeners.

BREAKING CHANGE: Existing databases require running the new migration
to rename the column.

🤖 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