test: update tests to reflect new error callback behavior

Adjusts test assertions after commit b4778f9 which changed the tool
system to only pass 'true' to error callbacks for actual exceptions,
not validation/processing errors.

Changed 3 assertions from expecting (:error? result) to be true to
expecting false, since validation errors now appear in results without
triggering MCP-level error callbacks.

Author: Bruce Hauman

test/clojure_mcp/other_tools/namespace/tool_test.clj

@@ -197,7 +197,7 @@
 
   (testing "list-vars-in-namespace tool execution with invalid namespace"
     (let [result (test-tool-execution :list-vars-in-namespace {"namespace" "nonexistent.ns"})]
-      (is (true? (:error? result)))
+      (is (false? (:error? result)))
       (is (string? (first (:result result))))
       (is (str/includes? (first (:result result)) "not found")))))
 

test/clojure_mcp/tools/eval/tool_test.clj

@@ -105,17 +105,17 @@
 
   (testing "Evaluation with error"
     (let [result (test-tool-execution "(throw (Exception. \"test error\"))")]
-      (is (true? (:error? result)))
+      (is (false? (:error? result)))
       (is (= 1 (count (:result result))))
       (is (str/includes? (first (:result result)) "test error"))
       (is (str/includes? (first (:result result)) "Execution error"))))
 
   (testing "Multiple expressions"
     (let [result (test-tool-execution "(println \"first\") (+ 10 20)")]
       (is (false? (:error? result)))
-       (is (= 3 (count (:result result))))
-       (is (= ["first\n=> nil" "*===============================================*" "=> 30"]
-              (:result result)))))
+      (is (= 3 (count (:result result))))
+      (is (= ["first\n=> nil" "*===============================================*" "=> 30"]
+             (:result result)))))
 
   (testing "Evaluation with linter warning"
     (let [result (test-tool-execution "(let [unused 1] (+ 2 3))")]
@@ -126,7 +126,7 @@
 
   (testing "Evaluation with linter error"
     (let [result (test-tool-execution "(def ^:dynamic 1)")]
-      (is (true? (:error? result)))
+      (is (false? (:error? result)))
       (is (= 1 (count (:result result))))
       (is (str/includes? (first (:result result)) "Can't parse")))))