Skip to content

Commit 1f33a57

Browse files
committed
fix(ci): skip agent-pack real pack roundtrip without fixture
CI runners lack /tmp deepagent pack trees; panicking failed the workspace test gate. Skip with a clear hint unless GBASE_REAL_PACK_DIR or a known local candidate is present.
1 parent 08b7518 commit 1f33a57

1 file changed

Lines changed: 35 additions & 20 deletions

File tree

crates/agent-pack/tests/real_pack_roundtrip.rs

Lines changed: 35 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,35 +1,44 @@
11
//! Integration: real Harbor pack `realpr-click-3442` round-trip + strip.
2+
//!
3+
//! Skips when the fixture pack is not present locally (CI default).
4+
//! Set `GBASE_REAL_PACK_DIR` or place the pack on one of the candidate paths.
25
36
use std::path::{Path, PathBuf};
47

58
use agent_pack::{load_pack, PackError, STRIPPED_FIELD_NAMES};
69

710
fn real_pack_candidates() -> Vec<PathBuf> {
8-
[
9-
"/tmp/deepagent/datasets/prod_hard_deepswe_med/tasks/realpr-click-3442",
10-
"/tmp/baseintel/deepagent/datasets/prod_hard_deepswe_med/tasks/realpr-click-3442",
11-
"/tmp/da_m29d_hf_pull_verify/tasks/realpr-click-3442",
12-
]
13-
.into_iter()
14-
.map(PathBuf::from)
15-
.collect()
16-
}
17-
18-
fn resolve_real_pack() -> PathBuf {
19-
for p in real_pack_candidates() {
20-
if p.join("task.toml").is_file() {
21-
return p;
22-
}
11+
let mut out = Vec::new();
12+
if let Ok(p) = std::env::var("GBASE_REAL_PACK_DIR") {
13+
out.push(PathBuf::from(p));
2314
}
24-
panic!(
25-
"realpr-click-3442 not found; tried {:?}",
26-
real_pack_candidates()
15+
out.extend(
16+
[
17+
"/tmp/deepagent/datasets/prod_hard_deepswe_med/tasks/realpr-click-3442",
18+
"/tmp/baseintel/deepagent/datasets/prod_hard_deepswe_med/tasks/realpr-click-3442",
19+
"/tmp/da_m29d_hf_pull_verify/tasks/realpr-click-3442",
20+
]
21+
.into_iter()
22+
.map(PathBuf::from),
2723
);
24+
out
25+
}
26+
27+
fn resolve_real_pack() -> Option<PathBuf> {
28+
real_pack_candidates()
29+
.into_iter()
30+
.find(|p| p.join("task.toml").is_file())
2831
}
2932

3033
#[test]
3134
fn realpr_click_3442_round_trip_strip_and_stable_digest() {
32-
let root = resolve_real_pack();
35+
let Some(root) = resolve_real_pack() else {
36+
eprintln!(
37+
"skip: realpr-click-3442 not found; tried {:?}; set GBASE_REAL_PACK_DIR to run",
38+
real_pack_candidates()
39+
);
40+
return;
41+
};
3342
let pack = load_pack(&root).unwrap_or_else(|e| panic!("load {}: {e}", root.display()));
3443

3544
assert_eq!(pack.task_id, "realpr-click-3442");
@@ -98,7 +107,13 @@ fn realpr_click_3442_round_trip_strip_and_stable_digest() {
98107

99108
#[test]
100109
fn real_pack_path_is_directory() {
101-
let root = resolve_real_pack();
110+
let Some(root) = resolve_real_pack() else {
111+
eprintln!(
112+
"skip: realpr-click-3442 not found; tried {:?}; set GBASE_REAL_PACK_DIR to run",
113+
real_pack_candidates()
114+
);
115+
return;
116+
};
102117
assert!(Path::new(&root).is_dir());
103118
}
104119

0 commit comments

Comments
 (0)