🚀 [Refactor]: Add null checks for Permissions and App properties in GitHubAppContext constructor

Author: MariusStorhaug

src/classes/public/Context/GitHubContext/GitHubAppContext.ps1

@@ -50,8 +50,12 @@
         $this.KeyVaultKeyReference = $Object.KeyVaultKeyReference
         $this.OwnerName = $Object.OwnerName
         $this.OwnerType = $Object.OwnerType
-        $this.Permissions = [GitHubPermission]::NewPermissionList($Object.Permissions)
+        if ($Object.Permissions) {
+            $this.Permissions = [GitHubPermission]::NewPermissionList($Object.Permissions)
+        }
         $this.Events = , ($Object.Events)
-        $this.App = [GitHubApp]::New($Object.App)
+        if ($Object.App) {
+            $this.App = [GitHubApp]::New($Object.App)
+        }
     }
 }

src/functions/private/Auth/Context/Set-GitHubContext.ps1

@@ -44,12 +44,14 @@
     }
 
     process {
-        Write-Debug 'Context:'
-        $contextObj | Out-String -Stream | ForEach-Object { Write-Debug $_ }
+        if ($DebugPreference -eq 'Continue') {
+            Write-Debug 'Context:'
+            $contextObj | Out-String -Stream | ForEach-Object { Write-Debug $_ }
+            Write-Debug "Getting info on the context [$($contextObj['AuthType'])]."
+        }
 
         # Run functions to get info on the temporary context.
         try {
-            Write-Debug "Getting info on the context [$($contextObj['AuthType'])]."
             switch -Regex (($contextObj['AuthType'])) {
                 'PAT|UAT|IAT' {
                     $viewer = Get-GitHubViewer -Context $contextObj
@@ -79,9 +81,9 @@
                         try {
                             $app = Get-GitHubApp -Slug $contextObj['Username'] -Context $contextObj
                             $contextObj['DisplayName'] = [string]$app.Name
-                            # TODO: $contextObj['App'] = $app
+                            $contextObj['App'] = $app
                         } catch {
-                            Write-Warning "Failed to get the GitHub App with the slug: [$($contextObj['Username'])]."
+                            Write-Warning "Unable to get the GitHub App: [$($contextObj['Username'])]."
                         }
                     }
                     if ($script:IsGitHubActions) {
@@ -116,9 +118,6 @@
                         }
                         $contextObj['Name'] = "$($contextObj['HostName'])/$($contextObj['Username'])/" +
                         "$($contextObj['InstallationType'])/$($contextObj['InstallationName'])"
-                    } else {
-                        $contextObj['Name'] = "$($contextObj['HostName'])/$($contextObj['Username'])/" +
-                        "$($contextObj['InstallationType'])/$($contextObj['InstallationName'])"
                     }
                 }
                 'App' {
@@ -132,18 +131,20 @@
                     $contextObj['Events'] = [string[]]$app.Events
                     $contextObj['OwnerName'] = [string]$app.Owner.Name
                     $contextObj['OwnerType'] = [string]$app.Owner.Type
-                    $contextObj['App'] = $app
+                    $contextObj['App'] = [GitHubApp]$app
                     $contextObj['Type'] = 'App'
                 }
                 default {
                     throw 'Failed to get info on the context. Unknown logon type.'
                 }
             }
-            Write-Debug "Found [$($contextObj['Type'])] with login: [$($contextObj['Name'])]"
-            $contextObj | Out-String -Stream | ForEach-Object { Write-Debug $_ }
-            Write-Debug '----------------------------------------------------'
-            if ($PSCmdlet.ShouldProcess('Context', 'Set')) {
+            if ($DebugPreference -eq 'Continue') {
+                Write-Debug "Found [$($contextObj['Type'])] with login: [$($contextObj['Name'])]"
+                $contextObj | Out-String -Stream | ForEach-Object { Write-Debug $_ }
+                Write-Debug '----------------------------------------------------'
                 Write-Debug "Saving context: [$($contextObj['Name'])]"
+            }
+            if ($PSCmdlet.ShouldProcess('Context', 'Set')) {
                 Set-Context -ID $($contextObj['Name']) -Context $contextObj -Vault $script:GitHub.ContextVault
                 if ($Default) {
                     Switch-GitHubContext -Context $contextObj['Name']