Skip to content

Commit d94ba9e

Browse files
author
S Raghav
committed
Refactor testcases and fix formatting
1 parent 3a359af commit d94ba9e

File tree

2 files changed

+22
-21
lines changed

2 files changed

+22
-21
lines changed

GitHubReferences.ps1

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ function Get-GitHubReference
101101

102102
$params = @{
103103
'UriFragment' = $uriFragment
104-
'Description' = $description
104+
'Description' = $description
105105
'AccessToken' = $AccessToken
106106
'TelemetryEventName' = $MyInvocation.MyCommand.Name
107107
'TelemetryProperties' = $telemetryProperties
@@ -167,8 +167,8 @@ function New-GitHubReference
167167
If not supplied here, the DefaultNoStatus configuration property value will be used.
168168
169169
.OUTPUTS
170-
[PSCustomObject]
171-
Details of the git reference created. Throws an Exception if the reference already exists
170+
[PSCustomObject]
171+
Details of the git reference created. Throws an Exception if the reference already exists
172172
173173
.EXAMPLE
174174
New-GitHubReference -OwnerName Powershell -RepositoryName PowerShellForGitHub -Reference heads/master -Sha aa218f56b14c9653891f9e74264a383fa43fefbd

Tests/GitHubReferences.Tests.ps1

Lines changed: 19 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ function Initialize-AppVeyor
3434
3535
#>
3636
[CmdletBinding()]
37-
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidUsingConvertToSecureStringWithPlainText", "", Justification="Needed to configure with the stored, encrypted string value in AppVeyor.")]
37+
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidUsingConvertToSecureStringWithPlainText", "",
38+
Justification="Needed to configure with the stored, encrypted string value in AppVeyor.")]
3839
param()
3940

4041
if ($env:AppVeyor)
@@ -84,7 +85,8 @@ try
8485
Describe 'Create a new reference(branch) in repository' {
8586
$repositoryName = [Guid]::NewGuid()
8687
$repo = New-GitHubRepository -RepositoryName $repositoryName -AutoInit
87-
$existingref = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference "heads/master"
88+
$masterRefName = "heads/master"
89+
$existingref = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $masterRefName
8890
$sha = $existingref.object.sha
8991

9092
Context 'On creating a valid reference in a new repository from a given SHA' {
@@ -106,18 +108,14 @@ try
106108
}
107109

108110
Context 'On creating an existing reference in a new repository from a given SHA' {
109-
$refName = "heads/master"
110-
111111
It 'Should throw an Exception' {
112-
{ New-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $refName -Sha $sha } | Should Throw
112+
{ New-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $masterRefName -Sha $sha } | Should Throw
113113
}
114114
}
115115

116116
Context 'On creating an existing reference in a new repository (specified by Uri) from a given SHA' {
117-
$refName = "heads/master"
118-
119117
It 'Should throw an exception' {
120-
{ New-GitHubReference -Uri $repo.svn_url -Reference $refName -Sha $sha } | Should Throw
118+
{ New-GitHubReference -Uri $repo.svn_url -Reference $masterRefName -Sha $sha } | Should Throw
121119
}
122120
}
123121

@@ -127,34 +125,35 @@ try
127125
Describe 'Getting a reference(branch) from repository' {
128126
$repositoryName = [Guid]::NewGuid()
129127
$repo = New-GitHubRepository -RepositoryName $repositoryName -AutoInit
130-
$refName = "refs/heads/master"
128+
$masterRefName = "heads/master"
129+
$randomRefName = "heads/someRandomRef"
131130

132131
Context 'On getting a valid reference from a new repository' {
133-
$reference = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference "heads/master"
132+
$reference = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $masterRefName
134133

135134
It 'Should return details of the reference' {
136-
$reference.ref | Should be $refName
135+
$reference.ref | Should be "refs/$masterRefName"
137136
}
138137
}
139138

140139
Context 'On getting an invalid reference from a new repository' {
141-
$reference = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference "heads/someRandomRef"
140+
$reference = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $randomRefName
142141

143142
It 'Should not return any details' {
144143
$reference | Should be $null
145144
}
146145
}
147146

148147
Context 'On getting a valid reference using Uri from a new repository' {
149-
$reference = Get-GitHubReference -Uri $repo.svn_url -Reference "heads/master"
148+
$reference = Get-GitHubReference -Uri $repo.svn_url -Reference $masterRefName
150149

151150
It 'Should return details of the reference' {
152-
$reference.ref | Should be $refName
151+
$reference.ref | Should be "refs/$masterRefName"
153152
}
154153
}
155154

156155
Context 'On getting an invalid reference using Uri from a new repository' {
157-
$reference = Get-GitHubReference -Uri $repo.svn_url -Reference "heads/someRandomRef"
156+
$reference = Get-GitHubReference -Uri $repo.svn_url -Reference $randomRefName
158157

159158
It 'Should not return any details' {
160159
$reference | Should be $null
@@ -167,10 +166,12 @@ try
167166
Describe 'Getting all references from repository' {
168167
$repositoryName = [Guid]::NewGuid()
169168
$repo = New-GitHubRepository -RepositoryName $repositoryName -AutoInit
170-
$existingref = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference "heads/master"
169+
$masterRefName = "heads/master"
170+
$secondRefName = "heads/branch1"
171+
$existingref = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $masterRefName
171172
$sha = $existingref.object.sha
172-
New-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference "heads/branch1" -Sha $sha
173-
$refNames = @("refs/heads/master", "refs/heads/branch1")
173+
New-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName -Reference $secondRefName -Sha $sha
174+
$refNames = @("refs/$masterRefName", "refs/$secondRefName")
174175

175176
Context 'On getting all references from a new repository' {
176177
$reference = Get-GitHubReference -OwnerName $ownerName -RepositoryName $repositoryName

0 commit comments

Comments
 (0)