diff options
| author | Josh Smith <qinusty@gmail.com> | 2018-09-07 15:36:43 +0100 |
|---|---|---|
| committer | Josh Smith <qinusty@gmail.com> | 2018-09-19 10:25:21 +0100 |
| commit | ca811a4d5905b61862c230e17925ab9152a78b16 (patch) | |
| tree | a9a647cff50c300f1634848a6061f94916c1f3ad /tests/frontend | |
| parent | 72b5902157316e173de2eec5b3a2772283eec3c7 (diff) | |
| download | buildstream-ca811a4d5905b61862c230e17925ab9152a78b16.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.py | 3 |
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 |