Merge pull request #212 from codecrafters-io/docker-local-tests

Add docker file for recording fixtures and running tests

Author: UdeshyaDhungana

docker_ci/docker-ci.sh

@@ -0,0 +1,28 @@
+#!/bin/bash
+
+set -e
+
+cd "$(dirname "$0")/.."
+
+# Build image
+docker_build() {
+    docker build -t redis-tester -f docker_ci/ubuntu.Dockerfile .
+}
+
+case "$1" in
+  test)
+    echo "🧪 Running tests using Docker container"
+    docker_build
+    docker run --rm -it -v "$(pwd)":/app -w /app redis-tester make test
+    ;;
+  record_fixtures)
+    echo "📝 Recording fixtures using Docker container"
+    docker_build
+    docker run --rm -it -v "$(pwd)":/app -w /app redis-tester make record_fixtures
+    ;;
+  *)
+    echo "Usage:"
+    echo "$0 [test|record_fixtures]"
+    exit 1
+    ;;
+esac

docker_ci/ubuntu.Dockerfile

@@ -0,0 +1,52 @@
+# Use official Go 1.24 image as base
+FROM golang:1.24-bookworm
+
+# Set environment variables
+ENV CGO_ENABLED=0
+ENV GOOS=linux
+ENV GOARCH=amd64
+
+# Install system dependencies
+RUN apt-get update && apt-get install -y \
+    curl \
+    wget \
+    git \
+    build-essential \
+    ca-certificates \
+    gnupg \
+    lsb-release \
+    python3 \
+    python3-pip \
+    python3-venv \
+    netcat-openbsd \
+    && rm -rf /var/lib/apt/lists/*
+
+
+# Install Redis
+RUN curl -fsSL https://packages.redis.io/gpg | gpg --dearmor -o /usr/share/keyrings/redis-archive-keyring.gpg \
+    && chmod 644 /usr/share/keyrings/redis-archive-keyring.gpg \
+    && echo "deb [signed-by=/usr/share/keyrings/redis-archive-keyring.gpg] https://packages.redis.io/deb $(lsb_release -cs) main" | tee /etc/apt/sources.list.d/redis.list \
+    && apt-get update \
+    && apt-get install -y redis \
+    && rm -rf /var/lib/apt/lists/*
+
+# Create workspace directory
+WORKDIR /workspace
+
+# Copy go mod files first for better caching
+COPY go.mod go.sum ./
+
+# Download Go dependencies
+RUN go mod download
+
+# Copy the rest of the source code
+COPY . .
+
+# Build the project
+RUN go build -o dist/main.out ./cmd/tester
+
+# Expose port for Redis (if needed)
+EXPOSE 6379
+
+# Keep the container alive
+CMD ["/bin/bash"]

internal/test_repl_replica_cmd_processing.go

@@ -3,6 +3,7 @@ package internal
 import (
 	"fmt"
 	"net"
+	"time"
 
 	"github.com/codecrafters-io/redis-tester/internal/instrumented_resp_connection"
 	"github.com/codecrafters-io/redis-tester/internal/redis_executable"
@@ -81,6 +82,10 @@ func testReplCmdProcessing(stageHarness *test_case_harness.TestCaseHarness) erro
 
 	logger.UpdateLastSecondaryPrefix("test")
 	replicaClient.UpdateBaseLogger(logger)
+	// Add a small delay to wait for commands to be propagated
+	// This was done in https://github.com/codecrafters-io/redis-tester/pull/212 to adjust fixtures
+	// when fixtures are recorded from docker container
+	time.Sleep(time.Millisecond)
 
 	for i := 1; i <= len(kvMap); i++ {
 		key, value := kvMap[i][0], kvMap[i][1]