SqlServerDsc: Re-enable Pester's new code coverage method (#1857)

- SqlServerDsc - Add empty constructor to classes to be able to use Pester's new code coverage method. See more information can be found in [pester/Pester#2306]
dsccommunity · Feb 25, 2023 · f6ce795 · f6ce795
1 parent 38a0da4
commit f6ce795
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -144,6 +144,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
     is loaded into the session. This will make it possible for classes and
     commands to use and return SQL types. If no module is found it will
     output a warning to install any of the dependent modules.
+  - Add empty constructor to classes to be able to use Pester's new code
+    coverage method. See more information can be found in [pester/Pester#2306](https://github.com/pester/Pester/issues/2306).
 - `Install-SqlServerDsc`
   - No longer throws an exception when parameter `AgtSvcAccount` is not specified.
 - SqlAgReplica

diff --git a/build.yaml b/build.yaml
@@ -98,9 +98,7 @@ Pester:
       CoveragePercentTarget: 85
       OutputPath: JaCoCo_coverage.xml
       OutputEncoding: ascii
-      # There is a bug in Pester when running unit tests for classes when 'UseBreakpoints' is turned off.
-      # See error in gist: https://gist.github.com/johlju/c16dfd9587c7e066e8825fc54b33a703
-      UseBreakpoints: true
+      UseBreakpoints: false
     TestResult:
       OutputFormat: NUnitXML
       OutputEncoding: ascii

diff --git a/source/Classes/002.DatabasePermission.ps1 b/source/Classes/002.DatabasePermission.ps1
@@ -114,6 +114,10 @@ class DatabasePermission : IComparable, System.IEquatable[Object]
     [System.String[]]
     $Permission
 
+    DatabasePermission ()
+    {
+    }
+
     [System.Boolean] Equals([System.Object] $object)
     {
         $isEqual = $false

diff --git a/source/Classes/002.ServerPermission.ps1 b/source/Classes/002.ServerPermission.ps1
@@ -81,6 +81,11 @@ class ServerPermission : IComparable, System.IEquatable[Object]
     [System.String[]]
     $Permission
 
+
+    ServerPermission ()
+    {
+    }
+
     <#
         TODO: It was not possible to move this to a parent class. But since these are
               generic functions for DatabasePermission and ServerPermission we

diff --git a/source/Classes/004.StartupParameters.ps1 b/source/Classes/004.StartupParameters.ps1
@@ -34,6 +34,10 @@ class StartupParameters
     [System.UInt32[]]
     $InternalTraceFlag
 
+    StartupParameters ()
+    {
+    }
+
     static [StartupParameters] Parse([System.String] $InstanceStartupParameters)
     {
         Write-Debug -Message (