test: correct mislabelled vacuous test in src/compile/types.rs

[github-actions]

Test Suite Reduction: src/compile/types.rs

What was wrong

• test_front_matter_safe_outputs_triggers_conclusion_job: the test name claimed to verify that a non-empty safe-outputs block triggers the conclusion job, but the body only parsed front matter and asserted !fm.safe_outputs.is_empty(). Two problems:
  1. Mislabelled — conclusion-job generation is compiler logic, not front-matter parser logic; no conclusion job is ever triggered in this test.
  2. Near-vacuous — !fm.safe_outputs.is_empty() trivially passes for any non-empty safe-outputs: block, regardless of correctness.

Changes

Test	Action	Reason
test_front_matter_safe_outputs_triggers_conclusion_job	Renamed + rewritten	Mislabelled name; near-vacuous assertion

New test: test_front_matter_safe_outputs_noop_object_form — verifies that noop: {} parses as a JSON object value (is_object() == true), distinguishing it from noop: false (boolean disable, tested in test_front_matter_safe_outputs_noop_disabled) and bare noop: (null/default). This is the actual observable contract of the front-matter parser for the object-form noop idiom.

Verification

• cargo test: all 2570 tests pass ✅
• cargo clippy --all-targets --all-features: no errors or unused-import warnings ✅

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• spsprodeus21.vssps.visualstudio.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "spsprodeus21.vssps.visualstudio.com"

See Network Configuration for more information.

Generated by Test Reducer · 120.2 AIC · ⌖ 11.8 AIC · ⊞ 8.7K · ◷