Skip to content

Updated MongoDB driver and some other packages#416

Merged
alexeyzimarev merged 4 commits intodevfrom
mongodb-update
Jun 24, 2025
Merged

Updated MongoDB driver and some other packages#416
alexeyzimarev merged 4 commits intodevfrom
mongodb-update

Conversation

@alexeyzimarev
Copy link
Copy Markdown
Contributor

No description provided.

@alexeyzimarev alexeyzimarev requested a review from Copilot June 20, 2025 12:12
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the MongoDB driver and several other package versions while also improving consistency in test configurations and refactoring variable names in projection tests.

  • Renames identifier parameters from "id" to "subscriptionId" in ProjectionTestBase for clearer intent.
  • Updates project and dependency versions across several files, including the MongoDB.Driver.
  • Removes outdated build targets and unused logging configuration overrides in test projects.

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/Mongo/test/Eventuous.Tests.Projections.MongoDB/ProjectionTestBase.cs Renamed constructor parameter & updated dependency injection of subscriptionId.
src/Mongo/test/Eventuous.Tests.Projections.MongoDB/ProjectingWithTypedHandlers.cs Formatted constructor for clarity.
src/Mongo/test/Eventuous.Tests.Projections.MongoDB/ProjectWithBuilder.cs Added a new test path for BookingCancelled events and adjusted disposal order.
src/Mongo/src/Eventuous.Projections.MongoDB/Options.cs Changed accessibility of Options by removing the public modifier.
src/Mongo/src/Eventuous.Projections.MongoDB/MongoProjector.cs Added XML documentation for new asynchronous handler methods.
src/Extensions/test/Eventuous.Tests.DependencyInjection/*.csproj and *.cs Updated package versions and adjusted namespaces for clarity.
src/Directory.Testable.targets Removed an outdated workaround target.
src/Core/test/Eventuous.Tests.Subscriptions.Base/Fixtures/SubscriptionFixtureBase.cs Removed a redundant ConfigureLogging override.
Directory.Packages.props Updated various package version numbers.
Comments suppressed due to low confidence (2)

src/Mongo/src/Eventuous.Projections.MongoDB/Options.cs:6

  • The public modifier was removed from Options, changing its accessibility from public to internal. Please confirm if restricting its usage was intended.
static class Options<TOptions> where TOptions: new() {

src/Mongo/test/Eventuous.Tests.Projections.MongoDB/ProjectWithBuilder.cs:53

  • The disposal of projectionFixture has been deferred until after the third Act call. Please confirm that this change in disposal order is intentional and does not affect subsequent test behavior.
        await projectionFixture.DisposeAsync();

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 20, 2025

Test Results

 42 files  + 28   42 suites  +28   3h 18m 21s ⏱️ + 2h 53m 18s
195 tests +  2  177 ✅  -  16  0 💤 ±0  18 ❌ +18 
585 runs  +390  565 ✅ +370  0 💤 ±0  20 ❌ +20 

For more details on these failures, see this check.

Results for commit d3813be. ± Comparison against base commit c4ca432.

This pull request removes 4 and adds 6 tests. Note that renamed tests count towards both.
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateByResolve
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateWithExplicitFunction
Eventuous.Tests.AspNetCore.AggregateFactoryRegistrationTests ‑ ShouldCreateTwoSeparateInstances
Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(CommitPositionSequence, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2025-06-20T17:09:14.9096006+00:00 })
Eventuous.Tests.DependencyInjection.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateByResolve
Eventuous.Tests.DependencyInjection.AggregateFactoryRegistrationTests ‑ ShouldCreateNewAggregateWithExplicitFunction
Eventuous.Tests.DependencyInjection.AggregateFactoryRegistrationTests ‑ ShouldCreateTwoSeparateInstances
Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(CommitPositionSequence, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2025-06-20T13:28:07.4247678+00:00 })
Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(CommitPositionSequence, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2025-06-21T15:11:33.4614364+00:00 })
Eventuous.Tests.Subscriptions.SequenceTests ‑ ShouldReturnFirstBefore(CommitPositionSequence, CommitPosition { Position: 0, Sequence: 2, Timestamp: 2025-06-21T15:11:37.1576553+00:00 })

♻️ This comment has been updated with latest results.

@alexeyzimarev alexeyzimarev merged commit 077ca21 into dev Jun 24, 2025
3 of 8 checks passed
@alexeyzimarev alexeyzimarev deleted the mongodb-update branch June 24, 2025 13:47
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.

2 participants