You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildstream.apache.org by tv...@apache.org on 2021/02/04 07:43:47 UTC
[buildstream] 01/12: tests/frontend/push.py: Allow pushing of
dependencies
This is an automated email from the ASF dual-hosted git repository.
tvb pushed a commit to branch juerg/cache-query-job-benchmark
in repository https://gitbox.apache.org/repos/asf/buildstream.git
commit 847703c48ad311a74d0028399b52fbd4fff90b56
Author: Jürg Billeter <j...@bitron.ch>
AuthorDate: Mon Sep 14 11:24:59 2020 +0200
tests/frontend/push.py: Allow pushing of dependencies
The assertions in `test_push_after_pull` are too strict. Pushing
dependencies to the second (empty) artifact server should not cause a
test failure.
---
tests/frontend/push.py | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tests/frontend/push.py b/tests/frontend/push.py
index 4e39c22..2106b6d 100644
--- a/tests/frontend/push.py
+++ b/tests/frontend/push.py
@@ -372,8 +372,8 @@ def test_push_after_pull(cli, tmpdir, datafiles):
#
result = cli.run(project=project, args=["build", "target.bst"])
result.assert_success()
- assert result.get_pulled_elements() == ["target.bst"]
- assert result.get_pushed_elements() == []
+ assert "target.bst" in result.get_pulled_elements()
+ assert "target.bst" not in result.get_pushed_elements()
# Delete the artifact locally again.
cli.remove_artifact_from_cache(project, "target.bst")
@@ -383,8 +383,8 @@ def test_push_after_pull(cli, tmpdir, datafiles):
cli.configure({"artifacts": [{"url": share1.repo, "push": True}, {"url": share2.repo, "push": True},]})
result = cli.run(project=project, args=["build", "target.bst"])
result.assert_success()
- assert result.get_pulled_elements() == ["target.bst"]
- assert result.get_pushed_elements() == ["target.bst"]
+ assert "target.bst" in result.get_pulled_elements()
+ assert "target.bst" in result.get_pushed_elements()
# Ensure that when an artifact's size exceeds available disk space