Make migration submit test consistent w/ unit test

Unify the assertions on metadata existing
nywilken · Aug 30, 2018 · 5d4e34b · 5d4e34b
1 parent 305edf2
commit 5d4e34b
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/cmd/submit_test.go b/cmd/submit_test.go
@@ -232,17 +232,20 @@ func TestLegacySolutionMetadataMigration(t *testing.T) {
 		Dir:             tmpDir,
 		UserViperConfig: v,
 	}
-	_, err = os.Stat(exercise.MetadataFilepath())
-	assert.Error(t, err)
+
+	ok, _ := exercise.HasLegacyMetadata()
+	assert.True(t, ok)
+	ok, _ = exercise.HasMetadata()
+	assert.False(t, ok)
 
 	err = runSubmit(cfg, pflag.NewFlagSet("fake", pflag.PanicOnError), []string{file})
 	assert.NoError(t, err)
 	assert.Equal(t, "This is a file.", submittedFiles["file.txt"])
 
-	_, err = os.Stat(exercise.MetadataFilepath())
-	assert.NoError(t, err)
-	_, err = os.Stat(exercise.LegacyMetadataFilepath())
-	assert.Error(t, err)
+	ok, _ = exercise.HasLegacyMetadata()
+	assert.False(t, ok)
+	ok, _ = exercise.HasMetadata()
+	assert.True(t, ok)
 }
 
 func TestSubmitWithEmptyFile(t *testing.T) {