test: remove duplicate PR trigger test in pr_filters (#804)

test_generate_pr_trigger_with_pr_trigger_and_pipeline_trigger was a duplicate
of test_generate_pr_trigger_with_explicit_pr_trigger_overrides_schedule.

Both tests verify the same behavior: when triggers.pr is explicitly set,
generate_pr_trigger returns early at line 499 of src/compile/common.rs,
bypassing the schedule/pipeline suppression logic. The presence of
pipeline: Some(...) vs pipeline: None in the OnConfig setup does not
affect the code path taken, since the function checks has_pipeline_trigger
only after the explicit pr: early return.

The remaining test has been renamed to _overrides_suppression (instead of
_overrides_schedule) and its comments now document that it covers both
the schedule and pipeline suppression cases.

Test count: 32 → 31

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]

src/compile/pr_filters.rs

@@ -114,37 +114,19 @@ mod tests {
     }
 
     #[test]
-    fn test_generate_pr_trigger_with_explicit_pr_trigger_overrides_schedule() {
+    fn test_generate_pr_trigger_with_explicit_pr_trigger_overrides_suppression() {
         let triggers = Some(OnConfig {
             pipeline: None,
             pr: Some(PrTriggerConfig::default()),
         schedule: None,
         });
         let result = generate_pr_trigger(&triggers, true);
         // PrTriggerConfig::default() has no branches/paths, so the native block is empty
-        // (meaning "trigger on all PRs" in ADO). The schedule suppression ("pr: none") must
-        // NOT be emitted because the explicit pr: key overrides it.
+        // (meaning "trigger on all PRs" in ADO). The schedule/pipeline suppression ("pr: none")
+        // must NOT be emitted because the explicit pr: key overrides it — regardless of whether
+        // has_schedule or has_pipeline_trigger is set.
         assert!(result.is_empty(), "default PrTriggerConfig should produce empty string (trigger on all PRs)");
-        assert!(!result.contains("pr: none"), "triggers.pr should override schedule suppression");
-    }
-
-    #[test]
-    fn test_generate_pr_trigger_with_pr_trigger_and_pipeline_trigger() {
-        let triggers = Some(OnConfig {
-            pipeline: Some(PipelineTrigger {
-                name: "Build".into(),
-                project: None,
-                branches: vec![],
-            filters: None,
-            }),
-            pr: Some(PrTriggerConfig::default()),
-        schedule: None,
-        });
-        let result = generate_pr_trigger(&triggers, false);
-        // Same as above: default PrTriggerConfig → empty (trigger on all PRs).
-        // Pipeline-only suppression must NOT be emitted because pr: is explicit.
-        assert!(result.is_empty(), "default PrTriggerConfig should produce empty string (trigger on all PRs)");
-        assert!(!result.contains("pr: none"), "triggers.pr should override pipeline trigger suppression");
+        assert!(!result.contains("pr: none"), "triggers.pr should override schedule/pipeline suppression");
     }
 
     #[test]