Merge pull request #66331 from foxyriver/fix-log-error

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. more clear err log **What this PR does / why we need it**: more clear err log **Release note**: ```release-note NONE ```

Merge pull request #66331 from foxyriver/fix-log-error
ad97a03e · Kubernetes Submit Queue · GitHub · e4465b6e · 5c953681 · ad97a03e
Unverified Commit ad97a03e authored Jul 26, 2018 by Kubernetes Submit Queue Committed by GitHub Jul 26, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

data_dir_test.go cluster/images/etcd/migrate/data_dir_test.go +4 -4

No files found.
--- a/cluster/images/etcd/migrate/data_dir_test.go
+++ b/cluster/images/etcd/migrate/data_dir_test.go
@@ -39,7 +39,7 @@ func TestExistingDataDirWithVersionFile(t *testing.T) {
 		t.Fatalf("Failed to check if data dir is empty: %v", err)
 	}
 	if isEmpty {
-		t.Errorf("Data directory is non-empty")
+		t.Errorf("Expected non-empty data directory to exist")
 	}
 	exists, err := d.versionFile.Exists()
 	if err != nil {
@@ -97,7 +97,7 @@ func TestNonexistingDataDir(t *testing.T) {
 		t.Fatalf("Failed to check if data dir is empty: %v", err)
 	}
 	if !isEmpty {
-		t.Errorf("Data directory is empty")
+		t.Errorf("Expected empty data directory to exist")
 	}
 	err = d.Initialize(targetVersion)
 	if err != nil {
@@ -115,7 +115,7 @@ func TestNonexistingDataDir(t *testing.T) {
 		t.Fatalf("Failed to check if data dir is empty: %v", err)
 	}
 	if isEmpty {
-		t.Errorf("Data directory is non-empty")
+		t.Errorf("Expected non-empty data directory to exist after Initialize()")
 	}
 	vp, err := d.versionFile.Read()
 	if err != nil {
@@ -145,7 +145,7 @@ func TestBackup(t *testing.T) {
 		t.Fatal(err)
 	}
 	if isEmpty {
-		t.Errorf("Expected non-empty backup directory afer Backup()")
+		t.Errorf("Expected non-empty backup directory to exist after Backup()")
 	}
 }