Skip to content

Commit fabb657

Browse files
committed
Merge branch 'master' into testfixes
2 parents 82bbd71 + fc621bd commit fabb657

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

.github/workflows/integration.yaml

-2
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,7 @@ name: CI
22

33
on:
44
push:
5-
branches: [ master ]
65
pull_request:
7-
branches: [ master ]
86

97
jobs:
108
integration:

tests/test_pubsub.py

+3-4
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
1-
import pytest
21
import threading
32
import time
4-
53
from unittest import mock
64

5+
import pytest
76
import redis
87
from redis.exceptions import ConnectionError
98

10-
from .conftest import _get_client
11-
from .conftest import skip_if_server_version_lt
9+
from .conftest import _get_client, skip_if_server_version_lt
1210

1311

1412
def wait_for_message(pubsub, timeout=0.1, ignore_subscribe_messages=False):
@@ -561,6 +559,7 @@ def exception_handler(ex, pubsub, thread):
561559
with mock.patch.object(p, 'get_message',
562560
side_effect=Exception('error')):
563561
pubsub_thread = p.run_in_thread(
562+
daemon=True,
564563
exception_handler=exception_handler
565564
)
566565

0 commit comments

Comments
 (0)