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

AzureDevOpsDsc: Increase unit test, code coverage #22

Open
SphenicPaul opened this issue Jan 25, 2021 · 1 comment
Open

AzureDevOpsDsc: Increase unit test, code coverage #22

SphenicPaul opened this issue Jan 25, 2021 · 1 comment
Labels
Backlog - Mid Term Backlog - Mid Term help wanted The issue is up for grabs for anyone in the community. tests The issue or pull request is about tests only.
Projects
Backlog - Mid Term

Comments

@SphenicPaul
Copy link
Contributor

Details of the scenario you tried and the problem that is occurring

Test coverage threshold in the build.yml file is currently set to 40 (%) to allow initial builds to execute/succeed - Although a number of classes within the unit tests are covered, Pester is not recognising the coverage.

This needs investigating and may require a refactoring of tests and/or classes (also note #10 relating to classes within the module).

Ideally, attempt to set code coverage threshold to 100% or 95%.
@SphenicPaul SphenicPaul changed the title AzureDevOpsDsc: Increase code coverage AzureDevOpsDsc: Increase unit test, code coverage Jan 25, 2021
@johlju johlju added help wanted The issue is up for grabs for anyone in the community. tests The issue or pull request is about tests only. labels Jan 25, 2021
@johlju
Copy link
Member

johlju commented Jan 25, 2021

Coverage threshold can be set to 85%, but if the actual code coverage is higher than that it's better.

@kilasuit kilasuit added this to To do in Backlog - Mid Term Jan 12, 2022
@kilasuit kilasuit added the Backlog - Mid Term Backlog - Mid Term label Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Backlog - Mid Term Backlog - Mid Term help wanted The issue is up for grabs for anyone in the community. tests The issue or pull request is about tests only.
Projects
Backlog - Mid Term
To do
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kilasuit @johlju @SphenicPaul