test: fix vacuous assertions in compiler_tests.rs (section 5) (#1476)

Five tests in the 3700-4800 line range only called assert_valid_yaml
with no additional content assertions, providing no regression value
beyond a parse check.

- test_standalone_pipeline_trigger_compiled_output_is_valid_yaml:
  add assertions on pipelines resource block, upstream name/project,
  and disabled CI/PR triggers.
- test_skip_integrity_valid_yaml_standalone: add assertion that
  'Verify pipeline integrity' is absent when --skip-integrity is passed.
- test_skip_integrity_valid_yaml_1es: same fix for the 1ES fixture.
- test_debug_pipeline_valid_yaml_standalone: add assertions that
  DEBUG="*" and 'Verify MCP backends' are present with --debug-pipeline.
- test_debug_pipeline_valid_yaml_1es: same fix for the 1ES fixture.

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: github-actions[bot]

tests/compiler_tests.rs

@@ -4345,11 +4345,31 @@ fn test_standalone_complete_agent_has_setup_and_teardown_jobs() {
     );
 }
 
-/// Test that the pipeline-trigger fixture produces valid YAML
+/// Test that the pipeline-trigger fixture produces valid YAML with correct structure
 #[test]
 fn test_standalone_pipeline_trigger_compiled_output_is_valid_yaml() {
     let compiled = compile_fixture("pipeline-trigger-agent.md");
     assert_valid_yaml(&compiled, "pipeline-trigger-agent.md");
+
+    // Verify pipeline resource block is present
+    assert!(
+        compiled.contains("pipelines:"),
+        "pipeline-trigger output should contain a pipelines resource block"
+    );
+    assert!(
+        compiled.contains("Build Pipeline"),
+        "pipeline-trigger output should reference the upstream pipeline by name"
+    );
+    assert!(
+        compiled.contains("OtherProject"),
+        "pipeline-trigger output should reference the upstream project"
+    );
+    // CI and PR triggers must be disabled — pipeline resource is the only trigger
+    assert!(
+        compiled.contains("trigger: none"),
+        "pipeline-trigger output should disable CI trigger"
+    );
+    assert!(compiled.contains("pr: none"), "pipeline-trigger output should disable PR trigger");
 }
 
 // ─── --skip-integrity flag tests ─────────────────────────────────────────
@@ -4381,17 +4401,27 @@ fn test_default_includes_integrity_step() {
     );
 }
 
-/// Test that --skip-integrity produces valid YAML for both standalone and 1ES
+/// Test that --skip-integrity produces valid YAML and removes the integrity check
+/// for both standalone (complete-agent) and 1ES targets. The minimal-agent
+/// fixture already covers the flag; these tests exercise the richer fixtures.
 #[test]
 fn test_skip_integrity_valid_yaml_standalone() {
     let compiled = compile_fixture_with_flags("complete-agent.md", &["--skip-integrity"]);
     assert_valid_yaml(&compiled, "complete-agent.md (skip-integrity)");
+    assert!(
+        !compiled.contains("Verify pipeline integrity"),
+        "complete-agent.md compiled with --skip-integrity should NOT contain the integrity check step"
+    );
 }
 
 #[test]
 fn test_skip_integrity_valid_yaml_1es() {
     let compiled = compile_fixture_with_flags("1es-test-agent.md", &["--skip-integrity"]);
     assert_valid_yaml(&compiled, "1es-test-agent.md (skip-integrity)");
+    assert!(
+        !compiled.contains("Verify pipeline integrity"),
+        "1es-test-agent.md compiled with --skip-integrity should NOT contain the integrity check step"
+    );
 }
 
 // ─── --debug-pipeline flag tests ─────────────────────────────────────────
@@ -4446,17 +4476,36 @@ fn test_default_excludes_debug_diagnostics() {
     );
 }
 
-/// Test that --debug-pipeline produces valid YAML for both targets
+/// Test that --debug-pipeline produces valid YAML and includes debug diagnostics
+/// for both standalone (complete-agent) and 1ES targets. The minimal-agent
+/// fixture already covers the individual debug signals; these tests exercise
+/// the richer fixtures to confirm no flag interaction regressions.
 #[test]
 fn test_debug_pipeline_valid_yaml_standalone() {
     let compiled = compile_fixture_with_flags("complete-agent.md", &["--debug-pipeline"]);
     assert_valid_yaml(&compiled, "complete-agent.md (debug-pipeline)");
+    assert!(
+        compiled.contains(r#"DEBUG="*""#),
+        "complete-agent.md compiled with --debug-pipeline should contain DEBUG=* env var"
+    );
+    assert!(
+        compiled.contains("Verify MCP backends"),
+        "complete-agent.md compiled with --debug-pipeline should contain probe step"
+    );
 }
 
 #[test]
 fn test_debug_pipeline_valid_yaml_1es() {
     let compiled = compile_fixture_with_flags("1es-test-agent.md", &["--debug-pipeline"]);
     assert_valid_yaml(&compiled, "1es-test-agent.md (debug-pipeline)");
+    assert!(
+        compiled.contains(r#"DEBUG="*""#),
+        "1es-test-agent.md compiled with --debug-pipeline should contain DEBUG=* env var"
+    );
+    assert!(
+        compiled.contains("Verify MCP backends"),
+        "1es-test-agent.md compiled with --debug-pipeline should contain probe step"
+    );
 }
 
 /// Test that both flags can be combined