Skip to content

[improve][test] Redis sink: optionally migrate embedded Redis to a Testcontainers image #112

Description

@david-streamlio

Current state (correction)

redis already has integration coverage: RedisSinkTest runs RedisSink end-to-end against a real embedded Redis server (redis.embedded.RedisServer, the embedded-redis test dependency). This is genuine functional coverage, not a mocked unit test — an earlier triage note incorrectly listed this as a gap.

Enhancement (optional, not a gap)

Migrate the embedded server to a Testcontainers-based redis image so the module:

  • matches the container-based approach used across the other connectors,
  • runs against an official Redis build rather than a bundled binary, and
  • can exercise auth / TLS / cluster configurations the embedded server can't.

This is a test-infrastructure improvement, not a coverage gap.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions