diff options
-rw-r--r-- | src/buildstream/testing/_sourcetests/source_determinism.py | 1 | ||||
-rw-r--r-- | src/buildstream/testing/_utils/site.py | 6 | ||||
-rw-r--r-- | tests/elements/filter.py | 2 | ||||
-rw-r--r-- | tests/examples/autotools.py | 1 | ||||
-rw-r--r-- | tests/frontend/configurable_warnings.py | 3 | ||||
-rw-r--r-- | tests/integration/autotools.py | 1 | ||||
-rw-r--r-- | tests/integration/cmake.py | 1 | ||||
-rw-r--r-- | tests/integration/compose.py | 1 | ||||
-rw-r--r-- | tests/integration/interactive_build.py | 2 | ||||
-rw-r--r-- | tests/integration/sandbox.py | 1 | ||||
-rw-r--r-- | tests/integration/script.py | 2 | ||||
-rw-r--r-- | tests/integration/shell.py | 8 | ||||
-rw-r--r-- | tests/integration/workspace.py | 2 | ||||
-rw-r--r-- | tests/sourcecache/push.py | 2 |
14 files changed, 5 insertions, 28 deletions
diff --git a/src/buildstream/testing/_sourcetests/source_determinism.py b/src/buildstream/testing/_sourcetests/source_determinism.py index f84d18ba2..d51d0e520 100644 --- a/src/buildstream/testing/_sourcetests/source_determinism.py +++ b/src/buildstream/testing/_sourcetests/source_determinism.py @@ -50,7 +50,6 @@ def create_test_directory(*path, mode=0o644): @pytest.mark.integration @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.skipif(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox, Must Fix") def test_deterministic_source_umask(cli, tmpdir, datafiles, kind): project = str(datafiles) element_name = "list.bst" diff --git a/src/buildstream/testing/_utils/site.py b/src/buildstream/testing/_utils/site.py index 5507df0f7..f3b45dbf6 100644 --- a/src/buildstream/testing/_utils/site.py +++ b/src/buildstream/testing/_utils/site.py @@ -70,12 +70,6 @@ casd_path = utils.get_host_tool("buildbox-casd") CASD_SEPARATE_USER = bool(os.stat(casd_path).st_mode & stat.S_ISUID) del casd_path -try: - utils.get_host_tool("buildbox") - HAVE_BUILDBOX = True -except ProgramNotFoundError: - HAVE_BUILDBOX = False - IS_LINUX = os.getenv("BST_FORCE_BACKEND", sys.platform).startswith("linux") IS_WSL = IS_LINUX and "Microsoft" in platform.uname().release IS_WINDOWS = os.name == "nt" diff --git a/tests/elements/filter.py b/tests/elements/filter.py index 0befa46f8..3b38094dc 100644 --- a/tests/elements/filter.py +++ b/tests/elements/filter.py @@ -8,7 +8,6 @@ import pytest from buildstream.testing import create_repo from buildstream.testing import cli # pylint: disable=unused-import -from buildstream.testing._utils.site import HAVE_SANDBOX from buildstream._exceptions import ErrorDomain from buildstream import _yaml @@ -28,7 +27,6 @@ def test_filter_include(datafiles, cli, tmpdir): assert not os.path.exists(os.path.join(checkout, "bar")) -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") @pytest.mark.datafiles(os.path.join(DATA_DIR, "basic")) def test_filter_include_dynamic(datafiles, cli, tmpdir): project = str(datafiles) diff --git a/tests/examples/autotools.py b/tests/examples/autotools.py index d4c12ecac..46b9d117e 100644 --- a/tests/examples/autotools.py +++ b/tests/examples/autotools.py @@ -47,7 +47,6 @@ def test_autotools_build(cli, datafiles): # Test running an executable built with autotools. @pytest.mark.skipif(MACHINE_ARCH != "x86-64", reason="Examples are written for x86-64") @pytest.mark.skipif(not IS_LINUX or not HAVE_SANDBOX, reason="Only available on linux with sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") @pytest.mark.skipif(HAVE_SANDBOX == "chroot", reason="This test is not meant to work with chroot sandbox") @pytest.mark.datafiles(DATA_DIR) def test_autotools_run(cli, datafiles): diff --git a/tests/frontend/configurable_warnings.py b/tests/frontend/configurable_warnings.py index 472df3eea..488dab233 100644 --- a/tests/frontend/configurable_warnings.py +++ b/tests/frontend/configurable_warnings.py @@ -9,7 +9,6 @@ from buildstream.plugin import CoreWarnings from buildstream._exceptions import ErrorDomain from buildstream import _yaml from buildstream.testing.runcli import cli # pylint: disable=unused-import -from buildstream.testing._utils.site import HAVE_SANDBOX TOP_DIR = os.path.join(os.path.dirname(os.path.realpath(__file__)), "configuredwarning") @@ -50,8 +49,6 @@ def build_project(datafiles, fatal_warnings): ], ) def test_fatal_warnings(cli, datafiles, element_name, fatal_warnings, expect_fatal, error_domain): - if HAVE_SANDBOX == "buildbox" and error_domain != ErrorDomain.STREAM: - pytest.xfail() project_path = build_project(datafiles, fatal_warnings) result = cli.run(project=project_path, args=["build", element_name]) diff --git a/tests/integration/autotools.py b/tests/integration/autotools.py index 7a3b946b2..86a06aa4f 100644 --- a/tests/integration/autotools.py +++ b/tests/integration/autotools.py @@ -49,7 +49,6 @@ def test_autotools_build(cli, datafiles): # amhello project for this. @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") def test_autotools_confroot_build(cli, datafiles): project = str(datafiles) checkout = os.path.join(cli.directory, "checkout") diff --git a/tests/integration/cmake.py b/tests/integration/cmake.py index f5d2f6cc9..db0f07531 100644 --- a/tests/integration/cmake.py +++ b/tests/integration/cmake.py @@ -33,7 +33,6 @@ def test_cmake_build(cli, datafiles): @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") def test_cmake_confroot_build(cli, datafiles): project = str(datafiles) checkout = os.path.join(cli.directory, "checkout") diff --git a/tests/integration/compose.py b/tests/integration/compose.py index 5fb79d051..2f38aa66c 100644 --- a/tests/integration/compose.py +++ b/tests/integration/compose.py @@ -133,7 +133,6 @@ def test_compose_include(cli, datafiles, include_domains, exclude_domains, expec @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") def test_compose_run_integration(cli, datafiles): project = str(datafiles) checkout = os.path.join(cli.directory, "checkout") diff --git a/tests/integration/interactive_build.py b/tests/integration/interactive_build.py index f4b974042..9c832814d 100644 --- a/tests/integration/interactive_build.py +++ b/tests/integration/interactive_build.py @@ -53,7 +53,7 @@ def test_failed_build_quit(element_name, build_session, choice): @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX in ["buildbox", "buildbox-run"], reason="Not working with BuildBox") +@pytest.mark.xfail(HAVE_SANDBOX == "buildbox-run", reason="Not working with BuildBox") @pytest.mark.datafiles(DATA_DIR) @pytest.mark.parametrize("element_name", ["interactive/failed-build.bst"]) def test_failed_build_log(element_name, build_session): diff --git a/tests/integration/sandbox.py b/tests/integration/sandbox.py index d772f5437..ec41f1d92 100644 --- a/tests/integration/sandbox.py +++ b/tests/integration/sandbox.py @@ -32,7 +32,6 @@ DATA_DIR = os.path.join(os.path.dirname(os.path.realpath(__file__)), "project") @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="The buildbox sandbox doesn't support shm") @pytest.mark.xfail(HAVE_SANDBOX == "chroot", reason="The chroot sandbox doesn't support shm") @pytest.mark.datafiles(DATA_DIR) def test_sandbox_shm(cli, datafiles): diff --git a/tests/integration/script.py b/tests/integration/script.py index b5e21452c..67dd310bd 100644 --- a/tests/integration/script.py +++ b/tests/integration/script.py @@ -91,7 +91,6 @@ def test_script_root(cli, datafiles): @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") @pytest.mark.xfail( HAVE_SANDBOX == "buildbox-run", reason="Read-only root directory not supported by buildbox-run", ) @@ -121,7 +120,6 @@ def test_script_no_root(cli, datafiles): @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") def test_script_cwd(cli, datafiles): project = str(datafiles) checkout = os.path.join(cli.directory, "checkout") diff --git a/tests/integration/shell.py b/tests/integration/shell.py index 394edeb1d..cea608016 100644 --- a/tests/integration/shell.py +++ b/tests/integration/shell.py @@ -158,7 +158,7 @@ def test_no_shell(cli, datafiles): @pytest.mark.parametrize("path", [("/etc/pony.conf"), ("/usr/share/pony/pony.txt")]) @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX in ["buildbox", "buildbox-run"], reason="Not working with BuildBox") +@pytest.mark.xfail(HAVE_SANDBOX == "buildbox-run", reason="Not working with BuildBox") def test_host_files(cli, datafiles, path): project = str(datafiles) ponyfile = os.path.join(project, "files", "shell-mount", "pony.txt") @@ -173,7 +173,7 @@ def test_host_files(cli, datafiles, path): @pytest.mark.parametrize("path", [("/etc"), ("/usr/share/pony")]) @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX in ["buildbox", "buildbox-run"], reason="Not working with BuildBox") +@pytest.mark.xfail(HAVE_SANDBOX == "buildbox-run", reason="Not working with BuildBox") def test_host_files_expand_environ(cli, datafiles, path): project = str(datafiles) hostpath = os.path.join(project, "files", "shell-mount") @@ -246,7 +246,7 @@ def test_host_files_missing(cli, datafiles, optional): @pytest.mark.parametrize("path", [("/etc/pony.conf"), ("/usr/share/pony/pony.txt")]) @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX in ["buildbox", "buildbox-run"], reason="Not working with BuildBox") +@pytest.mark.xfail(HAVE_SANDBOX == "buildbox-run", reason="Not working with BuildBox") def test_cli_mount(cli, datafiles, path): project = str(datafiles) ponyfile = os.path.join(project, "files", "shell-mount", "pony.txt") @@ -294,7 +294,7 @@ def test_workspace_visible(cli, datafiles): # Test that '--sysroot' works @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX in ["buildbox", "buildbox-run"], reason="Not working with BuildBox") +@pytest.mark.xfail(HAVE_SANDBOX == "buildbox-run", reason="Not working with BuildBox") def test_sysroot(cli, tmpdir, datafiles): project = str(datafiles) base_element = "base/base-alpine.bst" diff --git a/tests/integration/workspace.py b/tests/integration/workspace.py index 8b4baddef..372db672a 100644 --- a/tests/integration/workspace.py +++ b/tests/integration/workspace.py @@ -62,7 +62,6 @@ def test_workspace_mount_on_read_only_directory(cli, datafiles): @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") @pytest.mark.xfail(reason="Incremental builds are currently incompatible with workspace source plugin.") def test_workspace_commanddir(cli, datafiles): project = str(datafiles) @@ -318,7 +317,6 @@ def test_incremental_configure_commands_run_only_once(cli, datafiles): # @pytest.mark.datafiles(DATA_DIR) @pytest.mark.skipif(not HAVE_SANDBOX, reason="Only available with a functioning sandbox") -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") def test_workspace_missing_last_successful(cli, datafiles): project = str(datafiles) workspace = os.path.join(cli.directory, "workspace") diff --git a/tests/sourcecache/push.py b/tests/sourcecache/push.py index 0b7bb9954..771a94ca1 100644 --- a/tests/sourcecache/push.py +++ b/tests/sourcecache/push.py @@ -30,7 +30,6 @@ from buildstream._project import Project from buildstream import _yaml from buildstream.testing import cli # pylint: disable=unused-import from buildstream.testing import create_repo -from buildstream.testing._utils.site import HAVE_SANDBOX from tests.testutils import create_artifact_share, dummy_context DATA_DIR = os.path.join(os.path.dirname(os.path.realpath(__file__)), "project") @@ -225,7 +224,6 @@ def test_push_fail(cli, tmpdir, datafiles): assert "Pushed" not in res.stderr -@pytest.mark.xfail(HAVE_SANDBOX == "buildbox", reason="Not working with BuildBox") @pytest.mark.datafiles(DATA_DIR) def test_source_push_build_fail(cli, tmpdir, datafiles): project_dir = str(datafiles) |