fix: Remove unnecessary Push-Location and Pop-Location calls in BeforeAll and AfterAll scripts for cleaner execution flow

Author: MariusStorhaug

.github/workflows/CI.yml

@@ -212,17 +212,12 @@ jobs:
 
                 Write-Host "Running BeforeAll setup script: $beforeAllScript"
                 try {
-                    Push-Location 'tests'
                     & $beforeAllScript
                     Write-Host "BeforeAll script completed successfully: $beforeAllScript"
-                }
-                catch {
+                } catch {
                     Write-Error "BeforeAll script failed: $beforeAllScript - $_"
                     throw
                 }
-                finally {
-                    Pop-Location
-                }
             }
 
   Test-ModuleLocal:
@@ -285,17 +280,12 @@ jobs:
 
                 Write-Host "Running AfterAll teardown script: $afterAllScript"
                 try {
-                    Push-Location 'tests'
                     & $afterAllScript
                     Write-Host "AfterAll script completed successfully: $afterAllScript"
-                }
-                catch {
+                } catch {
                     Write-Warning "AfterAll script failed: $afterAllScript - $_"
                     # Don't throw for teardown scripts to ensure other cleanup scripts can run
                 }
-                finally {
-                    Pop-Location
-                }
             }
 
   Get-TestResults:

.github/workflows/workflow.yml

@@ -214,17 +214,12 @@ jobs:
 
                 Write-Host "Running BeforeAll setup script: $beforeAllScript"
                 try {
-                    Push-Location 'tests'
                     & $beforeAllScript
                     Write-Host "BeforeAll script completed successfully: $beforeAllScript"
-                }
-                catch {
+                } catch {
                     Write-Error "BeforeAll script failed: $beforeAllScript - $_"
                     throw
                 }
-                finally {
-                    Pop-Location
-                }
             }
 
   Test-ModuleLocal:
@@ -287,17 +282,12 @@ jobs:
 
                 Write-Host "Running AfterAll teardown script: $afterAllScript"
                 try {
-                    Push-Location 'tests'
                     & $afterAllScript
                     Write-Host "AfterAll script completed successfully: $afterAllScript"
-                }
-                catch {
+                } catch {
                     Write-Warning "AfterAll script failed: $afterAllScript - $_"
                     # Don't throw for teardown scripts to ensure other cleanup scripts can run
                 }
-                finally {
-                    Pop-Location
-                }
             }
 
   Get-TestResults: