clean up unnecessary require.NoError; use require.NoError instead of t.Fatal for consistency

Author: denik

integration/bundle/deploy_to_shared_test.go

@@ -25,7 +25,6 @@ func TestDeployBasicToSharedWorkspacePath(t *testing.T) {
 		"spark_version": defaultSparkVersion,
 		"root_path":     fmt.Sprintf("/Shared/%s", currentUser.UserName),
 	})
-	require.NoError(t, err)
 
 	t.Cleanup(func() {
 		destroyBundle(wt, ctx, bundleRoot)

integration/bundle/helpers_test.go

@@ -84,19 +84,15 @@ func deployBundle(t testutil.TestingT, ctx context.Context, path string) {
 	ctx = env.Set(ctx, "BUNDLE_ROOT", path)
 	c := testcli.NewRunnerWithContext(t, ctx, "bundle", "deploy", "--force-lock", "--auto-approve")
 	_, _, err := c.Run()
-	if err != nil {
-		t.Fatal(err)
-	}
+	require.NoError(t, err)
 }
 
 func deployBundleWithArgs(t testutil.TestingT, ctx context.Context, path string, args ...string) (string, string) {
 	ctx = env.Set(ctx, "BUNDLE_ROOT", path)
 	args = append([]string{"bundle", "deploy"}, args...)
 	c := testcli.NewRunnerWithContext(t, ctx, args...)
 	stdout, stderr, err := c.Run()
-	if err != nil {
-		t.Fatal(err)
-	}
+	require.NoError(t, err)
 	return stdout.String(), stderr.String()
 }
 
@@ -106,9 +102,7 @@ func deployBundleWithFlags(t testutil.TestingT, ctx context.Context, path string
 	args = append(args, flags...)
 	c := testcli.NewRunnerWithContext(t, ctx, args...)
 	_, _, err := c.Run()
-	if err != nil {
-		t.Fatal(err)
-	}
+	require.NoError(t, err)
 }
 
 func runResource(t testutil.TestingT, ctx context.Context, path, key string) (string, error) {