Remove stale zero-length CLI tests after validator relaxation (#472)

PR #472 (commit f13bbb3) relaxed
SequencingToMinimizeWeightedCompletionTime's validator to accept
zero-length tasks for the Lawler OLA reduction. Two CLI tests still
asserted the removed "task lengths must be positive" rejection, so CI
broke on the workspace test job. The new accept-zero-length semantics
are positively covered by unit tests in
src/unit_tests/models/misc/sequencing_to_minimize_weighted_completion_time.rs.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: isPANN

problemreductions-cli/tests/cli_tests.rs

@@ -3735,24 +3735,6 @@ fn test_create_help_describes_precedence_pairs_generically() {
     assert!(stdout.contains("Precedence pairs for MinimumTardinessSequencing, SchedulingWithIndividualDeadlines, or SequencingToMinimizeWeightedCompletionTime"));
 }
 
-#[test]
-fn test_create_sequencing_to_minimize_weighted_completion_time_rejects_zero_length() {
-    let output = pred()
-        .args([
-            "create",
-            "SequencingToMinimizeWeightedCompletionTime",
-            "--lengths",
-            "0,1,3",
-            "--weights",
-            "3,5,1",
-        ])
-        .output()
-        .unwrap();
-    assert!(!output.status.success());
-    let stderr = String::from_utf8_lossy(&output.stderr);
-    assert!(stderr.contains("task lengths must be positive"), "{stderr}");
-}
-
 #[test]
 fn test_create_with_edge_weights() {
     let output_file = std::env::temp_dir().join("pred_test_create_ew.json");
@@ -5889,47 +5871,6 @@ fn test_inspect_stdin() {
     );
 }
 
-#[test]
-fn test_inspect_rejects_zero_length_sequencing_problem_from_stdin() {
-    let create_out = pred()
-        .args([
-            "create",
-            "--example",
-            "SequencingToMinimizeWeightedCompletionTime",
-        ])
-        .output()
-        .unwrap();
-    assert!(
-        create_out.status.success(),
-        "stderr: {}",
-        String::from_utf8_lossy(&create_out.stderr)
-    );
-
-    let mut json: serde_json::Value = serde_json::from_slice(&create_out.stdout).unwrap();
-    json["data"]["lengths"][0] = serde_json::json!(0);
-    let invalid_json = serde_json::to_vec(&json).unwrap();
-
-    use std::io::Write;
-    let mut child = pred()
-        .args(["inspect", "-"])
-        .stdin(std::process::Stdio::piped())
-        .stdout(std::process::Stdio::piped())
-        .stderr(std::process::Stdio::piped())
-        .spawn()
-        .unwrap();
-    child
-        .stdin
-        .take()
-        .unwrap()
-        .write_all(&invalid_json)
-        .unwrap();
-    let result = child.wait_with_output().unwrap();
-
-    assert!(!result.status.success());
-    let stderr = String::from_utf8(result.stderr).unwrap();
-    assert!(stderr.contains("task lengths must be positive"), "{stderr}");
-}
-
 #[test]
 fn test_inspect_json_output() {
     let problem_file = std::env::temp_dir().join("pred_test_inspect_json_in.json");