Skip to content

Commit 18c5f77

Browse files
test: remove duplicate tests in safeoutputs add_pr_comment and comment_on_work_item (#833)
1 parent 991e31a commit 18c5f77

2 files changed

Lines changed: 0 additions & 28 deletions

File tree

src/safeoutputs/add_pr_comment.rs

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -630,26 +630,6 @@ allowed-statuses:
630630
);
631631
}
632632

633-
#[test]
634-
fn test_allowed_statuses_case_insensitive_reverse() {
635-
// Config has "active" but agent sends "Active" — should be allowed
636-
let config = AddPrCommentConfig {
637-
comment_prefix: None,
638-
allowed_repositories: Vec::new(),
639-
allowed_statuses: vec!["active".to_string()],
640-
include_stats: true,
641-
};
642-
let status = "Active";
643-
let matched = config
644-
.allowed_statuses
645-
.iter()
646-
.any(|s| s.eq_ignore_ascii_case(status));
647-
assert!(
648-
matched,
649-
"uppercase 'Active' should match config value 'active'"
650-
);
651-
}
652-
653633
#[test]
654634
fn test_sanitize_content_neutralizes_repository_pipeline_command() {
655635
let params = AddPrCommentParams {

src/safeoutputs/comment_on_work_item.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -490,13 +490,5 @@ max: 3
490490
assert!(config.target.is_none());
491491
}
492492

493-
#[test]
494-
fn test_config_partial_deserialize_uses_defaults() {
495-
let yaml = r#"
496-
target: "*"
497-
"#;
498-
let config: CommentOnWorkItemConfig = serde_yaml::from_str(yaml).unwrap();
499-
assert!(config.target.is_some());
500-
}
501493
}
502494

0 commit comments

Comments
 (0)