Commitc60ecc1

authored and

committed

fix(tests): improve test descriptions and error messages in TestLoad

1 parentea06b5e commitc60ecc1Copy full SHA for c60ecc1

File tree

-4

lines changed

-4

lines changed

Lines changed: 4 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -25,14 +25,14 @@ import (`
`25`	`25`	`)`
`26`	`26`
`27`	`27`	`funcTestLoad(t*testing.T) {`
`28`		`-t.Run("empty path returns an error",func(t*testing.T) {`
	`28`	`+t.Run("it fails if tha app path is empty",func(t*testing.T) {`
`29`	`29`	`app,err:=Load("")`
`30`	`30`	`assert.Error(t,err)`
`31`	`31`	`assert.Empty(t,app)`
`32`		`-assert.Contains(t,err.Error(),"empty")`
	`32`	`+assert.Contains(t,err.Error(),"empty app path")`
`33`	`33`	`})`
`34`	`34`
`35`		`-t.Run("it fails if the app pathis an existing file",func(t*testing.T) {`
	`35`	`+t.Run("it fails if the app pathexist but it's a file",func(t*testing.T) {`
`36`	`36`	`_,err:=Load("testdata/app.yaml")`
`37`	`37`	`assert.Error(t,err)`
`38`	`38`	`assert.Contains(t,err.Error(),"app path must be a directory")`
`@@ -44,7 +44,7 @@ func TestLoad(t *testing.T) {`
`44`	`44`	`assert.Contains(t,err.Error(),"app path is not valid")`
`45`	`45`	`})`
`46`	`46`
`47`		`-t.Run("itload an app correctly",func(t*testing.T) {`
	`47`	`+t.Run("itloads an app correctly",func(t*testing.T) {`
`48`	`48`	`app,err:=Load("testdata/AppSimple")`
`49`	`49`	`assert.NoError(t,err)`
`50`	`50`	`assert.NotEmpty(t,app)`

Comments

(0)