Fail on tool not existing

src/tool_registry.cpp

@@ -35,6 +35,8 @@ ToolRegistry ToolRegistry::create_subregistry(std::vector<std::string> const& na
         auto found = this->find(tool_name);
         if (found)
             sub_registry.register_tool(**found);
+        else
+            throw std::runtime_error("Could not find tool:" + tool_name);
     }
 
     return sub_registry;

tests/CMakeLists.txt

@@ -1,6 +1,7 @@
 # Add smoke test targets
 add_test(NAME smoke_basic COMMAND bash ${CMAKE_CURRENT_SOURCE_DIR}/run_test.sh ${CMAKE_CURRENT_SOURCE_DIR}/test_smoke_basic.sh)
 add_test(NAME smoke_tool_call COMMAND bash ${CMAKE_CURRENT_SOURCE_DIR}/run_test.sh ${CMAKE_CURRENT_SOURCE_DIR}/test_smoke_tool_call.sh)
+add_test(NAME invalid_tool_fails COMMAND bash ${CMAKE_CURRENT_SOURCE_DIR}/run_test.sh ${CMAKE_CURRENT_SOURCE_DIR}/invalid_tool_fails.sh)
 
 # Set test properties
 set_tests_properties(smoke_basic PROPERTIES
@@ -11,3 +12,7 @@ set_tests_properties(smoke_tool_call PROPERTIES
     TIMEOUT 30
     WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
 )
+set_tests_properties(invalid_tool_fails PROPERTIES
+    TIMEOUT 30
+    WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
+)

tests/invalid_tool_fails.sh

@@ -0,0 +1,17 @@
+#!/bin/bash
+
+be_done 'Done reading' >> mock
+
+if cat mock | "${DRHARNESS}" -t edit -d -m test -n -F chat.json ; then
+    echo "Tool exists and works: YEA"
+else
+    echo "Fail"
+    exit 1
+fi
+
+if cat mock | "${DRHARNESS}" -t asdfjklasdfj -d -m test -n -F chat.json ; then
+    echo "Didn't fail on tool not existing: yikes"
+    exit 1
+else
+    echo "Pass"
+fi