Browse Source

Fix formatting directive issues in tests.

As detected with go vet.

Simplify code slightly by removing unnecessary brackets around a
variable.
Dmitri Shuralyov 10 years ago
parent
commit
81ea1e5cb3
2 changed files with 3 additions and 3 deletions
  1. +1
    -1
      github/repos_releases_test.go
  2. +2
    -2
      tests/integration/authorizations_test.go

+ 1
- 1
github/repos_releases_test.go View File

@ -275,7 +275,7 @@ func TestRepositoriesService_DownloadReleaseAsset_APIError(t *testing.T) {
}
if loc != "" {
t.Error(`Repositories.DownloadReleaseAsset returned "%s", want empty ""`, loc)
t.Errorf(`Repositories.DownloadReleaseAsset returned "%s", want empty ""`, loc)
}
}


+ 2
- 2
tests/integration/authorizations_test.go View File

@ -52,7 +52,7 @@ func TestAuthorizationsBasicOperations(t *testing.T) {
failIfNotStatusCode(t, resp, 200)
if len(auths) != initialAuthCount+1 {
t.Fatalf("The number of Authorizations should have increased. Expected [%v], was [%v]", (initialAuthCount + 1), len(auths))
t.Fatalf("The number of Authorizations should have increased. Expected [%v], was [%v]", initialAuthCount+1, len(auths))
}
// Test updating the authorization
@ -94,7 +94,7 @@ func TestAuthorizationsBasicOperations(t *testing.T) {
failIfNotStatusCode(t, resp, 200)
if len(auths) != initialAuthCount {
t.Fatal("The number of Authorizations should match the initial count Expected [%v], got [%v]", (initialAuthCount), len(auths))
t.Fatalf("The number of Authorizations should match the initial count Expected [%v], got [%v]", initialAuthCount, len(auths))
}
}


Loading…
Cancel
Save