Update redis module so that it can work with redis-stack similar to testcontainers-go

[kevbook]

Expected Behaviour

const container = new RedisContainer('redis/redis-stack-server:7.2.0-v12')

This will fail because of the redis-server command.

testcontainers-node/packages/modules/redis/src/redis-container.ts

Lines 36 to 39 in 9f7fd4e

	this.withCommand([
	"redis-server",
	...(this.password ? [`--requirepass "${this.password}"`] : []),
	...(this.persistenceVolume ? ["--save 1 1 ", "--appendonly yes"] : []),

As you can see from the docker images, redis uses CMD [redis-server] but redis-stack-server uses a custom entrypoint script. So I'm happy to open a PR, basically based on image string passed, we'll need to change how container.start() is handled (It's messy but not sure how else to solve it when you have 2 Redis images now!)

• https://hub.docker.com/layers/library/redis/7.2/images/sha256-74de23f6746eb84af59a6ae86e5d3c4306078dd2a45cb02bf1556ac438436e95?context=explore
• https://hub.docker.com/layers/redis/redis-stack-server/7.2.0-v12/images/sha256-af91865c72c3c51b62a15fb3bcf9eae8d958672820fc91ae5c0b4c39ff5c9b53?context=explore (redis/redis-stack-server:7.2.0-v12)

[kevbook]

Just add this line. All tests should pass

/* Make sure the redis image is a not a `redis-stack` image as `redis-stack` images have a custom entrypoint, see https://hub.docker.com/r/redis/redis-stack-server */
    if (!this.imageName.image.includes("redis-stack")) {
      this.withCommand([
        "redis-server",
        ...(this.password ? [`--requirepass "${this.password}"`] : []),
        ...(this.persistenceVolume ? ["--save 1 1 ", "--appendonly yes"] : []),
      ]);
    }

With redis-stack this.withCommand will not be applicable as all custom args are passed via env vars.

[cristianrgreco]

@kevbook PR is welcome

[digital88]

@kevbook Hi, this runs without errors (at least on e7c499e):

  it("should start from redis-stack-server image", async () => {
    const container = await new RedisContainer("redis/redis-stack-server:7.2.0-v12").start();

    await container.stop();
  });

[crcarlo]

@digital88 @cristianrgreco Hi! Event though the container starts by using redis/redis-stack-server image, it doesn't seem to have JSON module enabled as you can verify from the test below.

https://github.com/crcarlo/testcontainers-node/blob/test/redis-stack-server-json-module/packages/modules/redis/src/redis-container.test.ts#L105

The test results in

   × RedisContainer > should start with redis-stack-server and json module 113ms
     → ERR unknown command 'JSON.SET', with args beginning with: 'key' '$' '"{\"name\":\"test\"}"' 

Also, by inspecting the container and running redis-cli I get

> MODULE LIST
(empty array)

[cristianrgreco]

@crcarlo Redis modules aren't implemented, feel free to raise a PR