Skip to content

Add unit test case for redis #52317

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

Lzzz666
Copy link
Contributor

@Lzzz666 Lzzz666 commented Jun 27, 2025

Related: #35442

The missing Redis tests from #35442 have been added by other contributors:

  • tests/providers/redis/operators/test_redis_publish.py
  • tests/providers/redis/sensors/test_redis_key.py

However, there was an issue with the naming, so I fixed it.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@jason810496 jason810496 self-requested a review June 27, 2025 02:42
@Lzzz666 Lzzz666 marked this pull request as draft June 27, 2025 03:19
@Lzzz666 Lzzz666 force-pushed the redis_test_module branch 2 times, most recently from c853fbb to b201ef6 Compare June 27, 2025 12:25
@Lzzz666 Lzzz666 changed the title Add unit test for redis version compatibility Modify test class name for redisKeySensor Jun 27, 2025
@Lzzz666 Lzzz666 changed the title Modify test class name for redisKeySensor Add unit test case for redis Jun 27, 2025
@Lzzz666 Lzzz666 force-pushed the redis_test_module branch from ab71e4e to cd05c6d Compare June 27, 2025 14:29
@Lzzz666 Lzzz666 marked this pull request as ready for review June 27, 2025 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant