1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
|
"""Test cli arguments and config."""
from __future__ import annotations
import os
from pathlib import Path
import pytest
from _pytest.monkeypatch import MonkeyPatch
from ansiblelint import cli
@pytest.fixture(name="base_arguments")
def fixture_base_arguments() -> list[str]:
"""Define reusable base arguments for tests in current module."""
return ["../test/skiptasks.yml"]
@pytest.mark.parametrize(
("args", "config"),
(
(["-p"], "test/fixtures/parseable.yml"),
(["-q"], "test/fixtures/quiet.yml"),
(["-r", "test/fixtures/rules/"], "test/fixtures/rulesdir.yml"),
(["-R", "-r", "test/fixtures/rules/"], "test/fixtures/rulesdir-defaults.yml"),
(["-s"], "test/fixtures/strict.yml"),
(["-t", "skip_ansible_lint"], "test/fixtures/tags.yml"),
(["-v"], "test/fixtures/verbosity.yml"),
(["-x", "bad_tag"], "test/fixtures/skip-tags.yml"),
(["--exclude", "test/"], "test/fixtures/exclude-paths.yml"),
(["--show-relpath"], "test/fixtures/show-abspath.yml"),
([], "test/fixtures/show-relpath.yml"),
),
)
def test_ensure_config_are_equal(
base_arguments: list[str], args: list[str], config: str
) -> None:
"""Check equality of the CLI options to config files."""
command = base_arguments + args
cli_parser = cli.get_cli_parser()
options = cli_parser.parse_args(command)
file_config = cli.load_config(config)
for key, val in file_config.items():
# config_file does not make sense in file_config
if key == "config_file":
continue
if key in {"exclude_paths", "rulesdir"}:
val = [Path(p) for p in val]
assert val == getattr(options, key)
@pytest.mark.parametrize(
("with_base", "args", "config"),
(
(True, ["--write"], "test/fixtures/config-with-write-all.yml"),
(True, ["--write=all"], "test/fixtures/config-with-write-all.yml"),
(True, ["--write", "all"], "test/fixtures/config-with-write-all.yml"),
(True, ["--write=none"], "test/fixtures/config-with-write-none.yml"),
(True, ["--write", "none"], "test/fixtures/config-with-write-none.yml"),
(
True,
["--write=rule-tag,rule-id"],
"test/fixtures/config-with-write-subset.yml",
),
(
True,
["--write", "rule-tag,rule-id"],
"test/fixtures/config-with-write-subset.yml",
),
(
True,
["--write", "rule-tag", "--write", "rule-id"],
"test/fixtures/config-with-write-subset.yml",
),
(
False,
["--write", "examples/playbooks/example.yml"],
"test/fixtures/config-with-write-all.yml",
),
(
False,
["--write", "examples/playbooks/example.yml", "non-existent.yml"],
"test/fixtures/config-with-write-all.yml",
),
),
)
def test_ensure_write_cli_does_not_consume_lintables(
base_arguments: list[str], with_base: bool, args: list[str], config: str
) -> None:
"""Check equality of the CLI --write options to config files."""
cli_parser = cli.get_cli_parser()
command = base_arguments + args if with_base else args
options = cli_parser.parse_args(command)
file_config = cli.load_config(config)
file_value = file_config.get("write_list")
orig_cli_value = getattr(options, "write_list")
cli_value = cli.WriteArgAction.merge_write_list_config(
from_file=[], from_cli=orig_cli_value
)
assert file_value == cli_value
def test_config_can_be_overridden(base_arguments: list[str]) -> None:
"""Check that config can be overridden from CLI."""
no_override = cli.get_config(base_arguments + ["-t", "bad_tag"])
overridden = cli.get_config(
base_arguments + ["-t", "bad_tag", "-c", "test/fixtures/tags.yml"]
)
assert no_override.tags + ["skip_ansible_lint"] == overridden.tags
def test_different_config_file(base_arguments: list[str]) -> None:
"""Ensures an alternate config_file can be used."""
diff_config = cli.get_config(
base_arguments + ["-c", "test/fixtures/ansible-config.yml"]
)
no_config = cli.get_config(base_arguments + ["-v"])
assert diff_config.verbosity == no_config.verbosity
def test_expand_path_user_and_vars_config_file(base_arguments: list[str]) -> None:
"""Ensure user and vars are expanded when specified as exclude_paths."""
config1 = cli.get_config(
base_arguments + ["-c", "test/fixtures/exclude-paths-with-expands.yml"]
)
config2 = cli.get_config(
base_arguments
+ ["--exclude", "~/.ansible/roles", "--exclude", "$HOME/.ansible/roles"]
)
assert str(config1.exclude_paths[0]) == os.path.expanduser("~/.ansible/roles")
assert str(config2.exclude_paths[0]) == os.path.expanduser("~/.ansible/roles")
assert str(config1.exclude_paths[1]) == os.path.expandvars("$HOME/.ansible/roles")
assert str(config2.exclude_paths[1]) == os.path.expandvars("$HOME/.ansible/roles")
def test_path_from_config_do_not_depend_on_cwd(
monkeypatch: MonkeyPatch,
) -> None: # Issue 572
"""Check that config-provided paths are decoupled from CWD."""
config1 = cli.load_config("test/fixtures/config-with-relative-path.yml")
monkeypatch.chdir("test")
config2 = cli.load_config("fixtures/config-with-relative-path.yml")
assert config1["exclude_paths"].sort() == config2["exclude_paths"].sort()
def test_path_from_cli_depend_on_cwd(
base_arguments: list[str], monkeypatch: MonkeyPatch
) -> None:
"""Check that CLI-provided paths are relative to CWD."""
# Issue 572
arguments = base_arguments + [
"--exclude",
"test/fixtures/config-with-relative-path.yml",
]
options1 = cli.get_cli_parser().parse_args(arguments)
assert "test/test" not in str(options1.exclude_paths[0])
test_dir = "test"
monkeypatch.chdir(test_dir)
options2 = cli.get_cli_parser().parse_args(arguments)
assert "test/test" in str(options2.exclude_paths[0])
@pytest.mark.parametrize(
"config_file",
(
pytest.param("test/fixtures/ansible-config-invalid.yml", id="invalid"),
pytest.param("/dev/null/ansible-config-missing.yml", id="missing"),
),
)
def test_config_failure(base_arguments: list[str], config_file: str) -> None:
"""Ensures specific config files produce error code 3."""
with pytest.raises(SystemExit, match="^3$"):
cli.get_config(base_arguments + ["-c", config_file])
def test_extra_vars_loaded(base_arguments: list[str]) -> None:
"""Ensure ``extra_vars`` option is loaded from file config."""
config = cli.get_config(
base_arguments + ["-c", "test/fixtures/config-with-extra-vars.yml"]
)
assert config.extra_vars == {"foo": "bar", "knights_favorite_word": "NI"}
|
