feat: Azure Cosmos DB outbox repository (NetEvolve.Pulse.CosmosDb)#422
feat: Azure Cosmos DB outbox repository (NetEvolve.Pulse.CosmosDb)#422
Conversation
15828ef to
75ca57d
Compare
|
@copilot Add Unit and Integration Tests |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #422 +/- ##
==========================================
+ Coverage 92.43% 92.62% +0.18%
==========================================
Files 159 164 +5
Lines 5960 6302 +342
Branches 539 561 +22
==========================================
+ Hits 5509 5837 +328
- Misses 298 306 +8
- Partials 153 159 +6 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Added in commit c639ca8: Unit tests (
Integration tests (
|
Agent-Logs-Url: https://github.com/dailydevops/pulse/sessions/8cc0eb88-c126-47a6-9ff3-96f416746c0b Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Agent-Logs-Url: https://github.com/dailydevops/pulse/sessions/8cc0eb88-c126-47a6-9ff3-96f416746c0b Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Add <AzureCosmosDisableNewtonsoftJsonCheck>true</AzureCosmosDisableNewtonsoftJsonCheck> to both integration and unit test .csproj files. This disables the explicit-reference check for Newtonsoft.Json in Azure Cosmos DB SDK, as these projects use System.Text.Json via CosmosSystemTextJsonSerializer.
Consolidate CosmosDb outbox classes under NetEvolve.Pulse.Outbox namespace. Update XML doc references to match new namespace. Refactor CosmosDbOutboxRepository to iterate over item IDs for deletion. Remove unused using directives for cleaner code.
Agent-Logs-Url: https://github.com/dailydevops/pulse/sessions/d31a8c5a-325d-4a30-be5b-45c7b12889dc Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
Agent-Logs-Url: https://github.com/dailydevops/pulse/sessions/6d32b9f0-7ff3-4f18-934d-989dd7532723 Co-authored-by: samtrion <3283596+samtrion@users.noreply.github.com>
3cf9291 to
7634142
Compare
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 30 minutes and 4 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (19)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
src/NetEvolve.Pulse.CosmosDb/project structureCosmosDbOutboxOptions,CosmosDbOutboxDocument,CosmosDbOutboxRepository,CosmosDbOutboxManagement,CosmosDbExtensionsMicrosoft.Azure.Cosmos3.58.0,Newtonsoft.Json13.0.4, andTestcontainers.CosmosDb4.11.0 toDirectory.Packages.propsCosmosDbExtensionsTests,CosmosDbOutboxRepositoryTests,CosmosDbOutboxManagementTests) — all passingCosmosDbOutboxTests) usingTestcontainers.CosmosDbwith the Linux emulator imageCosmosDbContainerFixture,CosmosDbDatabaseServiceFixture,CosmosDbOutboxInitializerto integration test infrastructureCosmosDbContainerFixturewith/*dockerimage*/comment, consistent with other container fixtures