mirror of
https://github.com/ansible-collections/community.general.git
synced 2026-06-15 04:17:41 +00:00
tests: let ruff tidy test_ini_file.py (CI was being picky)
This commit is contained in:
parent
fde6b13a7f
commit
099f91519d
1 changed files with 6 additions and 10 deletions
|
|
@ -56,6 +56,7 @@ def test_no_ignore_spaces_unchanged():
|
|||
|
||||
class FakeModule:
|
||||
"""Minimal AnsibleModule stub for do_ini unit tests."""
|
||||
|
||||
_diff = False
|
||||
check_mode = False
|
||||
tmpdir = tempfile.gettempdir()
|
||||
|
|
@ -73,10 +74,12 @@ class FakeModule:
|
|||
@pytest.fixture()
|
||||
def ini_file_path(tmp_path):
|
||||
"""Return a factory that writes content to a temp ini file and yields the path."""
|
||||
|
||||
def _make(content):
|
||||
p = tmp_path / "test.ini"
|
||||
p.write_text(content)
|
||||
return str(p)
|
||||
|
||||
return _make
|
||||
|
||||
|
||||
|
|
@ -119,12 +122,8 @@ def test_ini_file_doc_comment_lines_not_deleted_gh11919(ini_file_path):
|
|||
|
||||
result = open(path).read()
|
||||
|
||||
assert "; output_buffering\n" in result, (
|
||||
"Doc comment '; output_buffering' must not be deleted"
|
||||
)
|
||||
assert "output_buffering = 123\n" in result, (
|
||||
"Active config 'output_buffering = 123' must be present"
|
||||
)
|
||||
assert "; output_buffering\n" in result, "Doc comment '; output_buffering' must not be deleted"
|
||||
assert "output_buffering = 123\n" in result, "Active config 'output_buffering = 123' must be present"
|
||||
assert ";output_buffering = 4096\n" not in result, (
|
||||
"Old commented config ';output_buffering = 4096' must be replaced"
|
||||
)
|
||||
|
|
@ -133,10 +132,7 @@ def test_ini_file_doc_comment_lines_not_deleted_gh11919(ini_file_path):
|
|||
def test_ini_file_commented_config_replaced_not_duplicated_gh11919(ini_file_path):
|
||||
"""Companion to gh11919: the commented config line must be replaced in-place,
|
||||
not left behind alongside the new active line."""
|
||||
content = (
|
||||
"[PHP]\n"
|
||||
";output_buffering = 4096\n"
|
||||
)
|
||||
content = "[PHP]\n;output_buffering = 4096\n"
|
||||
path = ini_file_path(content)
|
||||
|
||||
ini_file.do_ini(
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue