summaryrefslogtreecommitdiff
path: root/tests/frontend
diff options
context:
space:
mode:
authorJosh Smith <qinusty@gmail.com>2018-09-07 15:36:43 +0100
committerJosh Smith <qinusty@gmail.com>2018-09-19 13:26:24 +0100
commit8a378557facfb20ba97991ca81413308264e5904 (patch)
treeb10d2cf016b8d80cefab23dc9eede4c328642da5 /tests/frontend
parentc7e5d8be081314e4ff8414677d163d1a961b0d05 (diff)
downloadbuildstream-8a378557facfb20ba97991ca81413308264e5904.tar.gz
Rework Skipped usage
The SKIPPED message type is now used to indicate the end of a task which was successful without having to perform the given task. This overhauls the use of `Queue.done()` and therefore queues do not need to provide a processed/skipped return value from `done()`. Instead this is replaced with the action of raising a `SkipJob` exception from within `Queue.process()`.
Diffstat (limited to 'tests/frontend')
-rw-r--r--tests/frontend/pull.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/frontend/pull.py b/tests/frontend/pull.py
index ed9a9643e..c883e2030 100644
--- a/tests/frontend/pull.py
+++ b/tests/frontend/pull.py
@@ -356,4 +356,5 @@ def test_pull_missing_notifies_user(caplog, cli, tmpdir, datafiles):
assert not result.get_pulled_elements(), \
"No elements should have been pulled since the cache was empty"
- assert "SKIPPED Remote ({}) does not have".format(share.repo) in result.stderr
+ assert "INFO Remote ({}) does not have".format(share.repo) in result.stderr
+ assert "SKIPPED Pull" in result.stderr