Use named type imports in compatibility tests

This commit is contained in:
Scott Werner 2026-10-23 15:05:51 -04:00
parent 7494cea227
commit 88b94737f4
2 changed files with 5 additions and 11 deletions

View file

@ -2158,7 +2158,7 @@ fn slack_service_ignores_server_env_tokens() {
#[cfg(unix)]
#[test]
fn worker_command_uses_captured_server_connection_configuration() {
use std::sync::atomic::{AtomicBool, Ordering};
use std::ffi::OsStr;
let storage_dir = tempfile::tempdir().unwrap();
let changed = Arc::new(AtomicBool::new(false));
@ -2203,12 +2203,9 @@ fn worker_command_uses_captured_server_connection_configuration() {
"server.internal",
),
] {
assert_eq!(
env.get(std::ffi::OsStr::new(name)),
Some(&Some(std::ffi::OsStr::new(value)))
);
assert_eq!(env.get(OsStr::new(name)), Some(&Some(OsStr::new(value))));
}
assert!(!env.contains_key(std::ffi::OsStr::new(EnvVars::DAYTONA_API_KEY)));
assert!(!env.contains_key(OsStr::new(EnvVars::DAYTONA_API_KEY)));
}
#[cfg(unix)]

View file

@ -1,6 +1,6 @@
use fabro_types::SandboxProviderKind;
use fabro_types::settings::InterpString;
use fabro_types::settings::run::{ApprovalMode, EnvironmentNetworkMode, RunGoal, RunMode};
use fabro_types::settings::{Duration, InterpString};
use crate::{MergeMap, SettingsLayer};
@ -554,10 +554,7 @@ fn lifecycle_defaults_and_explicit_false_remain_compatible_across_providers() {
!lifecycle.run_policy().stop_on_terminal,
"omission inherits independently"
);
assert_eq!(
lifecycle.auto_stop,
Some(fabro_types::settings::Duration::from_secs(1800))
);
assert_eq!(lifecycle.auto_stop, Some(Duration::from_secs(1800)));
}
}