Unskipping tests, fixing CommentId type

Author: HowardWolosky

GitHubGistComments.ps1

@@ -71,10 +71,10 @@ filter Get-GitHubGistComment
         [Parameter(ValueFromPipelineByPropertyName)]
         [Alias('GistCommentId')]
         [ValidateNotNullOrEmpty()]
-        [string] $Comment,
+        [int64] $Comment,
 
         [ValidateSet('Raw', 'Text', 'Html', 'Full')]
-        [string] $MediaType = 'Raw',
+        [string] $MediaType = 'Full',
 
         [string] $AccessToken,
 
@@ -187,7 +187,7 @@ filter Remove-GitHubGistComment
             Position = 2)]
         [Alias('GistCommentId')]
         [ValidateNotNullOrEmpty()]
-        [string] $Comment,
+        [int64] $Comment,
 
         [switch] $Force,
 
@@ -371,7 +371,7 @@ filter Set-GitHubGistComment
             Position = 2)]
         [Alias('GistCommentId')]
         [ValidateNotNullOrEmpty()]
-        [string] $Comment,
+        [int64] $Comment,
 
         [Parameter(
             Mandatory,

Tests/GitHubGists.tests.ps1

@@ -386,9 +386,10 @@ try
                 $files | Remove-Item -Force -ErrorAction SilentlyContinue | Out-Null
             }
 
-            # Marking this test to skip.  It works just fine, but takes 26 second to execute.
-            # Not worth it for the moderate improvement to code coverage.
-            It 'Should throw an exception because there are too many files' -Skip:(-not ([string]::IsNullOrEmpty($env:ciAccessToken))) {
+            # May want to consider skipping this test.
+            # It works just fine, but takes 26 second to execute.
+            # (May not be worth it for the moderate improvement to code coverage.)
+            It 'Should throw an exception because there are too many files' {
                 $gist = $files | New-GitHubGist
                 { $gist | Get-GitHubGist -Path $tempPath -Force } | Should -Throw
                 $gist | Remove-GitHubGist -Force
@@ -409,9 +410,10 @@ try
                     Remove-Item -Force -ErrorAction SilentlyContinue | Out-Null
             }
 
-            # Marking this test to skip.  It works just fine, but takes 82 second to execute.
-            # Not worth it for the moderate improvement to code coverage.
-            It 'Should throw an exception because the file is too large to download' -Skip:(-not ([string]::IsNullOrEmpty($env:ciAccessToken))) {
+            # May want to consider skipping this test.
+            # It works just fine, but takes 26 second to execute.
+            # (May not be worth it for the moderate improvement to code coverage.)
+            It 'Should throw an exception because the file is too large to download' {
                 $gist = $tenMegFile | New-GitHubGist
                 { $gist | Get-GitHubGist -Path $tempPath -Force } | Should -Throw
                 $gist | Remove-GitHubGist -Force