Simpler tests

Signed-off-by: David Gageot <david@gageot.net>

pkg/skaffold/apiversion/apiversion_test.go

@@ -16,21 +16,20 @@ limitations under the License.
 package apiversion
 
 import (
-	"reflect"
 	"testing"
 
 	"github.com/GoogleContainerTools/skaffold/testutil"
 	"github.com/blang/semver"
 )
 
 func TestMustParse(t *testing.T) {
-	_ = MustParse("skaffold/v1alpha4")
+	MustParse("skaffold/v1alpha4")
 }
 
 func TestMustParse_panic(t *testing.T) {
 	defer testutil.EnsureTestPanicked(t)
 
-	_ = MustParse("invalid version")
+	MustParse("invalid version")
 }
 
 func TestParseVersion(t *testing.T) {
@@ -81,13 +80,8 @@ func TestParseVersion(t *testing.T) {
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			got, err := Parse(tt.args.v)
-			if (err != nil) != tt.wantErr {
-				t.Errorf("Parse() error = %v, wantErr %v", err, tt.wantErr)
-				return
-			}
-			if !reflect.DeepEqual(got, tt.want) {
-				t.Errorf("Parse() = %v, want %v", got, tt.want)
-			}
+
+			testutil.CheckErrorAndDeepEqual(t, tt.wantErr, err, tt.want, got)
 		})
 	}
 }

pkg/skaffold/yamltags/tags_test.go

@@ -160,11 +160,11 @@ func TestValidateStructOneOf(t *testing.T) {
 func TestValidateStructInvalid(t *testing.T) {
 	defer testutil.EnsureTestPanicked(t)
 
-	invalidYamlTags := struct {
+	invalidTags := struct {
 		A string `yamltags:"invalidTag"`
-	}{A: "whatever"}
+	}{}
 
-	_ = ValidateStruct(invalidYamlTags)
+	ValidateStruct(invalidTags)
 }
 
 func TestIsZeroValue(t *testing.T) {