Implement the Redis Message Broker #11
Conversation
for more information, see https://pre-commit.ci
|
The build is failing due to an issue with the Python docker client. We should report it. |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
I opened an issue docker/docker-py#2948 |
| with subtests.test("Ensure docker container is running", message_broker_cls=message_broker_cls): | ||
| containers = client.containers.list() | ||
|
|
||
| assert len(containers) == 1 |
There was a problem hiding this comment.
We need to make this assertion specific.
|
This pull request introduces 2 alerts when merging 79ef3b0 into ec9a398 - view on LGTM.com new alerts:
|
|
This pull request introduces 2 alerts when merging ec42eb6 into ec9a398 - view on LGTM.com new alerts:
|
|
This pull request introduces 2 alerts when merging e05f307 into ec9a398 - view on LGTM.com new alerts:
|
…elery plugin is added by default to all tests, and it's disabled for unit tests by through additional arguments "-p no:celery".
|
This pull request introduces 2 alerts when merging e1bb9bd into ec9a398 - view on LGTM.com new alerts:
|
… to persist throughout the test session, so this is not the place to tear it down.
|
This pull request introduces 2 alerts when merging d101d37 into ec9a398 - view on LGTM.com new alerts:
|
|
This pull request introduces 2 alerts when merging 25600b5 into ec9a398 - view on LGTM.com new alerts:
|
|
This pull request introduces 2 alerts when merging 519fe03 into ec9a398 - view on LGTM.com new alerts:
|
…ontainer is not instantiated for the unit test.
|
This pull request introduces 2 alerts when merging 78a4833 into ec9a398 - view on LGTM.com new alerts:
|
|
this PR is current blocker to healthcheck branch right? |
This is the beginning of the effort to group tests with common test services together.
No description provided.