208 lines
6.8 KiB
Rust
208 lines
6.8 KiB
Rust
//! Integration tests for `Worker::restore_from_manifest`'s pre-build
|
|
//! validation paths.
|
|
//!
|
|
//! These cases all return before `prepare_worker_common` runs, so they
|
|
//! do not need a real LLM client or pod-registry environment — only the
|
|
//! session store needs to be present.
|
|
|
|
use std::sync::{LazyLock, Mutex};
|
|
|
|
use pod_store::{
|
|
CombinedStore, FsWorkerStore, WorkerActiveSegmentRef, WorkerMetadata, WorkerMetadataStore,
|
|
};
|
|
use session_store::{FsStore, StoreError};
|
|
use worker::{Worker, WorkerError};
|
|
|
|
const MINIMAL_MANIFEST_TOML: &str = r#"
|
|
[worker]
|
|
name = "restore-test"
|
|
pwd = "./"
|
|
|
|
[model]
|
|
scheme = "anthropic"
|
|
model_id = "test-model"
|
|
|
|
[engine]
|
|
max_tokens = 100
|
|
|
|
[[scope.allow]]
|
|
target = "./"
|
|
permission = "write"
|
|
"#;
|
|
|
|
/// Serialises tests that mutate runtime-dir env vars, mirroring the
|
|
/// pattern used by other integration tests in this crate.
|
|
static ENV_LOCK: LazyLock<Mutex<()>> = LazyLock::new(|| Mutex::new(()));
|
|
|
|
#[tokio::test]
|
|
async fn restore_from_worker_metadata_rejects_missing_metadata() {
|
|
let _lock = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
|
|
|
|
let store_tmp = tempfile::tempdir().unwrap();
|
|
let store = CombinedStore::new(
|
|
FsStore::new(store_tmp.path()).unwrap(),
|
|
FsWorkerStore::new(store_tmp.path().join("pods")).unwrap(),
|
|
);
|
|
let manifest = worker::WorkerManifest::from_toml(MINIMAL_MANIFEST_TOML).unwrap();
|
|
|
|
let result = Worker::restore_from_worker_metadata(
|
|
"restore-test",
|
|
manifest,
|
|
store,
|
|
worker::PromptLoader::builtins_only(),
|
|
)
|
|
.await;
|
|
|
|
match result {
|
|
Err(WorkerError::WorkerMetadataMissing { worker_name }) => {
|
|
assert_eq!(worker_name, "restore-test")
|
|
}
|
|
Err(other) => panic!("expected WorkerMetadataMissing, got {other:?}"),
|
|
Ok(_) => panic!("expected missing worker metadata to fail"),
|
|
}
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn restore_from_worker_metadata_rejects_pending_segment() {
|
|
let _lock = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
|
|
|
|
let store_tmp = tempfile::tempdir().unwrap();
|
|
let store = CombinedStore::new(
|
|
FsStore::new(store_tmp.path()).unwrap(),
|
|
FsWorkerStore::new(store_tmp.path().join("pods")).unwrap(),
|
|
);
|
|
let manifest = worker::WorkerManifest::from_toml(MINIMAL_MANIFEST_TOML).unwrap();
|
|
let session_id = session_store::new_session_id();
|
|
store
|
|
.write(&WorkerMetadata::new(
|
|
"restore-test",
|
|
Some(WorkerActiveSegmentRef::pending_segment(session_id)),
|
|
))
|
|
.unwrap();
|
|
|
|
let result = Worker::restore_from_worker_metadata(
|
|
"restore-test",
|
|
manifest,
|
|
store,
|
|
worker::PromptLoader::builtins_only(),
|
|
)
|
|
.await;
|
|
|
|
match result {
|
|
Err(WorkerError::WorkerMetadataPending {
|
|
worker_name,
|
|
session_id: actual,
|
|
}) => {
|
|
assert_eq!(worker_name, "restore-test");
|
|
assert_eq!(actual, session_id);
|
|
}
|
|
Err(other) => panic!("expected WorkerMetadataPending, got {other:?}"),
|
|
Ok(_) => panic!("expected pending worker metadata to fail"),
|
|
}
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn restore_from_worker_metadata_resolves_active_pointer_through_session_log() {
|
|
let _lock = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
|
|
|
|
let store_tmp = tempfile::tempdir().unwrap();
|
|
let store = CombinedStore::new(
|
|
FsStore::new(store_tmp.path()).unwrap(),
|
|
FsWorkerStore::new(store_tmp.path().join("pods")).unwrap(),
|
|
);
|
|
let manifest = worker::WorkerManifest::from_toml(MINIMAL_MANIFEST_TOML).unwrap();
|
|
let session_id = session_store::new_session_id();
|
|
let segment_id = session_store::new_segment_id();
|
|
store
|
|
.write(&WorkerMetadata::new(
|
|
"restore-test",
|
|
Some(WorkerActiveSegmentRef::active_segment(
|
|
session_id, segment_id,
|
|
)),
|
|
))
|
|
.unwrap();
|
|
|
|
let result = Worker::restore_from_worker_metadata(
|
|
"restore-test",
|
|
manifest,
|
|
store,
|
|
worker::PromptLoader::builtins_only(),
|
|
)
|
|
.await;
|
|
|
|
match result {
|
|
Err(WorkerError::Store(StoreError::NotFound(id))) => assert_eq!(id, segment_id),
|
|
Err(other) => panic!("expected Store(NotFound) from resolved segment, got {other:?}"),
|
|
Ok(_) => panic!("expected unknown resolved segment to fail"),
|
|
}
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn restore_from_manifest_rejects_unknown_segment() {
|
|
let _lock = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
|
|
|
|
let store_tmp = tempfile::tempdir().unwrap();
|
|
let store = CombinedStore::new(
|
|
FsStore::new(store_tmp.path()).unwrap(),
|
|
FsWorkerStore::new(store_tmp.path().join("pods")).unwrap(),
|
|
);
|
|
let manifest = worker::WorkerManifest::from_toml(MINIMAL_MANIFEST_TOML).unwrap();
|
|
|
|
// A freshly-minted id with no jsonl file at all → store returns
|
|
// NotFound, which `Worker::restore_from_manifest` surfaces verbatim
|
|
// as `WorkerError::Store`.
|
|
let unknown_sid = session_store::new_session_id();
|
|
let unknown_seg = session_store::new_segment_id();
|
|
let result = Worker::restore_from_manifest(
|
|
unknown_sid,
|
|
unknown_seg,
|
|
manifest,
|
|
store,
|
|
worker::PromptLoader::builtins_only(),
|
|
)
|
|
.await;
|
|
|
|
match result {
|
|
Err(WorkerError::Store(StoreError::NotFound(id))) => assert_eq!(id, unknown_seg),
|
|
Err(other) => panic!("expected Store(NotFound), got {other:?}"),
|
|
Ok(_) => panic!("expected unknown segment to fail"),
|
|
}
|
|
}
|
|
|
|
#[tokio::test]
|
|
async fn restore_from_manifest_rejects_empty_segment_log() {
|
|
let _lock = ENV_LOCK.lock().unwrap_or_else(|e| e.into_inner());
|
|
|
|
let store_tmp = tempfile::tempdir().unwrap();
|
|
let store = CombinedStore::new(
|
|
FsStore::new(store_tmp.path()).unwrap(),
|
|
FsWorkerStore::new(store_tmp.path().join("pods")).unwrap(),
|
|
);
|
|
let manifest = worker::WorkerManifest::from_toml(MINIMAL_MANIFEST_TOML).unwrap();
|
|
|
|
// Pre-create an empty `<sid>/<segid>.jsonl` so `read_all` succeeds
|
|
// with no entries. `collect_state` returns `entries_count = 0`,
|
|
// which `restore_from_manifest` rejects with `SegmentEmpty` *before*
|
|
// it gets as far as building the LLM client.
|
|
let sid = session_store::new_session_id();
|
|
let segid = session_store::new_segment_id();
|
|
let dir = store_tmp.path().join(sid.to_string());
|
|
std::fs::create_dir_all(&dir).unwrap();
|
|
std::fs::write(dir.join(format!("{segid}.jsonl")), b"").unwrap();
|
|
|
|
let result = Worker::restore_from_manifest(
|
|
sid,
|
|
segid,
|
|
manifest,
|
|
store,
|
|
worker::PromptLoader::builtins_only(),
|
|
)
|
|
.await;
|
|
|
|
match result {
|
|
Err(WorkerError::SegmentEmpty { segment_id }) => assert_eq!(segment_id, segid),
|
|
Err(other) => panic!("expected SegmentEmpty, got {other:?}"),
|
|
Ok(_) => panic!("expected empty segment log to fail"),
|
|
}
|
|
}
|