tests: add explicit error handling (#114)

Signed-off-by: Kent R. Spillner <kspillner@acm.org>

Co-authored-by: Kotaro Yoshimatsu <kotaro.yoshimatsu@gmail.com>

Author: sl4mmy

tests/branchrestrictions_test.go

@@ -44,7 +44,10 @@ func TestCreateBranchRestrictionsKindPush(t *testing.T) {
 		Kind:     "push",
 		Users:    []string{user},
 	}
-	res, _ := c.Repositories.BranchRestrictions.Create(opt)
+	res, err := c.Repositories.BranchRestrictions.Create(opt)
+	if err != nil {
+		t.Error(err)
+	}
 	jsonMap := res.(map[string]interface{})
 	if jsonMap["type"] != "branchrestriction" {
 		t.Error("is not branchrestriction type")
@@ -65,7 +68,10 @@ func TestCreateBranchRestrictionsKindRequirePassingBuilds(t *testing.T) {
 		Kind:     "require_passing_builds_to_merge",
 		Value:    2,
 	}
-	res, _ := c.Repositories.BranchRestrictions.Create(opt)
+	res, err := c.Repositories.BranchRestrictions.Create(opt)
+	if err != nil {
+		t.Error(err)
+	}
 	jsonMap := res.(map[string]interface{})
 	if jsonMap["type"] != "branchrestriction" {
 		t.Error("is not branchrestriction type")

tests/diff_test.go

@@ -32,7 +32,10 @@ func TestDiff(t *testing.T) {
 		RepoSlug: repo,
 		Spec:     spec,
 	}
-	res, _ := c.Repositories.Diff.GetDiff(opt)
+	res, err := c.Repositories.Diff.GetDiff(opt)
+	if err != nil {
+		t.Error(err)
+	}
 
 	pp.Println(res)