Adding upstream version 6.13.1.upstream/6.13.1upstream

Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>

1 files changed, 195 insertions, 0 deletions

diff --git a/test/test_commandline_invocations_same_as_config.py b/test/test_commandline_invocations_same_as_config.py
new file mode 100644
index 0000000..eecc0d9
--- /dev/null
+++ b/test/test_commandline_invocations_same_as_config.py
@@ -0,0 +1,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"}