Skip to content

Commit

Permalink
Fix formatting directive issues in tests.
Browse files Browse the repository at this point in the history
As detected with go vet.

Simplify code slightly by removing unnecessary brackets around a
variable.
  • Loading branch information
dmitshur committed Apr 22, 2016
1 parent 1d56631 commit 81ea1e5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion github/repos_releases_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
}

Expand Down
4 changes: 2 additions & 2 deletions tests/integration/authorizations_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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))
}

}
Expand Down

0 comments on commit 81ea1e5

Please sign in to comment.