✨ [Refactor]: Simplify repository creation and logging for user and organization owners in test scripts

Author: MariusStorhaug

tests/BeforeAll.ps1

@@ -62,15 +62,11 @@ LogGroup 'BeforeAll - Global Test Setup' {
                 }
 
                 # Create extra repositories needed by Secrets/Variables SelectedRepository tests.
-                foreach ($suffix in 2, 3) {
-                    $extraName = "$repoName-$suffix"
-                    switch ($OwnerType) {
-                        'user' {
-                            New-GitHubRepository -Name $extraName
-                        }
-                        'organization' {
-                            New-GitHubRepository -Organization $Owner -Name $extraName
-                        }
+                # Only organization owners need them — those tests are skipped for user owners.
+                if ($OwnerType -eq 'organization') {
+                    foreach ($suffix in 2, 3) {
+                        $extraName = "$repoName-$suffix"
+                        New-GitHubRepository -Organization $Owner -Name $extraName
                     }
                 }
             }

tests/Secrets.Tests.ps1

@@ -50,8 +50,6 @@ Describe 'Secrets' {
             switch ($OwnerType) {
                 'user' {
                     $repo = Get-GitHubRepository -Name $repoName
-                    $repo2 = Get-GitHubRepository -Name "$repoName-2"
-                    $repo3 = Get-GitHubRepository -Name "$repoName-3"
                 }
                 'organization' {
                     Get-GitHubSecret -Owner $Owner | Where-Object { $_.Name -like "$secretName*" } | Remove-GitHubSecret -Confirm:$false
@@ -74,8 +72,10 @@ Describe 'Secrets' {
             }
             LogGroup "Repository - [$repoName]" {
                 Write-Host ($repo | Select-Object * | Out-String)
-                Write-Host ($repo2 | Select-Object * | Out-String)
-                Write-Host ($repo3 | Select-Object * | Out-String)
+                if ($OwnerType -eq 'organization') {
+                    Write-Host ($repo2 | Select-Object * | Out-String)
+                    Write-Host ($repo3 | Select-Object * | Out-String)
+                }
             }
         }
 

tests/Variables.Tests.ps1

@@ -50,8 +50,6 @@ Describe 'Variables' {
             switch ($OwnerType) {
                 'user' {
                     $repo = Get-GitHubRepository -Name $repoName
-                    $repo2 = Get-GitHubRepository -Name "$repoName-2"
-                    $repo3 = Get-GitHubRepository -Name "$repoName-3"
                 }
                 'organization' {
                     Get-GitHubVariable -Owner $Owner | Where-Object { $_.Name -like "$variableName*" } | Remove-GitHubVariable -Confirm:$false
@@ -73,8 +71,10 @@ Describe 'Variables' {
             }
             LogGroup "Repository - [$repoName]" {
                 Write-Host ($repo | Select-Object * | Out-String)
-                Write-Host ($repo2 | Select-Object * | Out-String)
-                Write-Host ($repo3 | Select-Object * | Out-String)
+                if ($OwnerType -eq 'organization') {
+                    Write-Host ($repo2 | Select-Object * | Out-String)
+                    Write-Host ($repo3 | Select-Object * | Out-String)
+                }
             }
         }