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/10/05 06:51:21 UTC

[buildstream] 02/05: tests/format/dependencies.py: Remove test_no_recurse()

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

tvb pushed a commit to branch tristan/remove-plan-selection-mode
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 70aba4298ee4c750197ca44b6f1fe1434065a80a
Author: Tristan van Berkom <tr...@codethink.co.uk>
AuthorDate: Tue Oct 5 15:24:29 2021 +0900

    tests/format/dependencies.py: Remove test_no_recurse()
    
    This has long ago become a literal duplicate of test_scope_all(), as in
    it was testing "--deps plan" behavior which was already identical to
    "--deps all" behavior, and the "plan" option will be removed.
---
 tests/format/dependencies.py | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/tests/format/dependencies.py b/tests/format/dependencies.py
index 789df06..7847506 100644
--- a/tests/format/dependencies.py
+++ b/tests/format/dependencies.py
@@ -214,27 +214,6 @@ def test_scope_build_of_child(cli, datafiles):
 
 
 @pytest.mark.datafiles(DATA_DIR)
-def test_no_recurse(cli, datafiles):
-    project = os.path.join(str(datafiles), "dependencies2")
-    elements = ["target.bst"]
-
-    # We abuse the 'plan' scope here to ensure that we call
-    # element.dependencies() with recurse=False - currently, no `bst
-    # show` option does this directly.
-    element_list = cli.get_pipeline(project, elements, scope="plan")
-
-    assert element_list == [
-        "build-build.bst",
-        "run-build.bst",
-        "build.bst",
-        "dep-one.bst",
-        "run.bst",
-        "dep-two.bst",
-        "target.bst",
-    ]
-
-
-@pytest.mark.datafiles(DATA_DIR)
 @pytest.mark.parametrize(
     "target", ["merge-separate-lists.bst", "merge-single-list.bst",], ids=["separate-lists", "single-list"],
 )