From 82f10fa1cb8774c13148eccfa03072c54751068a Mon Sep 17 00:00:00 2001 From: Christian Millar Date: Tue, 6 Jan 2026 19:22:01 +0000 Subject: [PATCH] [TRUNK-17146] Test to confirm bazel bep codeowners We had changes that extracted codeowners from bazel bep and into codeowners, but tests didn't confirm it. Adding them so that we can depend on them. --- cli/tests/upload.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cli/tests/upload.rs b/cli/tests/upload.rs index a6b4091a..602a4dc8 100644 --- a/cli/tests/upload.rs +++ b/cli/tests/upload.rs @@ -240,6 +240,7 @@ async fn upload_bundle() { async fn upload_bundle_using_bep() { let temp_dir = tempdir().unwrap(); generate_mock_git_repo(&temp_dir); + generate_mock_codeowners(&temp_dir); let bep_path = generate_mock_bazel_bep(&temp_dir); let state = MockServerBuilder::new().spawn_mock_server().await; @@ -335,7 +336,9 @@ async fn upload_bundle_using_bep() { assert!(test_case_run.started_at.is_some()); assert!(test_case_run.finished_at.is_some()); assert!(!test_case_run.is_quarantined); - assert_eq!(test_case_run.codeowners.len(), 0); + assert_eq!(test_case_run.codeowners.len(), 2); + assert_eq!(test_case_run.codeowners[0].name, "@user"); + assert_eq!(test_case_run.codeowners[1].name, "@user2"); // HINT: View CLI output with `cargo test -- --nocapture` println!("{assert}");