From 7b6a2a93b787dc29b5cffbb4252ccd00ffc82b6b Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Sun, 8 Mar 2026 23:54:53 -0400 Subject: [PATCH] Fix asset collection tests to provide required asset globs The opt-in change (7400a9d5) left these two tests with empty asset_globs, which now means "no asset collection". Add the matching glob so the tests actually exercise asset collection. Co-Authored-By: Claude Opus 4.6 --- crates/arc-workflows/tests/integration.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/arc-workflows/tests/integration.rs b/crates/arc-workflows/tests/integration.rs index 76aab0bd8..a48e66d49 100644 --- a/crates/arc-workflows/tests/integration.rs +++ b/crates/arc-workflows/tests/integration.rs @@ -12961,7 +12961,7 @@ async fn asset_collection_local_sandbox_success() { git_author: arc_workflows::git::GitAuthor::default(), base_branch: None, pull_request_enabled: false, - asset_globs: Vec::new(), + asset_globs: vec!["test-results/**".to_string()], }; let outcome = engine @@ -13073,7 +13073,7 @@ async fn asset_collection_local_sandbox_on_failure() { git_author: arc_workflows::git::GitAuthor::default(), base_branch: None, pull_request_enabled: false, - asset_globs: Vec::new(), + asset_globs: vec!["test-results/**".to_string()], }; let outcome = engine