fix(check): print error about failing files now

Signed-off-by: Felipe Zipitria <[email protected]>

cmd/check.go

@@ -2,8 +2,10 @@ package cmd
 
 import (
 	"fmt"
+	"os"
 
 	"github.com/fzipi/go-ftw/test"
+	"github.com/rs/zerolog/log"
 
 	"github.com/kyokomi/emoji"
 	"github.com/spf13/cobra"
@@ -26,10 +28,16 @@ func init() {
 }
 
 func checkFiles(dir string) {
-	files := fmt.Sprintf("%s/**/*.y[a]ml", dir)
+	var exit int
+	files := fmt.Sprintf("%s/**/*.yaml", dir)
+	log.Debug().Msgf("ftw/check: checking files using glob pattern: %s", files)
 	tests, err := test.GetTestsFromFiles(files)
 	if err != nil {
-		emoji.Printf("ftw: :red_cross: oops, found %s\n", err.Error())
+		emoji.Printf("ftw/check: :collision: oops, found %s\n", err.Error())
+		exit = 1
+	} else {
+		emoji.Printf("ftw/check: checked %d files, everything looks good!\n", len(tests))
+		exit = 0
 	}
-	emoji.Printf("ftw: checked %d files, everything looks good!\n", len(tests))
+	os.Exit(exit)
 }

test/files.go

@@ -14,18 +14,23 @@ func GetTestsFromFiles(globPattern string) ([]FTWTest, error) {
 	var tests []FTWTest
 	var err error
 
-	testFiles, _ := filepath.Glob(globPattern)
+	testFiles, err := filepath.Glob(globPattern)
+
+	if err != nil {
+		log.Info().Msgf("ftw/test: error getting test files from %s", globPattern)
+		return tests, err
+	}
 
 	for _, test := range testFiles {
 		t, err := readTest(test)
 		if err != nil {
-			break
-		} else {
-			tests = append(tests, *t)
+			log.Debug().Msgf("ftw/test: error reading %s file. Is it patched?", test)
+			return tests, err
 		}
+		tests = append(tests, *t)
 	}
 
-	return tests, err
+	return tests, nil
 }
 
 func readTest(filename string) (t *FTWTest, err error) {

test/files_test.go

@@ -65,7 +65,7 @@ func TestGetFromBadYAML(t *testing.T) {
 	filename, _ := utils.CreateTempFileWithContent(wrongYamlTest, "test-yaml-*")
 	_, err := GetTestsFromFiles(filename)
 
-	if err != nil {
+	if err == nil {
 		t.Fatalf("Error!")
 	}
 }