X-Git-Url: https://ruderich.org/simon/gitweb/?a=blobdiff_plain;f=cmd%2Fsafcm%2Fmain_sync_test.go;h=fdbe98fba8764565551f7d4e8845789957c12db4;hb=9d0d090fc1d683accc8bd1b354425c23cbe9fb6a;hp=1f53e4b338902d8f243053752a852d4b1929cc5a;hpb=79a01721cf86d424dbf76ce91d8fa66fbff2f480;p=safcm%2Fsafcm.git

diff --git a/cmd/safcm/main_sync_test.go b/cmd/safcm/main_sync_test.go
index 1f53e4b..fdbe98f 100644
--- a/cmd/safcm/main_sync_test.go
+++ b/cmd/safcm/main_sync_test.go
@@ -177,14 +177,13 @@ func TestSyncSshEndToEnd(t *testing.T) {
 			`<LOG>[info]    [no-changes.example.org] remote helper upload in progress
 <LOG>[verbose] [no-changes.example.org] host groups: all <DET> <DET> no-changes.example.org
 <LOG>[verbose] [no-changes.example.org] host group priorities (descending): no-changes.example.org
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/" (no-changes.example.org): unchanged
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/etc" (no-changes.example.org): unchanged
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/tmp" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/etc" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/tmp" (no-changes.example.org): unchanged
 <LOG>[info]    [no-changes.example.org] no changes
 `,
 			nil,
 		},
-
 		{
 			"no changes",
 			skipUnlessCiRun,
@@ -203,13 +202,145 @@ func TestSyncSshEndToEnd(t *testing.T) {
 			`<LOG>[info]    [no-changes.example.org] remote helper upload in progress
 <LOG>[verbose] [no-changes.example.org] host groups: all <DET> <DET> no-changes.example.org
 <LOG>[verbose] [no-changes.example.org] host group priorities (descending): no-changes.example.org
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/" (no-changes.example.org): unchanged
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/etc" (no-changes.example.org): unchanged
-<LOG>[debug]   [no-changes.example.org] sync remote: files: "/tmp" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/etc" (no-changes.example.org): unchanged
+<LOG>[debug]   [no-changes.example.org] files: "/tmp" (no-changes.example.org): unchanged
 <LOG>[info]    [no-changes.example.org] no changes
 `,
 			nil,
 		},
+
+		{
+			"no effect commands (dry-run)",
+			false,
+			true,
+			[]string{"-n", "no-effect-commands.example.org"},
+			`<LOG>[info]    [no-effect-commands.example.org] remote helper upload in progress
+<LOG>[info]    [no-effect-commands.example.org] 
+will execute 2 command(s): (dry-run)
+"echo this is a command"
+"true"
+`,
+			nil,
+		},
+		{
+			"no effect commands (dry-run)",
+			false,
+			true,
+			[]string{"-n", "-log", "debug2", "no-effect-commands.example.org"},
+			`<LOG>[info]    [no-effect-commands.example.org] remote helper upload in progress
+<LOG>[verbose] [no-effect-commands.example.org] host groups: all <DET> <DET> no-effect-commands.example.org
+<LOG>[verbose] [no-effect-commands.example.org] host group priorities (descending): no-effect-commands.example.org
+<LOG>[info]    [no-effect-commands.example.org] 
+will execute 2 command(s): (dry-run)
+"echo this is a command"
+"true"
+`,
+			nil,
+		},
+		{
+			"no effect commands",
+			false,
+			true,
+			[]string{"no-effect-commands.example.org"},
+			`<LOG>[info]    [no-effect-commands.example.org] remote helper upload in progress
+<LOG>[info]    [no-effect-commands.example.org] 
+executed 2 command(s):
+"echo this is a command":
+   > this is a command
+"true"
+`,
+			nil,
+		},
+		{
+			"no effect commands (debug2)",
+			false,
+			true,
+			[]string{"-log", "debug2", "no-effect-commands.example.org"},
+			`<LOG>[info]    [no-effect-commands.example.org] remote helper upload in progress
+<LOG>[verbose] [no-effect-commands.example.org] host groups: all <DET> <DET> no-effect-commands.example.org
+<LOG>[verbose] [no-effect-commands.example.org] host group priorities (descending): no-effect-commands.example.org
+<LOG>[verbose] [no-effect-commands.example.org] commands: running "/bin/sh" "-c" "echo this is a command" (no-effect-commands.example.org)
+<LOG>[debug2]  [no-effect-commands.example.org] commands: command output:
+this is a command
+<LOG>[verbose] [no-effect-commands.example.org] commands: running "/bin/sh" "-c" "true" (no-effect-commands.example.org)
+<LOG>[info]    [no-effect-commands.example.org] 
+executed 2 command(s):
+"echo this is a command":
+   > this is a command
+"true"
+`,
+			nil,
+		},
+
+		{
+			"no effect commands failing (dry-run)",
+			false,
+			true,
+			[]string{"-n", "no-effect-commands-failing.example.org"},
+			`<LOG>[info]    [no-effect-commands-failing.example.org] remote helper upload in progress
+<LOG>[info]    [no-effect-commands-failing.example.org] 
+will execute 2 command(s): (dry-run)
+"echo this is a command"
+"echo failing; false"
+`,
+			nil,
+		},
+		{
+			"no effect commands failing (dry-run)",
+			false,
+			true,
+			[]string{"-n", "-log", "debug2", "no-effect-commands-failing.example.org"},
+			`<LOG>[info]    [no-effect-commands-failing.example.org] remote helper upload in progress
+<LOG>[verbose] [no-effect-commands-failing.example.org] host groups: all <DET> <DET> no-effect-commands-failing.example.org
+<LOG>[verbose] [no-effect-commands-failing.example.org] host group priorities (descending): no-effect-commands-failing.example.org
+<LOG>[info]    [no-effect-commands-failing.example.org] 
+will execute 2 command(s): (dry-run)
+"echo this is a command"
+"echo failing; false"
+`,
+			nil,
+		},
+		{
+			"no effect commands failing",
+			false,
+			true,
+			[]string{"no-effect-commands-failing.example.org"},
+			`<LOG>[info]    [no-effect-commands-failing.example.org] remote helper upload in progress
+<LOG>[info]    [no-effect-commands-failing.example.org] 
+executed 2 command(s):
+"echo this is a command":
+   > this is a command
+"echo failing; false", failed: "exit status 1":
+   > failing
+<LOG>[error]   [no-effect-commands-failing.example.org] commands: "echo failing; false" failed: exit status 1
+`,
+			fmt.Errorf("exit status 1"),
+		},
+		{
+			"no effect commands failing (debug2)",
+			false,
+			true,
+			[]string{"-log", "debug2", "no-effect-commands-failing.example.org"},
+			`<LOG>[info]    [no-effect-commands-failing.example.org] remote helper upload in progress
+<LOG>[verbose] [no-effect-commands-failing.example.org] host groups: all <DET> <DET> no-effect-commands-failing.example.org
+<LOG>[verbose] [no-effect-commands-failing.example.org] host group priorities (descending): no-effect-commands-failing.example.org
+<LOG>[verbose] [no-effect-commands-failing.example.org] commands: running "/bin/sh" "-c" "echo this is a command" (no-effect-commands-failing.example.org)
+<LOG>[debug2]  [no-effect-commands-failing.example.org] commands: command output:
+this is a command
+<LOG>[verbose] [no-effect-commands-failing.example.org] commands: running "/bin/sh" "-c" "echo failing; false" (no-effect-commands-failing.example.org)
+<LOG>[debug2]  [no-effect-commands-failing.example.org] commands: command output:
+failing
+<LOG>[info]    [no-effect-commands-failing.example.org] 
+executed 2 command(s):
+"echo this is a command":
+   > this is a command
+"echo failing; false", failed: "exit status 1":
+   > failing
+<LOG>[error]   [no-effect-commands-failing.example.org] commands: "echo failing; false" failed: exit status 1
+`,
+			fmt.Errorf("exit status 1"),
+		},
 	}
 
 	remotePath := fmt.Sprintf("/tmp/safcm-remote-%d", os.Getuid())
@@ -217,6 +348,20 @@ func TestSyncSshEndToEnd(t *testing.T) {
 	logRegexp := regexp.MustCompile(`^\d{4}/\d{2}/\d{2} \d{2}:\d{2}:\d{2} `)
 	detectedRegexp := regexp.MustCompile(`detected_\S+`)
 
+	t.Run("error before connection is established", func(t *testing.T) {
+		// Fake $PATH so safcm cannot find the `ssh` binary.
+		path := os.Getenv("PATH")
+		os.Setenv("PATH", "")
+		defer os.Setenv("PATH", path)
+
+		cmd := exec.Command("../../../../../safcm",
+			"sync", "-n", "no-settings.example.org")
+		_, err := cmd.CombinedOutput()
+		if err == nil {
+			t.Errorf("err = nil")
+		}
+	})
+
 	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {
 			if tc.remove {