From 5f041cc26741a63ec98439bd47b8bf469a85645a Mon Sep 17 00:00:00 2001 From: Lie Ryan Date: Thu, 14 Mar 2024 22:07:44 +1100 Subject: [PATCH] refactor: fix linter issues --- tests/test_core.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/test_core.py b/tests/test_core.py index b60d24a..89d177e 100644 --- a/tests/test_core.py +++ b/tests/test_core.py @@ -1,11 +1,9 @@ # type: ignore -from invoke import MockContext - -import yaml -from pytest import fixture from textwrap import dedent -import gitops.utils.kube as kube +from invoke import MockContext +from pytest import fixture + import gitops.utils.apps as apps from gitops import core @@ -14,10 +12,12 @@ def mock_invoke_run(monkeypatch): monkeypatch.setattr(core, "run", lambda cmd: None) + @fixture def confirm_yes(monkeypatch): monkeypatch.setattr(apps, "confirm", lambda: True) + @fixture def sample_app(tmp_path, monkeypatch): monkeypatch.setattr(apps, "get_apps_directory", lambda: tmp_path) @@ -50,7 +50,7 @@ def test_setenv_should_not_overwrite_if_there_is_existing_variable( deployment_yml, confirm_yes, ): - file = deployment_yml( + file = deployment_yml( # noqa: F841 """\ namespace: test chart: test @@ -112,7 +112,7 @@ def test_setenv_should_not_overwrite_if_only_ordering_has_changed( deployment_yml, confirm_yes, ): - file = deployment_yml( + file = deployment_yml( # noqa: F841 """\ namespace: test chart: test @@ -155,7 +155,7 @@ def test_unset_nonexistent_environment_variable( deployment_yml, confirm_yes, ): - file = deployment_yml( + file = deployment_yml( # noqa: F841 """\ namespace: test chart: test @@ -211,7 +211,7 @@ def test_preserve_empty_environment_variable( deployment_yml, confirm_yes, ): - file = deployment_yml( + file = deployment_yml( # noqa: F841 """\ namespace: test chart: test @@ -226,7 +226,7 @@ def test_preserve_null_environment_variable( deployment_yml, confirm_yes, ): - file = deployment_yml( + file = deployment_yml( # noqa: F841 """\ namespace: test chart: test