Skip to content

Implement keyword-based dynamic text replacement #1538

Implement keyword-based dynamic text replacement

Implement keyword-based dynamic text replacement #1538

Triggered via pull request November 28, 2024 15:53
Status Failure
Total duration 4m 36s
Artifacts 1

build-and-analyze.yml

on: pull_request
Build, test & analyze
4m 26s
Build, test & analyze
Fit to window
Zoom out
Zoom in

Annotations

3 errors and 10 warnings
Build, test & analyze
Process completed with exit code 1.
Build, test & analyze
Invalid request. Only 65535 characters are allowed; 222327 were supplied. - https://docs.github.com/rest/checks/runs#create-a-check-run
Build, test & analyze
At least one failing test was found
Build, test & analyze: src/Altinn.Notifications.Core/Services/SmsNotificationService.cs#L47
Rename parameter 'smsBody' to 'body' to match the interface declaration. (https://rules.sonarsource.com/csharp/RSPEC-927)
Build, test & analyze: src/Altinn.Notifications.Core/Services/KeywordsService.cs#L70
Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
Build, test & analyze: src/Altinn.Notifications/appsettings.json#L11
"pwd" detected here, make sure this is not a hard-coded credential. (https://rules.sonarsource.com/csharp/RSPEC-2068)
Build, test & analyze: src/Altinn.Notifications/appsettings.json#L12
"pwd" detected here, make sure this is not a hard-coded credential. (https://rules.sonarsource.com/csharp/RSPEC-2068)
Build, test & analyze: src/Altinn.Notifications.Core/Services/SmsNotificationService.cs#L47
Rename parameter 'smsBody' to 'body' to match the interface declaration. (https://rules.sonarsource.com/csharp/RSPEC-927)
Build, test & analyze: src/Altinn.Notifications.Core/Services/KeywordsService.cs#L70
Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
Build, test & analyze: src/Altinn.Notifications/appsettings.json#L11
"pwd" detected here, make sure this is not a hard-coded credential. (https://rules.sonarsource.com/csharp/RSPEC-2068)
Build, test & analyze: src/Altinn.Notifications/appsettings.json#L12
"pwd" detected here, make sure this is not a hard-coded credential. (https://rules.sonarsource.com/csharp/RSPEC-2068)
Build, test & analyze: src/Altinn.Notifications.Core/Services/SmsNotificationService.cs#L47
Rename parameter 'smsBody' to 'body' to match the interface declaration. (https://rules.sonarsource.com/csharp/RSPEC-927)
Build, test & analyze: src/Altinn.Notifications.Core/Services/KeywordsService.cs#L70
Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

Artifacts

Produced during runtime
Name Size
TestResults
615 KB