Support for Set-GitHubContent to Upload binary files

GitHubContents.ps1

@@ -28,7 +28,7 @@ filter Get-GitHubContent
         them individually.
 
     .PARAMETER Path
-        The file path for which to retrieve contents
+        The remote file path in the repository whose content should be changed.
 
     .PARAMETER BranchName
         The branch, or defaults to the default branch of not specified.
@@ -231,6 +231,9 @@ filter Set-GitHubContent
     .PARAMETER Content
         The new file content.
 
+    .PARAMETER ContentPath
+        The local path to a file whose content should replace what is currently stored remotely in the repo at the Path location.
+
     .PARAMETER Sha
         The SHA value of the current file if present. If this parameter is not provided, and the
         file currently exists in the specified branch of the repo, it will be read to obtain this
@@ -294,19 +297,30 @@ filter Set-GitHubContent
     param(
         [Parameter(
             Mandatory,
-            ParameterSetName = 'Elements')]
+            ParameterSetName = 'ElementsContent')]
+        [Parameter(
+            Mandatory,
+            ParameterSetName = 'ElementsContentPath')]
         [string] $OwnerName,
 
         [Parameter(
             Mandatory,
-            ParameterSetName = 'Elements')]
+            ParameterSetName = 'ElementsContent')]
+        [Parameter(
+            Mandatory,
+            ParameterSetName = 'ElementsContentPath')]
         [string] $RepositoryName,
 
         [Parameter(
             Mandatory,
             ValueFromPipelineByPropertyName,
             Position = 1,
-            ParameterSetName='Uri')]
+            ParameterSetName='UriContent')]
+        [Parameter(
+            Mandatory,
+            ValueFromPipelineByPropertyName,
+            Position = 1,
+            ParameterSetName='UriContentPath')]
         [Alias('RepositoryUrl')]
         [string] $Uri,
 
@@ -323,9 +337,22 @@ filter Set-GitHubContent
 
         [Parameter(
             Mandatory,
-            Position = 4)]
+            Position = 4,
+            ParameterSetName = 'ElementsContent')]
+        [Parameter(
+            Mandatory,
+            Position = 4,
+            ParameterSetName = 'UriContent')]
         [string] $Content,
 
+        [Parameter(
+            Mandatory,
+            ParameterSetName = 'ElementsContentPath')]
+        [Parameter(
+            Mandatory,
+            ParameterSetName = 'UriContentPath')]
+        [string] $ContentPath,
+
         [Parameter(ValueFromPipelineByPropertyName)]
         [string] $Sha,
 
@@ -358,7 +385,15 @@ filter Set-GitHubContent
 
     $uriFragment = "/repos/$OwnerName/$RepositoryName/contents/$Path"
 
