tests: Migrate dependencies test to tests/format

This used to be an internal test, converted this to use the `cli` fixture.

tests/format/dependencies.py

+import os
+import pytest
+
+from buildstream._exceptions import ErrorDomain, LoadErrorReason
+from tests.testutils import cli
+
+DATA_DIR = os.path.join(
+    os.path.dirname(os.path.realpath(__file__)),
+    'dependencies',
+)
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_two_files(cli, datafiles):
+    project = str(datafiles)
+
+    elements = cli.get_pipeline(project, ['target.bst'])
+    assert elements == ['firstdep.bst', 'target.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_shared_dependency(cli, datafiles):
+    project = str(datafiles)
+
+    elements = cli.get_pipeline(project, ['shareddeptarget.bst'])
+    assert elements == ['firstdep.bst', 'shareddep.bst', 'shareddeptarget.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_dependency_dict(cli, datafiles):
+    project = str(datafiles)
+    elements = cli.get_pipeline(project, ['target-depdict.bst'])
+    assert elements == ['firstdep.bst', 'target-depdict.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_invalid_dependency_declaration(cli, datafiles):
+    project = str(datafiles)
+    result = cli.run(project=project, args=['show', 'invaliddep.bst'])
+    result.assert_main_error(ErrorDomain.LOAD, LoadErrorReason.INVALID_DATA)
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_invalid_dependency_type(cli, datafiles):
+    project = str(datafiles)
+    result = cli.run(project=project, args=['show', 'invaliddeptype.bst'])
+    result.assert_main_error(ErrorDomain.LOAD, LoadErrorReason.INVALID_DATA)
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_circular_dependency(cli, datafiles):
+    project = str(datafiles)
+    result = cli.run(project=project, args=['show', 'circulartarget.bst'])
+    result.assert_main_error(ErrorDomain.LOAD, LoadErrorReason.CIRCULAR_DEPENDENCY)
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_build_dependency(cli, datafiles):
+    project = str(datafiles)
+
+    elements = cli.get_pipeline(project, ['builddep.bst'], scope='run')
+    assert elements == ['builddep.bst']
+
+    elements = cli.get_pipeline(project, ['builddep.bst'], scope='build')
+    assert elements == ['firstdep.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_runtime_dependency(cli, datafiles):
+    project = str(datafiles)
+    elements = cli.get_pipeline(project, ['runtimedep.bst'], scope='build')
+
+    # FIXME: The empty line should probably never happen here when there are no results.
+    assert elements == ['']
+    elements = cli.get_pipeline(project, ['runtimedep.bst'], scope='run')
+    assert elements == ['firstdep.bst', 'runtimedep.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_all_dependency(cli, datafiles):
+    project = str(datafiles)
+
+    elements = cli.get_pipeline(project, ['alldep.bst'], scope='build')
+    assert elements == ['firstdep.bst']
+
+    elements = cli.get_pipeline(project, ['alldep.bst'], scope='run')
+    assert elements == ['firstdep.bst', 'alldep.bst']
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_list_build_dependency(cli, datafiles):
+    project = str(datafiles)
+
+    # Check that the pipeline includes the build dependency
+    deps = cli.get_pipeline(project, ['builddep-list.bst'], scope="build")
+    assert "firstdep.bst" in deps
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_list_runtime_dependency(cli, datafiles):
+    project = str(datafiles)
+
+    # Check that the pipeline includes the runtime dependency
+    deps = cli.get_pipeline(project, ['runtimedep-list.bst'], scope="run")
+    assert "firstdep.bst" in deps
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_list_dependencies_combined(cli, datafiles):
+    project = str(datafiles)
+
+    # Check that runtime deps get combined
+    rundeps = cli.get_pipeline(project, ['list-combine.bst'], scope="run")
+    assert "firstdep.bst" not in rundeps
+    assert "seconddep.bst" in rundeps
+    assert "thirddep.bst" in rundeps
+
+    # Check that build deps get combined
+    builddeps = cli.get_pipeline(project, ['list-combine.bst'], scope="build")
+    assert "firstdep.bst" in builddeps
+    assert "seconddep.bst" not in builddeps
+    assert "thirddep.bst" in builddeps
+
+
+@pytest.mark.datafiles(DATA_DIR)
+def test_list_overlap(cli, datafiles):
+    project = str(datafiles)
+
+    # Check that dependencies get merged
+    rundeps = cli.get_pipeline(project, ['list-overlap.bst'], scope="run")
+    assert "firstdep.bst" in rundeps
+    builddeps = cli.get_pipeline(project, ['list-overlap.bst'], scope="build")
+    assert "firstdep.bst" in builddeps

tests/loader/dependencies/elements/alldep.bst → tests/format/dependencies/elements/alldep.bst

-kind: pony
+kind: manual
 description: This element has a dependency with type 'all'
 depends:
-  - filename: elements/firstdep.bst
+  - filename: firstdep.bst
     type: all

tests/loader/dependencies/elements/builddep-list.bst → tests/format/dependencies/elements/builddep-list.bst

 kind: stack
 description: This element has a build-only dependency specified via build-depends
 build-depends:
-  - elements/firstdep.bst
+  - firstdep.bst

tests/loader/dependencies/elements/builddep.bst → tests/format/dependencies/elements/builddep.bst

-kind: pony
+kind: manual
 description: This element has a build-only dependency
 depends:
-  - filename: elements/firstdep.bst
+  - filename: firstdep.bst
     type: build

tests/loader/dependencies/elements/circular-firstdep.bst → tests/format/dependencies/elements/circular-firstdep.bst

-kind: pony
+kind: manual
 description: Depend on another dep which depends on the target
 depends:
-- elements/circular-seconddep.bst
+- circular-seconddep.bst

tests/loader/dependencies/elements/circular-seconddep.bst → tests/format/dependencies/elements/circular-seconddep.bst

-kind: pony
+kind: manual
 description: Depend on the target, creating a circular dependency
 depends:
-- elements/circulartarget.bst
+- circulartarget.bst

tests/loader/dependencies/elements/circulartarget.bst → tests/format/dependencies/elements/circulartarget.bst

-kind: pony
+kind: stack
 description: This is a main target which introduces a circular dependency
 depends:
-- elements/circular-firstdep.bst
+- circular-firstdep.bst

tests/loader/dependencies/elements/invaliddeptype.bst → tests/format/dependencies/elements/invaliddeptype.bst

-kind: pony
+kind: manual
 description: This is an invalid dependency type
 depends:
-  - filename: elements/firstdep.bst
+  - filename: firstdep.bst
     type: should be build or runtime

tests/loader/dependencies/elements/list-combine.bst → tests/format/dependencies/elements/list-combine.bst

 kind: stack
 description: This element depends on three elements in different ways
 build-depends:
-- elements/firstdep.bst
+- firstdep.bst
 runtime-depends:
-- elements/seconddep.bst
+- seconddep.bst
 depends:
-- elements/thirddep.bst
+- thirddep.bst

tests/loader/dependencies/elements/list-overlap.bst → tests/format/dependencies/elements/list-overlap.bst

 kind: stack
 description: This element depends on two elements in different ways
 build-depends:
-- elements/firstdep.bst
+- firstdep.bst
 depends:
-- filename: elements/firstdep.bst
+- filename: firstdep.bst
   type: runtime

tests/loader/dependencies/elements/runtimedep-list.bst → tests/format/dependencies/elements/runtimedep-list.bst

 kind: stack
 description: This element has a runtime-only dependency
 runtime-depends:
-  - elements/firstdep.bst
+  - firstdep.bst

tests/loader/dependencies/elements/runtimedep.bst → tests/format/dependencies/elements/runtimedep.bst

-kind: pony
+kind: manual
 description: This element has a runtime-only dependency
 depends:
-  - filename: elements/firstdep.bst
+  - filename: firstdep.bst
     type: runtime

tests/loader/dependencies/elements/shareddep.bst → tests/format/dependencies/elements/shareddep.bst

-kind: shareddep
+kind: stack
 description: This is the first dependency
 depends:
-- elements/firstdep.bst
+- firstdep.bst

tests/loader/dependencies/elements/shareddeptarget.bst → tests/format/dependencies/elements/shareddeptarget.bst

-kind: pony
+kind: stack
 description: This is the main target
 depends:
-- elements/firstdep.bst
-- elements/shareddep.bst
+- firstdep.bst
+- shareddep.bst

tests/loader/dependencies/elements/target-depdict.bst → tests/format/dependencies/elements/target-depdict.bst

-kind: pony
+kind: manual
 description: This is the main target
 depends:
-- filename: elements/firstdep.bst
+- filename: firstdep.bst

tests/loader/dependencies/elements/target.bst → tests/format/dependencies/elements/target.bst

-kind: pony
+kind: stack
 description: This is the main target
 depends:
-- elements/firstdep.bst
+- firstdep.bst