mirror of
https://github.com/openai/codex.git
synced 2026-05-24 04:54:52 +00:00
Narrow exec server URL accessor
Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
@@ -74,6 +74,7 @@ const SUBMIT_TURN_COMPLETE_TIMEOUT: Duration = Duration::from_secs(30);
|
||||
#[derive(Debug)]
|
||||
pub struct TestEnv {
|
||||
environment: codex_exec_server::Environment,
|
||||
exec_server_url: Option<String>,
|
||||
cwd: AbsolutePathBuf,
|
||||
local_cwd_temp_dir: Option<Arc<TempDir>>,
|
||||
remote_container_name: Option<String>,
|
||||
@@ -87,6 +88,7 @@ impl TestEnv {
|
||||
codex_exec_server::Environment::create_for_tests(/*exec_server_url*/ None)?;
|
||||
Ok(Self {
|
||||
environment,
|
||||
exec_server_url: None,
|
||||
cwd,
|
||||
local_cwd_temp_dir: Some(local_cwd_temp_dir),
|
||||
remote_container_name: None,
|
||||
@@ -101,10 +103,6 @@ impl TestEnv {
|
||||
&self.environment
|
||||
}
|
||||
|
||||
pub fn exec_server_url(&self) -> Option<&str> {
|
||||
self.environment.exec_server_url()
|
||||
}
|
||||
|
||||
fn local_cwd_temp_dir(&self) -> Option<Arc<TempDir>> {
|
||||
self.local_cwd_temp_dir.clone()
|
||||
}
|
||||
@@ -124,7 +122,7 @@ pub async fn test_env() -> Result<TestEnv> {
|
||||
Some(remote_env) => {
|
||||
let websocket_url = remote_exec_server_url()?;
|
||||
let environment =
|
||||
codex_exec_server::Environment::create_for_tests(Some(websocket_url))?;
|
||||
codex_exec_server::Environment::create_for_tests(Some(websocket_url.clone()))?;
|
||||
let cwd = remote_aware_cwd_path();
|
||||
environment
|
||||
.get_filesystem()
|
||||
@@ -136,6 +134,7 @@ pub async fn test_env() -> Result<TestEnv> {
|
||||
.await?;
|
||||
Ok(TestEnv {
|
||||
environment,
|
||||
exec_server_url: Some(websocket_url),
|
||||
cwd,
|
||||
local_cwd_temp_dir: None,
|
||||
remote_container_name: Some(remote_env.container_name),
|
||||
@@ -386,7 +385,7 @@ impl TestCodexBuilder {
|
||||
let exec_server_url = self
|
||||
.exec_server_url
|
||||
.clone()
|
||||
.or_else(|| test_env.exec_server_url().map(str::to_owned));
|
||||
.or_else(|| test_env.exec_server_url.clone());
|
||||
let local_runtime_paths = codex_exec_server::ExecServerRuntimePaths::new(
|
||||
std::env::current_exe()?,
|
||||
/*codex_linux_sandbox_exe*/ None,
|
||||
|
||||
@@ -344,6 +344,7 @@ mod tests {
|
||||
);
|
||||
assert_eq!(provider.environments["ssh-dev"], ssh_transport);
|
||||
assert!(environments["ssh-dev"].is_remote());
|
||||
assert_eq!(environments["ssh-dev"].exec_server_url(), None);
|
||||
assert_eq!(
|
||||
provider.default_environment_id(),
|
||||
Some("ssh-dev".to_string())
|
||||
|
||||
Reference in New Issue
Block a user