-    $encodedContent = [Convert]::ToBase64String([System.Text.Encoding]::UTF8.GetBytes($Content))
+    if ($PSBoundParameters.ContainsKey('ContentPath'))
+    {
+        $ContentPath = Resolve-UnverifiedPath -Path $ContentPath
+        $encodedContent = [Convert]::ToBase64String([System.IO.File]::ReadAllBytes($ContentPath))
+    }
+    else
+    {
+        $encodedContent = [Convert]::ToBase64String([System.Text.Encoding]::UTF8.GetBytes($Content))
+    }
 
     $hashBody = @{
         message = $CommitMessage

Tests/GitHubContents.tests.ps1

@@ -296,7 +296,7 @@ Describe 'GitHubContents/Set-GitHubContent' {
             $setGitHubContentParms = @{
                 Path = "$filePath/$fileName"
                 CommitMessage = $commitMessage
-                Branch = $branchName
+                BranchName = $branchName
                 Content = $content
                 Uri = $repo.svn_url
                 CommitterName = $committerName
@@ -399,6 +399,126 @@ Describe 'GitHubContents/Set-GitHubContent' {
         }
     }
 
+    Context 'When setting new file content by path' {
+        BeforeAll {
+            $filePath = 'notes'
+            $fileName = 'hello-by-path.txt'
+            $commitMessage = 'Commit Message'
+            $content = 'This is the content for hello-by-path.txt'
+            $branchName = 'master'
+            $committerName = 'John Doe'
+            $committerEmail = '[email protected]'
+            $authorName = 'Jane Doe'
+            $authorEmail = '[email protected]'
+
+            $contentPath = New-TemporaryFile
+            [System.IO.File]::WriteAllText($contentPath, $content)
+
+            $setGitHubContentParms = @{
+                Path = "$filePath/$fileName"
+                CommitMessage = $commitMessage
+                BranchName = $branchName
+                ContentPath = $contentPath.FullName
+                Uri = $repo.svn_url
+                CommitterName = $committerName
+                CommitterEmail = $committerEmail
+                authorName = $authorName
+                authorEmail = $authorEmail
+            }
+
+            $result = Set-GitHubContent @setGitHubContentParms -PassThru
+        }
+
+        It 'Should have the expected type and additional properties' {
+            $result.PSObject.TypeNames[0] | Should -Be 'GitHub.Content'
+            $result.content.name | Should -Be $fileName
+            $result.content.path | Should -Be "$filePath/$fileName"
+            $result.content.url | Should -Be ("https://api.github.com/repos/$($script:ownerName)" +
+                "/$repoName/contents/$filePath/$($fileName)?ref=$branchName")
+            $result.commit.author.name | Should -Be $authorName
+            $result.commit.author.email | Should -Be $authorEmail
+            $result.commit.committer.name | Should -Be $committerName
+            $result.commit.committer.email | Should -Be $committerEmail
+            $result.commit.message | Should -Be $commitMessage
+        }
+
+        It 'Should have written the correct content' {
+            $getGitHubContentParms = @{
+                Path = "$filePath/$fileName"
+                Uri = $repo.svn_url
+                MediaType = 'Raw'
+                ResultAsString = $true
+            }
+
+            $writtenContent = Get-GitHubContent @getGitHubContentParms
+
+            $content | Should -Be $writtenContent
+        }
+
+        AfterAll {
+            Remove-Item -Path $contentPath
+        }
+    }
+
+    Context 'When overwriting file content by path' {
+        BeforeAll {
+            $filePath = 'notes'
+            $fileName = 'hello-by-path.txt'
+            $commitMessage = 'Commit Message 2'
+            $content = 'This is the new content for hello-by-path.txt'
+            $branchName = 'master'
+            $committerName = 'John Doe'
+            $committerEmail = '[email protected]'
+            $authorName = 'Jane Doe'
+            $authorEmail = '[email protected]'
+
+            $contentPath = New-TemporaryFile
+            [System.IO.File]::WriteAllText($contentPath, $content)
+
+            $setGitHubContentParms = @{
+                Path = "$filePath/$fileName"
+                CommitMessage = $commitMessage
+                Branch = $branchName
+                ContentPath = $contentPath.FullName
+                Uri = $repo.svn_url
+                CommitterName = $committerName
+                CommitterEmail = $committerEmail
+                authorName = $authorName
+                authorEmail = $authorEmail
+            }
+
+            $result = Set-GitHubContent @setGitHubContentParms -PassThru
+        }
+
+        It 'Should have the expected type and additional properties' {
+            $result.PSObject.TypeNames[0] | Should -Be 'GitHub.Content'
+            $result.content.name | Should -Be $fileName
+            $result.content.path | Should -Be "$filePath/$fileName"
+            $result.content.url | Should -Be ("https://api.github.com/repos/$($script:ownerName)" +
+                "/$repoName/contents/$filePath/$($fileName)?ref=$branchName")
+            $result.commit.author.name | Should -Be $authorName
+            $result.commit.author.email | Should -Be $authorEmail
+            $result.commit.committer.name | Should -Be $committerName
+            $result.commit.committer.email | Should -Be $committerEmail
+            $result.commit.message | Should -Be $commitMessage
+        }
+
+        It 'Should have written the correct content' {
+            $getGitHubContentParms = @{
+                Path = "$filePath/$fileName"
+                Uri = $repo.svn_url
+                MediaType = 'Raw'
+                ResultAsString = $true
+            }
+
+            $content | Should -Be (Get-GitHubContent @getGitHubContentParms)
+        }
+
+        AfterAll {
+            Remove-Item -Path $contentPath
+        }
+    }
+
     Context 'When Specifying only one Committer parameter' {
         BeforeAll {
             $setGitHubContentParms = @{