Add tests for steps within group steps

Author: petetomasik

pipeline_test.go

@@ -825,3 +825,36 @@ func TestGeneratePipelineWithSecretsInGroup(t *testing.T) {
 	assert.Contains(t, string(got), "- PROD_DB_HOST")
 	assert.Contains(t, string(got), "- PROD_DB_PASS")
 }
+
+func TestGeneratePipelineWithNotifyInGroup(t *testing.T) {
+	steps := []Step{{
+		Group: "Test Group",
+		Steps: []Step{{
+			Label:   "Run Tests",
+			Command: "echo 'test'",
+			Notify: []StepNotify{{
+				GithubStatus: GithubStatusNotification{
+					Context: "buildkite/test/status",
+				},
+			}},
+		}},
+	}}
+
+	plugin := Plugin{}
+
+	tmp, hasPipeline, err := generatePipeline(steps, plugin)
+	assert.NoError(t, err)
+	assert.True(t, hasPipeline)
+	defer os.Remove(tmp.Name())
+
+	content, err := os.ReadFile(tmp.Name())
+	assert.NoError(t, err)
+
+	output := string(content)
+
+	// Verify correct notify syntax (not rawnotify)
+	assert.Contains(t, output, "notify:")
+	assert.NotContains(t, output, "rawnotify")
+	assert.Contains(t, output, "github_commit_status:")
+	assert.Contains(t, output, "context: buildkite/test/status")
+}

plugin_test.go

@@ -944,6 +944,43 @@ func TestPluginShouldClearRawEnvFromNestedSteps(t *testing.T) {
 	assert.Nil(t, secondStep.Build.RawEnv, "Second nested step Build.RawEnv should be nil")
 }
 
+func TestPluginShouldProcessNotifyInNestedSteps(t *testing.T) {
+	param := `[{
+		"github.com/buildkite-plugins/monorepo-diff-buildkite-plugin#v1.0.0": {
+			"watch": [{
+				"path": "foo/**",
+				"config": {
+					"group": "Foo",
+					"steps": [{
+						"command": "./scripts/a.sh",
+						"label": "Run A",
+						"notify": [{
+							"github_commit_status": {
+								"context": "buildkite/foo/a"
+							}
+						}]
+					}]
+				}
+			}]
+		}
+	}]`
+
+	got, err := initializePlugin(param)
+	assert.NoError(t, err)
+
+	// Verify nested step exists
+	assert.Equal(t, 1, len(got.Watch[0].Step.Steps))
+
+	nestedStep := got.Watch[0].Step.Steps[0]
+
+	// Verify RawNotify is cleared
+	assert.Nil(t, nestedStep.RawNotify, "RawNotify should be nil after processing")
+
+	// Verify Notify is populated
+	assert.Equal(t, 1, len(nestedStep.Notify), "Notify should have 1 entry")
+	assert.Equal(t, "buildkite/foo/a", nestedStep.Notify[0].GithubStatus.Context)
+}
+
 func TestPluginShouldPreserveDependsOnString(t *testing.T) {
 	param := `[{
 		"github.com/buildkite-plugins/monorepo-diff-buildkite-plugin#commit": {