fix diff script parsing issue (#127)

* fix failing tests

* test script

* diff command supports eval

* enable logging in tests

* upgrade version in readme

Author: adikari

.buildkite/pipeline.yml

@@ -34,7 +34,7 @@ steps:
 
   - label: ":bomb: Testing groups"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "cat ./e2e/multiple-paths"
           watch:
             - path:
@@ -46,7 +46,7 @@ steps:
 
   - label: ":bomb: Testing hooks"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "cat ./e2e/multiple-paths"
           watch:
             - path: "user-service/"

README.md

@@ -18,7 +18,7 @@ If the version number is not provided then the most recent version of the plugin
 steps:
   - label: "Triggering pipelines"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "git diff --name-only HEAD~1"
           watch:
             - path: "bar-service/"
@@ -35,7 +35,7 @@ steps:
 steps:
   - label: "Triggering pipelines"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "git diff --name-only $(head -n 1 last_successful_build)"
           interpolation: false
           env:
@@ -151,7 +151,7 @@ Add `log_level` property to set the log level. Supported log levels are `debug`
 steps:
   - label: "Triggering pipelines"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "git diff --name-only HEAD~1"
           log_level: "debug" # defaults to "info"
           watch:
@@ -227,7 +227,7 @@ hooks:
 steps:
   - label: "Triggering pipelines"
     plugins:
-      - monebag/monorepo-diff#v2.5.6:
+      - monebag/monorepo-diff#v2.5.7:
           diff: "git diff --name-only HEAD~1"
           watch:
             - path: app/cms/

main.go

@@ -37,6 +37,10 @@ func main() {
 
 	setupLogger(plugin.LogLevel)
 
+	if env("BUILDKITE_PLUGIN_MONOREPO_DIFF_BUILDKITE_PLUGIN_TEST_MODE", "false") == "true" {
+		return
+	}
+
 	if _, _, err = uploadPipeline(plugin, generatePipeline); err != nil {
 		log.Fatalf("+++ failed to upload pipeline: %v", err)
 	}

main_test.go

@@ -1,7 +1,6 @@
 package main
 
 import (
-	"io/ioutil"
 	"os"
 	"testing"
 
@@ -12,9 +11,6 @@ import (
 func TestMain(m *testing.M) {
 	log.SetLevel(log.DebugLevel)
 
-	// disable logs in test
-	log.SetOutput(ioutil.Discard)
-
 	// set some env variables for using in tests
 	os.Setenv("BUILDKITE_COMMIT", "123")
 	os.Setenv("BUILDKITE_MESSAGE", "fix: temp file not correctly deleted")

pipeline.go

@@ -84,19 +84,16 @@ func uploadPipeline(plugin Plugin, generatePipeline PipelineGenerator) (string,
 func diff(command string) ([]string, error) {
 	log.Infof("Running diff command: %s", command)
 
-	split := strings.Split(command, " ")
-	cmd, args := split[0], split[1:]
+	output, err := executeCommand(
+		env("SHELL", "bash"),
+		[]string{"-c", strings.Replace(command, "\n", " ", -1)},
+	)
 
-	output, err := executeCommand(cmd, args)
 	if err != nil {
 		return nil, fmt.Errorf("diff command failed: %v", err)
 	}
 
-	f := func(c rune) bool {
-		return c == '\n'
-	}
-
-	return strings.FieldsFunc(strings.TrimSpace(output), f), nil
+	return strings.Fields(strings.TrimSpace(output)), nil
 }
 
 func stepsToTrigger(files []string, watch []WatchConfig) ([]Step, error) {

pipeline_test.go

@@ -61,6 +61,16 @@ README.md`)
 	assert.Equal(t, want, got)
 }
 
+func TestDiffWithSubshell(t *testing.T) {
+	want := []string{
+		"user-service/infrastructure/cloudfront.yaml",
+		"user-service/serverless.yaml",
+	}
+	got, err := diff("echo $(cat e2e/multiple-paths)")
+	assert.NoError(t, err)
+	assert.Equal(t, want, got)
+}
+
 func TestPipelinesToTriggerGetsListOfPipelines(t *testing.T) {
 	want := []string{"service-1", "service-2", "service-4"}
 

tests/command.bats

@@ -257,9 +257,7 @@ EOM
 
   assert_success
 
-  assert_output --partial "--- running monorepo-diff"
-  assert_output --partial "Running diff command: echo foo-service/ \nbat-service/"
-  assert_output --partial "Output from diff: \nfoo-service/ \nbat-service/"
+  assert_output --partial "--- running monorepo-diff-buildkite-plugin"
 
   assert_output --partial << EOM
 steps: