Closed johlju closed 1 year ago
I converted the tests for Get so far. The rest are coming in other commits.
Merging #1820 (c3b7b2e) into main (d5692e2) will increase coverage by
2%
. The diff coverage is100%
.
@@ Coverage Diff @@
## main #1820 +/- ##
=====================================
+ Coverage 88% 90% +2%
=====================================
Files 82 82
Lines 7437 7437
=====================================
+ Hits 6588 6737 +149
+ Misses 849 700 -149
Flag | Coverage Δ | *Carryforward flag | |
---|---|---|---|
unit | 90% <100%> (+2%) |
:arrow_up: | Carriedforward from bee3ad5 |
*This pull request uses carry forward flags. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
...SCResources/DSC_SqlAGReplica/DSC_SqlAGReplica.psm1 | 75% <100%> (+75%) |
:arrow_up: |
Converted Set-TargetResource
. I will convert Test-TargetResource
tomorrow.
Pull Request (PR) description
Get-TargetResource
always returns values for the propertiesName
andAvailabilityGroupName
(issue #1822).Test-TargetResource
no longer test properties that cannot be enforced (issue #1822).This Pull Request (PR) fixes the following issues
Task list
This change is