Skip to content

Failing test with query timeout in test_graph.py #2591

Closed
@Ernest0x

Description

@Ernest0x

Version: Current master

Platform: Whatever platform the CI tests run on

Description: After merging upstream changes to my branch for #2470 I realized that there was one CI test failing (https://github.com/redis/redis-py/actions/runs/4174611486/jobs/7228412511) which seemed irrelevant with the (small) changes I introduce in that PR. Then I took a look at previous CI runs and I found out that the same test had failed for another, irrelevant branch too. My guess is that the failing test at

client.graph().query("MATCH (a), (b), (c), (d) RETURN *", timeout=1)
assumes that the query will be taking long enough time to cause a ResponseError when a 1-millisecond timeout argument is passed in the query call. So, it could be that the query has become fast enough lately so that the timeout is not reached.

Metadata

Metadata

Assignees

Labels

maintenanceMaintenance (CI, Releases, etc)

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions