Skip to content

Commit

Permalink
Merge pull request #2136 from dgageot/fix-yoda-condition
Browse files Browse the repository at this point in the history
Fix yoda condition
  • Loading branch information
priyawadhwa authored May 16, 2019
2 parents 17f7009 + 58d6a78 commit 14436d7
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions cmd/skaffold/app/flags/build_output_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -96,15 +96,7 @@ func TestBuildOutputSet(t *testing.T) {

func TestBuildOutputString(t *testing.T) {
flag := NewBuildOutputFileFlag("test.in")
if "test.in" != flag.String() {
t.Errorf("Flag String() does not match. Expected test.in, Actual %s", flag.String())
}
}

func TestBuildOutputType(t *testing.T) {
flag := NewBuildOutputFileFlag("test.in")
expectedFlagType := "*flags.BuildOutputFileFlag"
if flag.Type() != expectedFlagType {
t.Errorf("Flag returned wrong type. Expected %s, Actual %s", expectedFlagType, flag.Type())
}
testutil.CheckDeepEqual(t, "test.in", flag.String())
testutil.CheckDeepEqual(t, "*flags.BuildOutputFileFlag", flag.Type())
}

0 comments on commit 14436d7

Please sign in to comment.