Added unti tests for utils.GetCompatibleWith

Author: cmaglie

go.mod

@@ -7,6 +7,8 @@ replace go.bug.st/serial => github.com/cmaglie/go-serial v0.0.0-20200923162623-b
 
 require (
 	github.com/arduino/arduino-cli v0.0.0-20200924151007-69ac12c98b2b
+	github.com/arduino/go-paths-helper v1.3.2
 	github.com/pkg/errors v0.9.1
+	github.com/stretchr/testify v1.6.1
 	go.bug.st/serial v1.1.1
 )

main.go

@@ -34,7 +34,7 @@ func main() {
 	flag.Parse()
 
 	if ctx.Compatible != "" {
-		el, _ := json.Marshal(utils.GetCompatibleWith(ctx.Compatible))
+		el, _ := json.Marshal(utils.GetCompatibleWith(ctx.Compatible, ""))
 		fmt.Println(string(el))
 		os.Exit(0)
 	}

utils/utils.go

@@ -34,7 +34,7 @@ func isPreferred(existing bool, path string, board combo) bool {
 	return true
 }
 
-func GetCompatibleWith(name string) map[string][]firmware {
+func GetCompatibleWith(name string, rootPath string) map[string][]firmware {
 
 	files := make(map[string][]firmware)
 
@@ -51,8 +51,10 @@ func GetCompatibleWith(name string) map[string][]firmware {
 	if knownBoards[strings.ToLower(name)].match == "" {
 		listAll = true
 	}
-
-	exePath, _ := os.Executable()
+	exePath := rootPath
+	if exePath == "" {
+		exePath, _ = os.Executable()
+	}
 	root := filepath.Dir(exePath)
 	root = filepath.Join(root, "firmwares")
 	loader := regexp.MustCompile(knownBoards[name].loader)

utils/utils_test.go

@@ -0,0 +1,34 @@
+package utils
+
+import (
+	"testing"
+
+	"github.com/arduino/go-paths-helper"
+	"github.com/stretchr/testify/require"
+)
+
+func TestGetCompatibleWith(t *testing.T) {
+	root, err := paths.Getwd()
+	require.NoError(t, err)
+	require.NoError(t, root.ToAbs())
+	testrunner := func(board string) {
+		res := GetCompatibleWith(board, root.String())
+		require.NotNil(t, res)
+		hasLoader := false
+		for _, e := range res {
+			for _, i := range e {
+				if i.IsLoader {
+					require.False(t, hasLoader, "loader must be unique")
+					hasLoader = true
+					require.NotEmpty(t, i.Name)
+					require.NotEmpty(t, i.Path)
+				}
+			}
+		}
+		require.True(t, hasLoader, "loader must be present")
+	}
+
+	testrunner("mkrwifi1010")
+	testrunner("mkr1000")
+	testrunner("nano_33_iot")
+}