mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 10:59:59 +00:00
kevinveenbirkenbach
a3402eac5b
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)
726 B
726 B
Jinja Lints 🪄
Lint tests that enforce correct Jinja2 filter and lookup composition in Ansible templates and task definitions.
Tests in this directory MUST only cover Jinja2 expression composition rules.
Rules that validate where dotted-path targets are declared live in the sibling ../variables/ directory.
test_no_lookup_config_jinja_default.pyforbidslookup('config', application_id, 'path') | default(X). The two-termconfiglookup raises on a missing key, so the Jinjadefault(...)filter never fires; the fallback MUST be passed as the lookup's third argument instead.
For framework, directory layout, and make test-lint usage see lint.md.