Migrated TestLogOptions to main_test.go and deleted test_main.py

Author: MatteoPologruto

internal/integrationtest/main/main_test.go

@@ -25,6 +25,7 @@ import (
 	"github.com/stretchr/testify/require"
 	semver "go.bug.st/relaxed-semver"
 	"go.bug.st/testsuite"
+	"go.bug.st/testsuite/requirejson"
 )
 
 func TestHelp(t *testing.T) {
@@ -69,8 +70,7 @@ func TestVersion(t *testing.T) {
 	// Checks if Application's value is arduino-cli
 	require.Equal(t, jsonMap["Application"], "arduino-cli")
 
-	// Checks if VersionString's value is git-snapshot,
-	// nightly or a valid semantic versioning
+	// Checks if VersionString's value is git-snapshot, nightly or a valid semantic versioning
 	switch version := jsonMap["VersionString"]; version {
 	case "git-snapshot":
 		require.Contains(t, version, "git-snapshot")
@@ -85,9 +85,60 @@ func TestVersion(t *testing.T) {
 	require.NotEmpty(t, jsonMap["Commit"])
 }
 
-func TestInventoryCreation(t *testing.T) {
+func TestLogOptions(t *testing.T) {
 	// Using version as a test command
+	env := testsuite.NewEnvironment(t)
+	defer env.CleanUp()
+
+	cli := integrationtest.NewArduinoCliWithinEnvironment(env, &integrationtest.ArduinoCLIConfig{
+		ArduinoCLIPath:         paths.New("..", "..", "..", "arduino-cli"),
+		UseSharedStagingFolder: true,
+	})
+
+	// No logs
+	stdout, _, _ := cli.Run("version")
+	trimOut := strings.TrimSpace(string(stdout))
+	outLines := strings.Split(trimOut, "\n")
+	require.Len(t, outLines, 1)
 
+	// Plain text logs on stdout
+	stdout, _, _ = cli.Run("version", "-v")
+	trimOut = strings.TrimSpace(string(stdout))
+	outLines = strings.Split(trimOut, "\n")
+	require.Greater(t, len(outLines), 1)
+	require.True(t, strings.HasPrefix(outLines[0], "\x1b[36mINFO\x1b[0m")) // account for the colors
+
+	// Plain text logs on file
+	logFile := cli.DataDir().Join("log.txt")
+	_, _, _ = cli.Run("version", "--log-file", logFile.String())
+	lines, _ := logFile.ReadFileAsLines()
+	require.True(t, strings.HasPrefix(lines[0], "time=\""))
+
+	// json on stdout
+	stdout, _, _ = cli.Run("version", "-v", "--log-format", "JSON")
+	trimOut = strings.TrimSpace(string(stdout))
+	outLines = strings.Split(trimOut, "\n")
+	requirejson.Contains(t, []byte(outLines[0]), `{ "level" }`)
+
+	// Check if log.json contains readable json in each line
+	var v interface{}
+	logFileJson := cli.DataDir().Join("log.json")
+	_, _, _ = cli.Run("version", "--log-format", "JSON", "--log-file", logFileJson.String())
+	fileContent, err := logFileJson.ReadFileAsLines()
+	require.NoError(t, err)
+	for _, line := range fileContent {
+		// it seems that vscode has problems reading a stream of JSONs. In this case, the last line of the file is empty and
+		// causes the check to fail
+		if line == "" {
+			continue
+		}
+		err = json.Unmarshal([]byte(line), &v)
+		require.NoError(t, err)
+	}
+}
+
+func TestInventoryCreation(t *testing.T) {
+	// Using version as a test command
 	env := testsuite.NewEnvironment(t)
 	defer env.CleanUp()