Skip to content

Commit

Permalink
chore: relax assertion in test_noreply_pipeline (#3908)
Browse files Browse the repository at this point in the history
*adjust assert condition

Signed-off-by: kostas <[email protected]>
  • Loading branch information
kostasrim authored Oct 14, 2024
1 parent cf2e94f commit 588d6cc
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tests/dragonfly/pymemcached_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ def test_basic(memcached_client: MCClient):
# Noreply (and pipeline) tests


@pytest.mark.skip("Flaky")
@dfly_args(DEFAULT_ARGS)
def test_noreply_pipeline(df_server: DflyInstance, memcached_client: MCClient):
"""
Expand All @@ -68,7 +67,7 @@ def test_noreply_pipeline(df_server: DflyInstance, memcached_client: MCClient):
assert memcached_client.get_many(keys) == {k: v.encode() for k, v in zip(keys, values)}

info = Redis(port=df_server.port).info()
assert info["total_pipelined_commands"] > len(keys) / 6 # sometimes CI is slow
assert info["total_pipelined_commands"] > 0 # sometimes CI is slow


@dfly_args(DEFAULT_ARGS)
Expand Down

0 comments on commit 588d6cc

Please sign in to comment.