[buildstream] 35/41: tests: Test push/pull on all platforms

[tv...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

tvb pushed a commit to branch jmac/googlecas_and_virtual_directories_1
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit bcde352d0562c4d8694a2dadfe8ae760898c509c
Author: Jürg Billeter <j...@bitron.ch>
AuthorDate: Fri May 11 17:47:57 2018 +0200

    tests: Test push/pull on all platforms
---
 tests/artifactcache/junctions.py | 2 --
 tests/frontend/pull.py           | 6 ------
 tests/frontend/push.py           | 4 ----
 3 files changed, 12 deletions(-)

diff --git a/tests/artifactcache/junctions.py b/tests/artifactcache/junctions.py
index 3fcea97..378d007 100644
--- a/tests/artifactcache/junctions.py
+++ b/tests/artifactcache/junctions.py
@@ -2,7 +2,6 @@ import os
 import shutil
 import pytest
 from tests.testutils import cli, create_artifact_share
-from tests.testutils.site import IS_LINUX
 
 from buildstream import _yaml
 
@@ -37,7 +36,6 @@ def project_set_artifacts(project, url):
     _yaml.dump(_yaml.node_sanitize(project_config), filename=project_conf_file)
 
 
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull(cli, tmpdir, datafiles):
     project = os.path.join(str(datafiles), 'foo')
diff --git a/tests/frontend/pull.py b/tests/frontend/pull.py
index 8c30d01..0d38909 100644
--- a/tests/frontend/pull.py
+++ b/tests/frontend/pull.py
@@ -2,7 +2,6 @@ import os
 import shutil
 import pytest
 from tests.testutils import cli, create_artifact_share
-from tests.testutils.site import IS_LINUX
 
 # Project directory
 DATA_DIR = os.path.join(
@@ -40,7 +39,6 @@ def assert_not_shared(cli, share, project, element_name):
 #  * `bst build` pushes all build elements to configured 'push' cache
 #  * `bst pull --deps all` downloads everything from cache after local deletion
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_all(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -84,7 +82,6 @@ def test_push_pull_all(cli, tmpdir, datafiles):
 #  * `bst build` pushes all build elements ONLY to configured 'push' cache
 #  * `bst pull` finds artifacts that are available only in the secondary cache
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_pull_secondary_cache(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -126,7 +123,6 @@ def test_pull_secondary_cache(cli, tmpdir, datafiles):
 #  * `bst push --remote` pushes to the given remote, not one from the config
 #  * `bst pull --remote` pulls from the given remote
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_specific_remote(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -176,7 +172,6 @@ def test_push_pull_specific_remote(cli, tmpdir, datafiles):
 #
 #  * In non-strict mode, dependency changes don't block artifact reuse
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_non_strict(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -228,7 +223,6 @@ def test_push_pull_non_strict(cli, tmpdir, datafiles):
 
 
 # Regression test for https://gitlab.com/BuildStream/buildstream/issues/202
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_pull_track_non_strict(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
diff --git a/tests/frontend/push.py b/tests/frontend/push.py
index 721cd63..459c340 100644
--- a/tests/frontend/push.py
+++ b/tests/frontend/push.py
@@ -2,7 +2,6 @@ import os
 import pytest
 from buildstream._exceptions import ErrorDomain
 from tests.testutils import cli, create_artifact_share
-from tests.testutils.site import IS_LINUX
 
 # Project directory
 DATA_DIR = os.path.join(
@@ -40,7 +39,6 @@ def assert_not_shared(cli, share, project, element_name):
 #  * `bst push` fails if there are no remotes configured for pushing
 #  * `bst push` successfully pushes to any remote that is configured for pushing
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push(cli, tmpdir, datafiles):
     project = str(datafiles)
@@ -98,7 +96,6 @@ def test_push(cli, tmpdir, datafiles):
 
 # Tests that `bst push --deps all` pushes all dependencies of the given element.
 #
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_all(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -148,7 +145,6 @@ def test_push_all(cli, tmpdir, datafiles):
 # Tests that `bst build` won't push artifacts to the cache it just pulled from.
 #
 # Regression test for https://gitlab.com/BuildStream/buildstream/issues/233.
-@pytest.mark.skipif(not IS_LINUX, reason='Only available on linux')
 @pytest.mark.datafiles(DATA_DIR)
 def test_push_after_pull(cli, tmpdir, datafiles):
     project = os.path.join(datafiles.dirname, datafiles.basename)