- Amended robot test.

Author: general-kroll-4-life

.github/workflows/build.yml

@@ -260,7 +260,7 @@ jobs:
     - name: Set up Go 1.x
       uses: actions/[email protected]
       with:
-        go-version: '~1.23'
+        go-version: '~1.24'
         check-latest: true
         cache: true
       id: go
@@ -480,7 +480,7 @@ jobs:
     - name: Set up Go 1.x
       uses: actions/[email protected]
       with:
-        go-version: '~1.23'
+        go-version: '~1.24'
         check-latest: true
         cache: true
       id: go

Dockerfile

@@ -135,6 +135,8 @@ COPY --from=registrymock /opt/test/stackql ${TEST_ROOT_DIR}/
 
 COPY --from=builder /work/stackql/build/stackql ${TEST_ROOT_DIR}/build/
 
+COPY --from=builder /work/stackql/build/stackql_mcp_client ${TEST_ROOT_DIR}/build/
+
 RUN  if [ "${RUN_INTEGRATION_TESTS}" = "1" ]; then env PYTHONPATH="$PYTHONPATH:${TEST_ROOT_DIR}/test/python" robot ${TEST_ROOT_DIR}/test/robot/functional; fi
 
 FROM ubuntu:22.04 AS app
@@ -161,6 +163,8 @@ ENV PATH="${APP_DIR}:${PATH}"
 
 COPY --from=integration ${TEST_ROOT_DIR}/build/stackql ${APP_DIR}/
 
+COPY --from=integration ${TEST_ROOT_DIR}/build/stackql_mcp_client ${APP_DIR}/
+
 RUN apt-get update \
     && apt-get install -y ca-certificates openssl netcat-traditional jq dnsutils \
     && update-ca-certificates

test/python/stackql_test_tooling/stackql_context.py

@@ -12,6 +12,7 @@
 from registry_cfg import RegistryCfg
 
 _exe_name = 'stackql'
+_mcp_client_exe_name = 'stackql_mcp_client'
 
 IS_WINDOWS = '0'
 if os.name == 'nt':
@@ -143,10 +144,16 @@ def get_json_from_local_file(fp :str) -> typing.Any:
   REPOSITORY_ROOT_UNIX = get_unix_path(repository_root)
 
   def get_stackql_exe(execution_env :str, is_preinstalled :bool):
-    _default_stackqk_exe = ' '.join(get_unix_path(os.path.join(repository_root, 'build', _exe_name)).splitlines())
+    _default_stackql_exe = ' '.join(get_unix_path(os.path.join(repository_root, 'build', _exe_name)).splitlines())
     if is_preinstalled:
       return 'stackql'
-    return _default_stackqk_exe
+    return _default_stackql_exe
+
+  def get_stackql_mcp_client_exe(execution_env :str, is_preinstalled :bool):
+    _default_stackql_mcp_client_exe = ' '.join(get_unix_path(os.path.join(repository_root, 'build', _mcp_client_exe_name)).splitlines())
+    if is_preinstalled:
+      return _mcp_client_exe_name
+    return _default_stackql_mcp_client_exe
 
   def get_registry_mocked(execution_env :str) -> RegistryCfg:
     return RegistryCfg(
@@ -941,6 +948,7 @@ def get_registry_mock_url(execution_env :str) -> str:
     'SQL_CLIENT_EXPORT_BACKEND':                      get_export_sql_backend(execution_env, sql_backend_str),
     'SQL_CLIENT_EXPORT_CONNECTION_ARG':               get_export_sql_connection_arg(execution_env, sql_backend_str),
     'STACKQL_EXE':                                    get_stackql_exe(execution_env, must_use_stackql_preinstalled),
+    'STACKQL_MCP_CLIENT_EXE':                         get_stackql_mcp_client_exe(execution_env, must_use_stackql_preinstalled),
     'SUMOLOGIC_SECRET_STR':                           SUMOLOGIC_SECRET_STR,
     ## queries and expectations
     'AWS_CC_VIEW_SELECT_PROJECTION_BUCKET_COMPLEX_EXPECTED':                  AWS_CC_VIEW_SELECT_PROJECTION_BUCKET_COMPLEX_EXPECTED,

test/robot/functional/mcp.robot

@@ -2,15 +2,14 @@
 Resource          ${CURDIR}/stackql.resource
 
 *** Test Cases *** 
-MCP HTTP Server List Tools 
-    Pass Execution If    "${EXECUTION_PLATFORM}" == "docker"    Skipping MCP test in docker
-    ${serverProcess}=    Start Process    ${REPOSITORY_ROOT}${/}build${/}stackql
+MCP HTTP Server List Tools
+    ${serverProcess}=    Start Process    ${STACKQL_EXE}
     ...                                   mcp
     ...                                   \-\-mcp.server.type\=http 
     ...                                   \-\-mcp.config
-    ...                                  {"server": {"transport": "http", "address": "127.0.0.1:9912"} }
+    ...                                   {"server": {"transport": "http", "address": "127.0.0.1:9912"} }
     Sleep         5s
-    ${result}=    Run Process          ${REPOSITORY_ROOT}${/}build${/}stackql_mcp_client
+    ${result}=    Run Process          ${STACKQL_MCP_CLIENT_EXE}
     ...                  exec
     ...                  \-\-client\-type\=http 
     ...                  \-\-url\=http://127.0.0.1:9912