PING, because the reported port was once a fiction #4

Manually merged
mi merged 1 commits from redis-quirk into main 2026-09-11 10:23:01 +02:00
Owner

Werkdock's quirk table listed redis as moving its port through
REDIS_PORT. Redis does not read it, so up to werkdock 0.7.0 the port
Testcontainers was given had nothing listening on it — and no test here
would have noticed, because every test was a PostgreSQL one.

This is the cheapest possible guard against that class of mistake: a
GenericContainer, no client library, PING and PONG over a plain socket.
It asks the only question that matters — is a server really answering
where werkdock said it would — and it asks it about an image that is not
a database, which is where the table was wrong.

Needs werkdock 0.8.0 and redis:7-alpine in the store.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com

Werkdock's quirk table listed redis as moving its port through REDIS_PORT. Redis does not read it, so up to werkdock 0.7.0 the port Testcontainers was given had nothing listening on it — and no test here would have noticed, because every test was a PostgreSQL one. This is the cheapest possible guard against that class of mistake: a GenericContainer, no client library, PING and PONG over a plain socket. It asks the only question that matters — is a server really answering where werkdock said it would — and it asks it about an image that is not a database, which is where the table was wrong. Needs werkdock 0.8.0 and redis:7-alpine in the store. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
mi added 1 commit 2026-09-11 10:22:43 +02:00
Werkdock's quirk table listed redis as moving its port through
REDIS_PORT. Redis does not read it, so up to werkdock 0.7.0 the port
Testcontainers was given had nothing listening on it — and no test here
would have noticed, because every test was a PostgreSQL one.

This is the cheapest possible guard against that class of mistake: a
GenericContainer, no client library, PING and PONG over a plain socket.
It asks the only question that matters — is a server really answering
where werkdock said it would — and it asks it about an image that is not
a database, which is where the table was wrong.

Needs werkdock 0.8.0 and redis:7-alpine in the store.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
mi closed this pull request 2026-09-11 10:22:47 +02:00
mi reopened this pull request 2026-09-11 10:22:51 +02:00
mi manually merged commit a6e98d7b1e into main 2026-09-11 10:23:01 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mi/werkprobe#4