Skip to content

Commit

Permalink
update docstrings and pin pytest
Browse files Browse the repository at this point in the history
  • Loading branch information
smn committed Aug 23, 2016
1 parent 614efb3 commit 4760c3c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
7 changes: 6 additions & 1 deletion consular/tests/test_main.py
Original file line number Diff line number Diff line change
Expand Up @@ -654,7 +654,8 @@ def test_sync_app_tasks_grouped(self):
@inlineCallbacks
def test_sync_app_tasks_task_lost(self):
"""
When syncing an app with a task, Consul is updated with a service entry
When syncing an app with a task that has the TASK_LOST state,
Consul should not be updated with a service entry
for the task.
"""
d = self.consular.sync_app_tasks({'id': '/my-app'})
Expand Down Expand Up @@ -1112,6 +1113,10 @@ def test_purge_dead_services(self):

@inlineCallbacks
def test_purge_task_lost_services(self):
"""
When a task has anything but the TASK_RUNNING state it should
be deregistered from Consul
"""
d = self.consular.purge_dead_services()
consul_request = yield self.requests.get()
consul_request['deferred'].callback(
Expand Down
2 changes: 1 addition & 1 deletion requirements-dev.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
pytest
pytest>=3.0.0
pytest-coverage
pytest-xdist
flake8
Expand Down

0 comments on commit 4760c3c

Please sign in to comment.