diff --git a/codex-rs/core/src/approval_handler.rs b/codex-rs/core/src/approval_handler.rs
index ca98da1f28..778350ee14 100644
--- a/codex-rs/core/src/approval_handler.rs
+++ b/codex-rs/core/src/approval_handler.rs
@@ -346,7 +346,7 @@ mod tests {
                 decision,
             } => {
                 assert_eq!(id, event.effective_approval_id());
-                assert_eq!(turn_id, Some(event.turn_id.clone()));
+                assert_eq!(turn_id.as_deref(), Some(event.turn_id.as_str()));
                 decision
             }
             _ => unreachable!(),
diff --git a/codex-rs/core/tests/suite/approvals.rs b/codex-rs/core/tests/suite/approvals.rs
index d19863dae8..7896455123 100644
--- a/codex-rs/core/tests/suite/approvals.rs
+++ b/codex-rs/core/tests/suite/approvals.rs
@@ -1992,7 +1992,7 @@ async fn external_exec_approval_handler_approves_without_emitting_prompt() -> Re
 
     let mut builder = test_codex().with_home(Arc::clone(&home)).with_config({
         move |config| {
-            config.features.enable(Feature::UnifiedExec);
+            let _ = config.features.enable(Feature::UnifiedExec);
             config.permissions.approval_policy = Constrained::allow_any(approval_policy);
             config.approval_handler = Some(ApprovalHandlerConfig {
                 command: vec![