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.

Sign up for GitHub

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

Jump to bottom

[PM-10600] Unit Test coverage and refactor for Push Notifications #5011

Draft
wants to merge 10 commits into
base: km/pm-10600
Choose a base branch
from
+793 −150

PM-10600: Throw Xunit assert exception when sending request via BaseI…

7bea641
Select commit
Loading
Failed to load commit list.
Draft

[PM-10600] Unit Test coverage and refactor for Push Notifications #5011

PM-10600: Throw Xunit assert exception when sending request via BaseI…
7bea641
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Nov 9, 2024 in 0s

38.29% of diff hit (target 42.78%)

View this Pull Request on Codecov

38.29% of diff hit (target 42.78%)

Annotations

Check warning on line 97 in src/Core/Services/Implementations/BaseIdentityClientService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Core/Services/Implementations/BaseIdentityClientService.cs#L96-L97 

Added lines #L96 - L97 were not covered by tests

Check warning on line 20 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L20 

Added line #L20 was not covered by tests

Check warning on line 24 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L24 

Added line #L24 was not covered by tests

Check warning on line 50 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L49-L50 

Added lines #L49 - L50 were not covered by tests

Check warning on line 59 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L59 

Added line #L59 was not covered by tests

Check warning on line 69 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L66-L69 

Added lines #L66 - L69 were not covered by tests

Check warning on line 78 in src/Notifications/AzureQueueHostedService.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/AzureQueueHostedService.cs#L77-L78 

Added lines #L77 - L78 were not covered by tests

Check warning on line 14 in src/Notifications/Controllers/SendController.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/Controllers/SendController.cs#L14 

Added line #L14 was not covered by tests

Check warning on line 16 in src/Notifications/Controllers/SendController.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/Controllers/SendController.cs#L16 

Added line #L16 was not covered by tests

Check warning on line 24 in src/Notifications/Controllers/SendController.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/Controllers/SendController.cs#L23-L24 

Added lines #L23 - L24 were not covered by tests

Check warning on line 27 in src/Notifications/Controllers/SendController.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/Controllers/SendController.cs#L27 

Added line #L27 was not covered by tests

Check warning on line 43 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L43 

Added line #L43 was not covered by tests

Check warning on line 48 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L48 

Added line #L48 was not covered by tests

Check warning on line 60 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L60 

Added line #L60 was not covered by tests

Check warning on line 72 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L72 

Added line #L72 was not covered by tests

Check warning on line 81 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L81 

Added line #L81 was not covered by tests

Check warning on line 88 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L88 

Added line #L88 was not covered by tests

Check warning on line 95 in src/Notifications/HubHelpers.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/HubHelpers.cs#L95 

Added line #L95 was not covered by tests

Check warning on line 64 in src/Notifications/Startup.cs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/Notifications/Startup.cs#L64 

Added line #L64 was not covered by tests
View more details on Codecov