run container tests on CI

Author: anonrig

.github/workflows/_bazel.yml

@@ -86,6 +86,9 @@ jobs:
           # resulting in a snowballing cache size and cache download/upload times.
       - name: Setup Runner
         uses: ./.github/actions/setup-runner
+      - name: Setup Docker on Linux
+        if: runner.os == 'Linux'
+        uses: docker/setup-docker-action@v4
       - name: Setup lld on macOS
         if: runner.os == 'macOS' && inputs.macos_use_lld
         run: |

build/ci.bazelrc

@@ -89,6 +89,9 @@ test:ci-linux --test_size_filters=
 test:ci-macOS --test_size_filters=
 test:ci-windows --test_size_filters=
 
+# This ensures tests tagged with "requires-container-engine" run only in CI, not locally
+test:ci-linux --test_tag_filters=+requires-container-engine
+
 # Enable reporting whenever WPT tests are run
 test:ci-test --config=wpt-test
 common:wpt-test --test_env=GEN_TEST_REPORT=1

src/workerd/server/tests/container-client/BUILD.bazel

@@ -5,5 +5,8 @@ wd_test(
     src = "test.wd-test",
     args = ["--experimental"],
     data = ["test.js"],
-    tags = ["off-by-default"],
+    tags = [
+        "off-by-default",
+        "requires-container-engine",
+    ],
 )

src/workerd/server/tests/container-client/test.js

@@ -223,8 +223,8 @@ export const testBasics = {
 // Test exit code monitor functionality
 export const testExitCode = {
   async test(_ctrl, env) {
-    const id = env.CONTAINER.idFromName('testExitCode');
-    const stub = env.CONTAINER.get(id);
+    const id = env.MY_CONTAINER.idFromName('testExitCode');
+    const stub = env.MY_CONTAINER.get(id);
     await stub.testExitCode();
   },
 };