removed some text in checkanswerscontroller #3832
jag-nahl-airelogiccode-pr-check.yml
on: pull_request
Lint & Commit
59s
Build database upgrader
14s
Run javascript tests
17s
Run Jest Tests for Contentful
4s
Build and run unit tests
2m 11s
Annotations
2 errors and 13 warnings
|
Dfe.PlanTech.Web.UnitTests.Controllers.CheckAnswersControllerTests ► ConfirmAnswers_Should_Redirect_To_CheckAnswers:
tests/Dfe.PlanTech.Web.UnitTests/Controllers/CheckAnswersControllerTests.cs#L167
Failed test found in:
tests/Dfe.PlanTech.Web.UnitTests/TestResults/test-results.trx
Error:
Assert.Equal() Failure: Values differ
Expected: Unable to save. Please try again. If this problem continues you can
Actual: Unable to save. Please try again. If this problem
|
|
Build and run unit tests
Failed test were found and 'fail-on-error' option is set to true
|
|
Build database upgrader
The "--output" option isn't supported when building a solution. Specifying a solution-level output path results in all projects copying outputs to the same directory, which can lead to inconsistent builds.
|
|
Build database upgrader
The "--output" option isn't supported when building a solution. Specifying a solution-level output path results in all projects copying outputs to the same directory, which can lead to inconsistent builds.
|
|
Run Jest Tests for Contentful
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Build and run unit tests
The "--output" option isn't supported when building a solution. Specifying a solution-level output path results in all projects copying outputs to the same directory, which can lead to inconsistent builds.
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Content/Models/ContentSupport/ContentItem.cs#L11
'ContentItem.Data' hides inherited member 'ContentItemBase.Data'. Use the new keyword if hiding was intended.
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Content/Models/ContentSupport/ContentItemBase.cs#L9
'ContentItemBase.Data' hides inherited member 'ContentBase.Data'. Use the new keyword if hiding was intended.
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Content/Interfaces/IRichTextContent.cs#L40
Reduce the number of generic parameters in the 'IRichTextContent' interface to no more than the 2 authorized. (https://rules.sonarsource.com/csharp/RSPEC-2436)
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Questionnaire/Interfaces/IRecommendationSection.cs#L7
Reduce the number of generic parameters in the 'IRecommendationSection' interface to no more than the 2 authorized. (https://rules.sonarsource.com/csharp/RSPEC-2436)
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Questionnaire/Interfaces/ISubTopicRecommendation.cs#L7
Reduce the number of generic parameters in the 'ISubTopicRecommendation' interface to no more than the 2 authorized. (https://rules.sonarsource.com/csharp/RSPEC-2436)
|
|
Build and run unit tests:
src/Dfe.PlanTech.Domain/Content/Models/Options/TextRendererOptions.cs#L22
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
Build and run unit tests:
src/Dfe.PlanTech.Application/Responses/Queries/GetLatestResponsesQuery.cs#L11
Parameter 'IPlanTechDbContext db' is captured into the state of the enclosing type and its value is also used to initialize a field, property, or event.
|
|
Build and run unit tests:
src/Dfe.PlanTech.Application/Content/Queries/GetContentSupportPageQuery.cs#L28
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|
|
Build and run unit tests:
src/Dfe.PlanTech.Application/Content/Queries/GetPageQuery.cs#L56
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
|