Files
computer-playbook/tests/integration/roles/meta/test_presence.py
kevinveenbirkenbachandClaude Opus 5 5137465c6f style: delete every comment the comments-valid policy rejects
3529 invalid comments in 528 files, grandfathered in while the linter
only saw the working-tree diff. Removal ran through the linter's own
collectors, so detection and deletion cannot drift: a comment whose
opener line is blank before it and whose closer line is blank after it
loses all its lines, anything else loses only the comment region, and a
line left holding a bare delimiter goes with it.

Five files keep their comments behind a nocheck marker instead. Four are
upstream configs whose inline docs are upstream's own (mautrix, systemd
journald, nextcloud-oidc-login, MediaWiki). In littlejs the
commented-out entries ARE the deactivation mechanism for an example
tile, so deleting them would delete configuration.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-18 03:22:02 +02:00

37 lines
955 B
Python

import os
import unittest
from pathlib import Path
from utils.roles.mapping import ROLE_FILE_META_MAIN
ROLES_DIR = str(
Path(str(Path(str(Path(__file__).parent)) / "../../../../roles")).resolve()
)
class TestRolesHaveMetaMain(unittest.TestCase):
def test_each_role_has_meta_main(self):
missing_meta = []
for role in os.listdir(ROLES_DIR):
if role.startswith(".") or role == "__pycache__":
continue
role_path = str(Path(ROLES_DIR) / role)
if not Path(role_path).is_dir():
continue
meta_main = str(Path(role_path) / ROLE_FILE_META_MAIN)
if not Path(meta_main).is_file():
missing_meta.append(role)
if missing_meta:
self.fail(
"The following roles are missing meta/main.yml:\n"
+ "\n".join(missing_meta)
)
if __name__ == "__main__":
unittest.main()