Skip to content
This repository has been archived by the owner on May 8, 2023. It is now read-only.

Fix/rewrite outbox tests #1856

Merged
merged 8 commits into from
Nov 10, 2022
Merged

Fix/rewrite outbox tests #1856

merged 8 commits into from
Nov 10, 2022

Conversation

jonasdmoeller
Copy link
Member

Refactored OutboxProcessorEndpoint integration tests to not use function host, but instead mock calls to DomainEventPublisher.

@cla-bot cla-bot bot added the cla-signed label Nov 7, 2022
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 15:32 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 15:32 Inactive
@codecov
Copy link

codecov bot commented Nov 7, 2022

Codecov Report

Merging #1856 (8b1a9a5) into main (5c427d3) will decrease coverage by 0.65%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #1856      +/-   ##
==========================================
- Coverage   84.59%   83.93%   -0.66%     
==========================================
  Files         468      447      -21     
  Lines        8114     7889     -225     
  Branches      431       22     -409     
==========================================
- Hits         6864     6622     -242     
+ Misses       1164     1146      -18     
- Partials       86      121      +35     
Flag Coverage Δ
business 87.74% <50.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...e.Core/MessagingExtensions/ServiceBusDispatcher.cs 57.14% <0.00%> (ø)
...re.Core/InternalMessaging/DomainEventDispatcher.cs 100.00% <100.00%> (+38.46%) ⬆️
...ergyHub.Charges.Domain/Charges/ChargeIdentifier.cs 0.00% <0.00%> (-100.00%) ⬇️
...ain/Dtos/Events/MarketParticipantCreatedCommand.cs 14.28% <0.00%> (-85.72%) ⬇️
...es.Domain/Dtos/Events/GridAreaOwnerAddedCommand.cs 33.33% <0.00%> (-66.67%) ⬇️
...Dtos/Events/MarketParticipantNameChangedCommand.cs 33.33% <0.00%> (-66.67%) ⬇️
...os/Events/MarketParticipantStatusChangedCommand.cs 33.33% <0.00%> (-66.67%) ⬇️
...vents/MarketParticipantB2CActorIdChangedCommand.cs 33.33% <0.00%> (-66.67%) ⬇️
....ApplyDBMigrationsApp/Helpers/EnvironmentFilter.cs 0.00% <0.00%> (-62.50%) ⬇️
....Domain/Dtos/Events/GridAreaOwnerRemovedCommand.cs 50.00% <0.00%> (-50.00%) ⬇️
... and 71 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 15:59 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 15:59 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 15:59 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 16:03 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 16:03 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 7, 2022 16:03 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 9, 2022 13:24 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 9, 2022 13:24 Inactive
@jonasdmoeller jonasdmoeller temporarily deployed to AzureAuth November 9, 2022 13:24 Inactive
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@jonasdmoeller jonasdmoeller merged commit 7b7abee into main Nov 10, 2022
@jonasdmoeller jonasdmoeller deleted the fix/rewrite_outbox_tests branch November 10, 2022 07:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants