tests for shell

main.go

@@ -271,8 +271,6 @@ func run(
 	runner *remoteExecutionRunner,
 	stdin string,
 ) error {
-	debugf(`running command`)
-
 	execution, err := runner.run(cluster, nil)
 	if err != nil {
 		return hierr.Errorf(

remote_execution_runner.go

@@ -33,14 +33,18 @@ func wrapCommandIntoShell(command string, shell string, args []string) string {
 		return command
 	}
 
+	command = strings.Replace(shell, `{}`, command, -1)
+
+	if len(args) == 0 {
+		return command
+	}
+
 	escapedArgs := []string{}
 	for _, arg := range args {
 		escapedArgs = append(escapedArgs, escapeCommandArgumentStrict(arg))
 	}
 
-	return strings.Replace(shell, `{}`, command, -1) +
-		" _ " +
-		strings.Join(escapedArgs, " ")
+	return command + " _ " + strings.Join(escapedArgs, " ")
 }
 
 func joinCommand(command []string) string {

tests/testcases/sync/can-create-run-directory-under-sudo.test.sh

@@ -0,0 +1,17 @@
+tests:put test-file <<EOF
+line1
+line2
+EOF
+
+tests:ensure :orgalorg:with-key -exmn 'true' -S test-file
+
+:check-uploaded-file() {
+    local container_ip="$2"
+
+    # We need ssh there, because of /var/run is a tmpfs and not accessible
+    # from host.
+    tests:ensure :ssh "$container_ip" cat "/var/run/orgalorg/*/test-file"
+    tests:assert-no-diff "test-file" "stdout"
+}
+
+containers:do :check-uploaded-file

tests/testcases/sync/can-pass-additional-args-to-sync-tool.test.sh

@@ -0,0 +1,9 @@
+tests:put test-file <<EOF
+line1
+line2
+EOF
+
+tests:ensure :orgalorg:with-key -emn 'echo 1:$1 2:$2' \
+    --args first --args second -r /tmp -S test-file
+
+tests:assert-stdout "1:first 2:second"