Fix playbook/rulebook detection

.github/workflows/tox.yml

@@ -70,7 +70,7 @@ jobs:
       WSLENV: FORCE_COLOR:PYTEST_REQPASS:TOXENV:GITHUB_STEP_SUMMARY
       # Number of expected test passes, safety measure for accidental skip of
       # tests. Update value if you add/remove tests.
-      PYTEST_REQPASS: 781
+      PYTEST_REQPASS: 782
 
     steps:
       - name: Activate WSL1

src/ansiblelint/file_utils.py

@@ -242,7 +242,7 @@ def __init__(
         self.abspath = self.path.expanduser().absolute()
 
         if self.kind == "yaml":
-            self._guess_kind()
+            self.data  # pylint: disable=pointless-statement
 
     def _guess_kind(self) -> None:
         if self.kind == "yaml":
@@ -371,6 +371,11 @@ def data(self) -> Any:
                     )
 
                     self._data = parse_yaml_linenumbers(self)
+                    # now that _data is not empty, we can try guessing if playbook or rulebook
+                    # it has to be done before append_skipped_rules() call as it's relying
+                    # on self.kind.
+                    if self.kind == "yaml":
+                        self._guess_kind()
                     # Lazy import to avoid delays and cyclic-imports
                     if "append_skipped_rules" not in globals():
                         # pylint: disable=import-outside-toplevel

src/ansiblelint/testing/__init__.py

@@ -45,11 +45,11 @@ def run(self, filename: str) -> list[MatchError]:
         """Lints received filename."""
         return self._call_runner(filename)
 
-    def run_playbook(self, playbook_text: str) -> list[MatchError]:
+    def run_playbook(
+        self, playbook_text: str, prefix: str = "playbook"
+    ) -> list[MatchError]:
         """Lints received text as a playbook."""
-        with tempfile.NamedTemporaryFile(
-            mode="w", suffix=".yml", prefix="playbook"
-        ) as fh:
+        with tempfile.NamedTemporaryFile(mode="w", suffix=".yml", prefix=prefix) as fh:
             fh.write(playbook_text)
             fh.flush()
             results = self._call_runner(fh.name)

test/test_skiputils.py

@@ -53,6 +53,13 @@ def test_playbook_noqa(default_text_runner: RunFromText) -> None:
     assert len(results) == 1
 
 
+def test_playbook_noqa2(default_text_runner: RunFromText) -> None:
+    """Check that noqa is properly taken into account on vars and tasks."""
+    results = default_text_runner.run_playbook(PLAYBOOK_WITH_NOQA, "test")
+    # Should raise error at "SOME_VAR".
+    assert len(results) == 1
+
+
 @pytest.mark.parametrize(
     ("lintable", "yaml", "expected_form"),
     (