Improve style and readability

Author: MatteoPologruto

Diff for: internal/integrationtest/lib/lib_test.go

@@ -1081,7 +1081,7 @@ func TestLibCommandsWithLibraryHavingInvalidVersion(t *testing.T) {
 	stdout, _, err := cli.Run("lib", "list", "--format", "json")
 	require.NoError(t, err)
 	requirejson.Len(t, stdout, 1)
-	requirejson.Query(t, stdout, ".[0] | .library | .version", "\"0.16.1\"")
+	requirejson.Query(t, stdout, ".[0] | .library | .version", `"0.16.1"`)
 
 	// Changes the version of the currently installed library so that it's
 	// invalid
@@ -1119,9 +1119,7 @@ func TestInstallZipLibWithMacosMetadata(t *testing.T) {
 	// Verifies library is not already installed
 	require.NoDirExists(t, libInstallDir.String())
 
-	wd, err := paths.Getwd()
-	require.NoError(t, err)
-	zipPath := wd.Parent().Join("testdata", "fake-lib.zip")
+	zipPath := paths.New("..", "testdata", "fake-lib.zip")
 	// Test zip-path install
 	stdout, _, err := cli.Run("lib", "install", "--zip-path", zipPath.String())
 	require.NoError(t, err)
@@ -1156,9 +1154,7 @@ func TestInstallZipInvalidLibrary(t *testing.T) {
 	// Verifies library is not already installed
 	require.NoDirExists(t, libInstallDir.String())
 
-	wd, err := paths.Getwd()
-	require.NoError(t, err)
-	zipPath := wd.Parent().Join("testdata", "lib-without-header.zip")
+	zipPath := paths.New("..", "testdata", "lib-without-header.zip")
 	// Test zip-path install
 	_, stderr, err := cli.Run("lib", "install", "--zip-path", zipPath.String())
 	require.Error(t, err)
@@ -1168,7 +1164,7 @@ func TestInstallZipInvalidLibrary(t *testing.T) {
 	// Verifies library is not already installed
 	require.NoDirExists(t, libInstallDir.String())
 
-	zipPath = wd.Parent().Join("testdata", "lib-without-properties.zip")
+	zipPath = paths.New("..", "testdata", "lib-without-properties.zip")
 	// Test zip-path install
 	_, stderr, err = cli.Run("lib", "install", "--zip-path", zipPath.String())
 	require.Error(t, err)
@@ -1192,7 +1188,7 @@ func TestInstallGitInvalidLibrary(t *testing.T) {
 	libProperties := repoDir.Join("library.properties")
 	f, err := libProperties.Create()
 	require.NoError(t, err)
-	f.Close()
+	require.NoError(t, f.Close())
 	tree, err := repo.Worktree()
 	require.NoError(t, err)
 	_, err = tree.Add("library.properties")
@@ -1218,7 +1214,7 @@ func TestInstallGitInvalidLibrary(t *testing.T) {
 	require.NoError(t, libHeader.Parent().MkdirAll())
 	f, err = libHeader.Create()
 	require.NoError(t, err)
-	f.Close()
+	require.NoError(t, f.Close())
 	tree, err = repo.Worktree()
 	require.NoError(t, err)
 	_, err = tree.Add("src/lib-without-properties.h")
@@ -1246,11 +1242,9 @@ func TestUpgradeDoesNotTryToUpgradeBundledCoreLibrariesInSketchbook(t *testing.T
 	require.NoError(t, platformInstallDir.Parent().MkdirAll())
 
 	// Install platform in Sketchbook hardware dir
-	wd, err := paths.Getwd()
-	require.NoError(t, err)
-	require.NoError(t, wd.Parent().Join("testdata", testPlatformName).CopyDirTo(platformInstallDir))
+	require.NoError(t, paths.New("..", "testdata", testPlatformName).CopyDirTo(platformInstallDir))
 
-	_, _, err = cli.Run("update")
+	_, _, err := cli.Run("update")
 	require.NoError(t, err)
 
 	// Install latest version of library identical to one
@@ -1262,8 +1256,8 @@ func TestUpgradeDoesNotTryToUpgradeBundledCoreLibrariesInSketchbook(t *testing.T
 	require.NoError(t, err)
 	requirejson.Len(t, stdout, 2)
 	// Verify both libraries have the same name
-	requirejson.Query(t, stdout, ".[0] | .library | .name", "\"USBHost\"")
-	requirejson.Query(t, stdout, ".[1] | .library | .name", "\"USBHost\"")
+	requirejson.Query(t, stdout, ".[0] | .library | .name", `"USBHost"`)
+	requirejson.Query(t, stdout, ".[1] | .library | .name", `"USBHost"`)
 
 	stdout, _, err = cli.Run("lib", "upgrade")
 	require.NoError(t, err)
@@ -1280,11 +1274,9 @@ func TestUpgradeDoesNotTryToUpgradeBundledCoreLibraries(t *testing.T) {
 	require.NoError(t, platformInstallDir.Parent().MkdirAll())
 
 	// Install platform in Sketchbook hardware dir
-	wd, err := paths.Getwd()
-	require.NoError(t, err)
-	require.NoError(t, wd.Parent().Join("testdata", testPlatformName).CopyDirTo(platformInstallDir))
+	require.NoError(t, paths.New("..", "testdata", testPlatformName).CopyDirTo(platformInstallDir))
 
-	_, _, err = cli.Run("update")
+	_, _, err := cli.Run("update")
 	require.NoError(t, err)
 
 	// Install latest version of library identical to one
@@ -1296,8 +1288,8 @@ func TestUpgradeDoesNotTryToUpgradeBundledCoreLibraries(t *testing.T) {
 	require.NoError(t, err)
 	requirejson.Len(t, stdout, 2)
 	// Verify both libraries have the same name
-	requirejson.Query(t, stdout, ".[0] | .library | .name", "\"USBHost\"")
-	requirejson.Query(t, stdout, ".[1] | .library | .name", "\"USBHost\"")
+	requirejson.Query(t, stdout, ".[0] | .library | .name", `"USBHost"`)
+	requirejson.Query(t, stdout, ".[1] | .library | .name", `"USBHost"`)
 
 	stdout, _, err = cli.Run("lib", "upgrade")
 	require.NoError(t, err)
@@ -1472,7 +1464,7 @@ func TestInstallWithGitUrlLocalFileUri(t *testing.T) {
 	})
 	require.NoError(t, err)
 
-	_, _, err = cli.RunWithCustomEnv(envVar, "lib", "install", "--git-url", "file:///"+repoDir.String())
+	_, _, err = cli.RunWithCustomEnv(envVar, "lib", "install", "--git-url", "file://"+repoDir.String())
 	require.NoError(t, err)
 
 	// Verifies library is installed