diff options
author | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-02-12 11:33:49 +0100 |
---|---|---|
committer | Cédric Ollivier <cedric.ollivier@orange.com> | 2018-02-13 07:41:27 +0100 |
commit | 9e9e8c9b89fc7c419d1824190217897118800275 (patch) | |
tree | 246e617ac78afbd09aa8bf928556291bf2f0a6b3 /functest/tests | |
parent | b1522c34d95f9bb48fc9cfca91f281da2f958953 (diff) |
Simplify functest/cli/commands/cli_env.py
CONST shouldn't be used for getting/setting env vars.
It adds complexity and may raise side effects.
It also removes the obsolete env var CI_DEBUG.
Change-Id: I8a8fde0fa781351d5eabd2698ca8aae9dee76fb8
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/tests')
-rw-r--r-- | functest/tests/unit/cli/commands/test_cli_env.py | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/functest/tests/unit/cli/commands/test_cli_env.py b/functest/tests/unit/cli/commands/test_cli_env.py index 88377234..d17da166 100644 --- a/functest/tests/unit/cli/commands/test_cli_env.py +++ b/functest/tests/unit/cli/commands/test_cli_env.py @@ -48,9 +48,6 @@ class CliEnvTesting(unittest.TestCase): elif var == 'BUILD_TAG': os.environ['BUILD_TAG'] = '' reg_string = r"| BUILD TAG: None|" - elif var == 'DEBUG': - os.environ['CI_DEBUG'] = '' - reg_string = r"| DEBUG FLAG: false\s*|" with mock.patch('functest.cli.commands.cli_env.click.echo') \ as mock_click_echo: @@ -72,9 +69,6 @@ class CliEnvTesting(unittest.TestCase): def test_show_missing_ci_build_tag(self, *args): self._test_show_missing_env_var('BUILD_TAG', *args) - def test_show_missing_ci_debug(self, *args): - self._test_show_missing_env_var('DEBUG', *args) - if __name__ == "__main__": logging.disable(logging.CRITICAL) |