openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-09-07 15:40:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'etraut/tui-cleanup-4' into etraut/tui-cleanup-5

Browse Source
This commit is contained in:
Eric Traut
2026-04-29 21:10:13 -07:00
parent 69a35a62e6 3cc2143c27
commit 21cbc3d13c
2 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
codex-rs/tui/src/app_server_session.rs
View File

@@ -1874,8 +1874,8 @@ mod tests {
.to_legacy_sandbox_policy(cwd.as_path())
.expect("read-only profile must be legacy-compatible")
.into();
let split_profile = PermissionProfile::Managed {
file_system: ManagedFileSystemPermissions::Restricted {
let response_profile = AppServerPermissionProfile::Managed {
file_system: PermissionProfileFileSystemPermissions::Restricted {
entries: vec![
FileSystemSandboxEntry {
path: FileSystemPath::Special {
@@ -1894,9 +1894,9 @@ mod tests {
],
glob_scan_max_depth: None,
},
network: NetworkSandboxPolicy::Restricted,
network: PermissionProfileNetworkPermissions { enabled: false },
};
let response_profile = split_profile.clone().into();
let split_profile: PermissionProfile = response_profile.clone().into();
assert_eq!(
permission_profile_from_thread_response(

14
codex-rs/tui/src/status/tests.rs
View File

@@ -29,9 +29,9 @@ use codex_protocol::config_types::ApprovalsReviewer;
use codex_protocol::config_types::ReasoningSummary;
use codex_protocol::models::ActivePermissionProfile;
use codex_protocol::models::ActivePermissionProfileModification;
use codex_protocol::models::ManagedFileSystemPermissions;
use codex_protocol::models::PermissionProfile;
use codex_protocol::openai_models::ReasoningEffort;
use codex_protocol::permissions::NetworkSandboxPolicy;
use insta::assert_snapshot;
use pretty_assertions::assert_eq;
use ratatui::prelude::*;
@@ -287,7 +287,7 @@ async fn status_permissions_named_read_only_profile_shows_builtin_label() {
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
config
.permissions
@@ -310,7 +310,7 @@ async fn status_permissions_read_only_profile_shows_additional_writable_roots()
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
let extra_root = test_path_buf("/workspace/extra").abs();
let file_system_policy = PermissionProfile::read_only()
@@ -346,7 +346,7 @@ async fn status_permissions_named_workspace_profile_shows_builtin_label() {
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
config
.permissions
@@ -370,7 +370,7 @@ async fn status_permissions_workspace_auto_review_shows_reviewer_label() {
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
config
.permissions
@@ -393,7 +393,7 @@ async fn status_permissions_named_profile_shows_additional_writable_roots() {
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
let extra_root = test_path_buf("/workspace/extra").abs();
config
@@ -428,7 +428,7 @@ async fn status_permissions_broadened_workspace_profile_shows_builtin_label() {
config
.permissions
.approval_policy
.set(AskForApproval::OnRequest)
.set(AskForApproval::OnRequest.to_core())
.expect("set approval policy");
config
.permissions