Merge pull request docker#291 from thaJeztah/test_nits

credentials: fix minor nits in tests
sudoforge · May 30, 2023 · c842499 · c842499
2 parents 83d38ea + c1c7dd7
commit c842499
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/credentials/credentials_test.go b/credentials/credentials_test.go
@@ -88,7 +88,7 @@ func TestStoreMissingServerURL(t *testing.T) {
 
 	h := newMemoryStore()
 
-	if err := Store(h, in); IsCredentialsMissingServerURL(err) == false {
+	if err := Store(h, in); !IsCredentialsMissingServerURL(err) {
 		t.Error(err)
 	}
 }
@@ -108,7 +108,7 @@ func TestStoreMissingUsername(t *testing.T) {
 
 	h := newMemoryStore()
 
-	if err := Store(h, in); IsCredentialsMissingUsername(err) == false {
+	if err := Store(h, in); !IsCredentialsMissingUsername(err) {
 		t.Error(err)
 	}
 }
@@ -176,7 +176,7 @@ func TestGetMissingServerURL(t *testing.T) {
 	buf := strings.NewReader("")
 	w := new(bytes.Buffer)
 
-	if err := Get(h, buf, w); IsCredentialsMissingServerURL(err) == false {
+	if err := Get(h, buf, w); !IsCredentialsMissingServerURL(err) {
 		t.Error(err)
 	}
 }
@@ -229,8 +229,8 @@ func TestEraseMissingServerURL(t *testing.T) {
 	}
 
 	buf := strings.NewReader("")
-	if err := Erase(h, buf); IsCredentialsMissingServerURL(err) == false {
-		t.Fatal(err)
+	if err := Erase(h, buf); !IsCredentialsMissingServerURL(err) {
+		t.Error(err)
 	}
 }