Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(data-stream): add publishing tests #48

Merged
merged 13 commits into from
Jul 13, 2024

Conversation

Jurshsmith
Copy link
Member

@Jurshsmith Jurshsmith commented Jul 9, 2024

Besides populating the tests related to publishing NATS subjects, this PR does the following:

  • Introduces Subject & SubjectName Enums to allow colocating all their string(unsafe) definitions
    and allow building each subject and their names in a type-safe manner.
  • Allows passing a connection_id optionally to contextualize each NATS connection. This will
    allow us to run tests in parallel and equally might serve some sandboxing use cases in the future.

@Jurshsmith Jurshsmith marked this pull request as draft July 9, 2024 15:19
@Jurshsmith Jurshsmith self-assigned this Jul 9, 2024
This trait describes the interface required for `fuel-core-nats` to extend `fuel-core`.
Equally, it will be useful for mocking fuel-core related components in our integration
tests.
A subject prefix could be empty ("") or some random string value. This helps
contextualize each text case and could come in handy for other sandboxing
usecases in the future.
@Jurshsmith Jurshsmith force-pushed the aj/feat/fuel-core-nats-publishing-tests branch from 176527f to 0d872ab Compare July 12, 2024 10:32
@Jurshsmith Jurshsmith marked this pull request as ready for review July 12, 2024 10:39
@pedronauck pedronauck enabled auto-merge (squash) July 13, 2024 02:34
@pedronauck pedronauck merged commit 766b3e8 into main Jul 13, 2024
6 checks passed
@pedronauck pedronauck deleted the aj/feat/fuel-core-nats-publishing-tests branch July 13, 2024 02:56
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.

None yet

2 participants