0 Fork 0

mirror of https://github.com/kevinveenbirkenbach/computer-playbook.git synced 2026-11-06 02:54:26 +00:00

Files

T

kevinveenbirkenbach a3402eac5b test(lint): group ansible lints into per-concern subdirs + add dotted-path test

Restructures tests/lint/ansible/ into five concern-keyed subdirectories
(jinja/, modules/, services/, shell/, sql/) plus the pre-existing roles/
and variables/. Each new subdir ships its own __init__.py exporting
PROJECT_ROOT and a scope-focused README.md following the project's
documentation conventions (RFC 2119 keywords, emoji-after-headline,
one-sentence-per-line, no historical context).

12 existing tests move into their concern bucket via git mv so history
follows the file. Adds tests/lint/ansible/variables/test_dotted_path_keys_exist.py
which catches the bug class where a Jinja dotted path's head resolves
via group_vars/all/ or the consuming role's own vars/defaults but a
deeper segment does not exist as a key in the dict tree (e.g. the
previous `LDAP.SERVER.REACH_DOMAIN` typo in web-app-odoo).

Touches three external references so cross-links keep resolving:
- docs/contributing/actions/testing/suppression.md (run-once, shared)
- roles/sys-svc-compose/tasks/utils/network/README.md (network-via-util)
- tests/lint/ansible/sql/test_no_inline_multiline_sql.py (self-path exclude)

2026-05-14 17:12:56 +02:00

590 B

Raw Permalink Blame History

SQL Lints 💾

Lint tests that enforce safe inline-SQL practices inside Ansible task definitions.

Tests in this directory MUST only cover SQL-shape rules inside Ansible tasks.

test_no_inline_multiline_sql.py caps inline multi-line SQL in Ansible tasks at three lines. Larger statements MUST live in a dedicated .sql file under the role's files/ directory and be loaded explicitly so linters, formatters, and database tooling can operate on them.

For framework, directory layout, and make test-lint usage see lint.md.

590 B Raw Permalink Blame History

SQL Lints 💾

590 B

Raw Permalink Blame History