diff options
author | Benjamin Schubert <ben.c.schubert@gmail.com> | 2019-02-28 19:35:18 +0000 |
---|---|---|
committer | bst-marge-bot <marge-bot@buildstream.build> | 2019-03-01 19:05:30 +0000 |
commit | 402cc7af5464f34d29909148dfb0876e56f97124 (patch) | |
tree | 2004c4888e528db25172ecb116d23713d3469873 /tests/format/variables.py | |
parent | d6d33c94751bd47ca77d2b2e4ac246a2dd1328a6 (diff) | |
download | buildstream-402cc7af5464f34d29909148dfb0876e56f97124.tar.gz |
tests: Remove unused parameters in functions
For parameters that are required as part of an API, prefix them
by "_" to make it clear they are unused
Diffstat (limited to 'tests/format/variables.py')
-rw-r--r-- | tests/format/variables.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/format/variables.py b/tests/format/variables.py index 5a0d21206..3fa0f4790 100644 --- a/tests/format/variables.py +++ b/tests/format/variables.py @@ -37,7 +37,7 @@ def print_warning(msg): ('qmake.bst', 'make-install', "make -j1 INSTALL_ROOT=\"/buildstream-install\" install"), ]) @pytest.mark.datafiles(os.path.join(DATA_DIR, 'defaults')) -def test_defaults(cli, datafiles, tmpdir, target, varname, expected): +def test_defaults(cli, datafiles, target, varname, expected): project = os.path.join(datafiles.dirname, datafiles.basename) result = cli.run(project=project, silent=True, args=[ 'show', '--deps', 'none', '--format', '%{vars}', target @@ -63,7 +63,7 @@ def test_defaults(cli, datafiles, tmpdir, target, varname, expected): ('qmake.bst', 'make-install', "make -j1 INSTALL_ROOT=\"/custom/install/root\" install"), ]) @pytest.mark.datafiles(os.path.join(DATA_DIR, 'overrides')) -def test_overrides(cli, datafiles, tmpdir, target, varname, expected): +def test_overrides(cli, datafiles, target, varname, expected): project = os.path.join(datafiles.dirname, datafiles.basename) result = cli.run(project=project, silent=True, args=[ 'show', '--deps', 'none', '--format', '%{vars}', target @@ -74,7 +74,7 @@ def test_overrides(cli, datafiles, tmpdir, target, varname, expected): @pytest.mark.datafiles(os.path.join(DATA_DIR, 'missing_variables')) -def test_missing_variable(cli, datafiles, tmpdir): +def test_missing_variable(cli, datafiles): project = os.path.join(datafiles.dirname, datafiles.basename) result = cli.run(project=project, silent=True, args=[ 'show', '--deps', 'none', '--format', '%{config}', 'manual.bst' @@ -97,7 +97,7 @@ def test_cyclic_variables(cli, datafiles): @pytest.mark.parametrize('protected_var', PROTECTED_VARIABLES) @pytest.mark.datafiles(os.path.join(DATA_DIR, 'protected-vars')) -def test_use_of_protected_var_project_conf(cli, tmpdir, datafiles, protected_var): +def test_use_of_protected_var_project_conf(cli, datafiles, protected_var): project = str(datafiles) conf = { 'name': 'test', @@ -125,7 +125,7 @@ def test_use_of_protected_var_project_conf(cli, tmpdir, datafiles, protected_var @pytest.mark.parametrize('protected_var', PROTECTED_VARIABLES) @pytest.mark.datafiles(os.path.join(DATA_DIR, 'protected-vars')) -def test_use_of_protected_var_element_overrides(cli, tmpdir, datafiles, protected_var): +def test_use_of_protected_var_element_overrides(cli, datafiles, protected_var): project = str(datafiles) conf = { 'name': 'test', @@ -157,7 +157,7 @@ def test_use_of_protected_var_element_overrides(cli, tmpdir, datafiles, protecte @pytest.mark.parametrize('protected_var', PROTECTED_VARIABLES) @pytest.mark.datafiles(os.path.join(DATA_DIR, 'protected-vars')) -def test_use_of_protected_var_in_element(cli, tmpdir, datafiles, protected_var): +def test_use_of_protected_var_in_element(cli, datafiles, protected_var): project = str(datafiles) element = { 'kind': 'import', |