scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-08-28 05:27:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Fix asset collection tests to provide required asset globs

Browse source 
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 <noreply@anthropic.com>
This commit is contained in:
Bryan Helmkamp 2026-03-08 23:54:53 -04:00
parent 3100fd3a62
commit 7b6a2a93b7
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crates/arc-workflows/tests/integration.rs
View file

@ -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