From 839cbc28b88d9fb04d233f6d4eeb2592207c83f1 Mon Sep 17 00:00:00 2001
From: Bryan Helmkamp <bryan@brynary.com>
Date: Sun, 8 Mar 2026 13:11:39 -0400
Subject: [PATCH] Fix stale test: goal is now optional in run config

The missing_required_fields test was not updated when goal was changed
from String to Option<String> in c143d2c8.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 crates/arc-workflows/src/cli/run_config.rs | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/crates/arc-workflows/src/cli/run_config.rs b/crates/arc-workflows/src/cli/run_config.rs
index e0c3eb9cf..72924c570 100644
--- a/crates/arc-workflows/src/cli/run_config.rs
+++ b/crates/arc-workflows/src/cli/run_config.rs
@@ -589,13 +589,17 @@ graph = "p.dot"
     }
 
     #[test]
-    fn missing_required_fields() {
+    fn goal_is_optional() {
         let no_goal = r#"
 version = 1
 graph = "p.dot"
 "#;
-        assert!(parse_run_config(no_goal).is_err());
+        let config = parse_run_config(no_goal).unwrap();
+        assert!(config.goal.is_none());
+    }
 
+    #[test]
+    fn graph_is_required() {
         let no_graph = r#"
 version = 1
 goal = "x"