Skip to content

Commit 5981a5e

Browse files
authored
Merge pull request #43 from damklis/develop
Add test for context manager
2 parents 1e09bc0 + d07112b commit 5981a5e

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

.coveragerc

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,5 @@ omit =
77
exclude_lines =
88
pragma: no cover
99
def __repr__
10-
def __enter__
11-
def __exit__
1210
raise NotImplementedError
1311
if __name__ == .__main__.:

airflow/modules/tests/proxypool/test_redis_proxypool_client.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,3 +63,11 @@ def test_get_proxy(redis, redis_config, redis_mock, proxies):
6363
result = redis_client.get_proxy()
6464

6565
assert result == expected
66+
67+
68+
@patch("proxypool.redis_proxypool_client.redis.StrictRedis")
69+
def test_redis_client_context_manager(redis, redis_config):
70+
key = "test"
71+
72+
with RedisProxyPoolClient(key, redis_config) as redis_client:
73+
pass

0 commit comments

Comments
 (0)