From 2df1014ee339de8754f1c01ae7a0bcdb6b5064d0 Mon Sep 17 00:00:00 2001 From: giles Date: Sat, 28 Feb 2026 23:30:17 +0000 Subject: [PATCH] Add Node.js to test containers for sexp.js parity tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Node 20 from Debian packages — needed to run test_sexp_js.py which verifies JS renderer output matches Python renderer output. Co-Authored-By: Claude Opus 4.6 --- test/Dockerfile | 2 +- test/Dockerfile.unit | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/test/Dockerfile b/test/Dockerfile index 0658adf..8970515 100644 --- a/test/Dockerfile +++ b/test/Dockerfile @@ -12,7 +12,7 @@ ENV PYTHONDONTWRITEBYTECODE=1 \ WORKDIR /app RUN apt-get update && apt-get install -y --no-install-recommends \ - ca-certificates \ + ca-certificates nodejs \ && rm -rf /var/lib/apt/lists/* COPY shared/requirements.txt ./requirements.txt diff --git a/test/Dockerfile.unit b/test/Dockerfile.unit index a2f8dfe..5e8098e 100644 --- a/test/Dockerfile.unit +++ b/test/Dockerfile.unit @@ -8,6 +8,10 @@ ENV PYTHONDONTWRITEBYTECODE=1 \ WORKDIR /app +# Node.js for sexp.js parity tests +RUN apt-get update && apt-get install -y --no-install-recommends nodejs && \ + rm -rf /var/lib/apt/lists/* + # Install shared deps (includes pytest, pytest-asyncio) COPY shared/requirements.txt ./requirements-shared.txt RUN pip install --no-cache-dir -r requirements-shared.txt pytest-watch