Skip to content

Add integration tests for connection-level behavior #34

Description

@mack42

Source: Audit finding (LOW)

Major modules have zero test coverage: `src/server/connection.rs`, `src/server/mod.rs`, `src/command/list.rs`, `src/command/set.rs`, `src/command/hash.rs`, `src/storage/store.rs`. The 79 existing unit tests are concentrated in the parser, RDB, and a few storage internals.

Specifically missing:

  • MULTI / EXEC / DISCARD atomicity
  • WATCH / EXEC cancellation (per-key version is new in 0.11.0 — needs regression coverage)
  • Pub/Sub message ordering and pattern-match dispatch
  • Client tracking invalidation (forward + reverse index)
  • AUTH / HELLO ACL flows
  • Pipelining boundaries

The `redis = "0.27"` dev-dependency is already present. Suggested layout: `tests/integration_*.rs` spawning a server on an ephemeral port and exercising via the real Redis client.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requesttier-2Medium priority

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions