Avoid false positives for handler in roles handlers directory (#3838)

Co-authored-by: Sorin Sbarnea <ssbarnea@redhat.com>

.github/workflows/tox.yml

@@ -70,7 +70,7 @@ jobs:
     env:
       # Number of expected test passes, safety measure for accidental skip of
       # tests. Update value if you add/remove tests.
-      PYTEST_REQPASS: 833
+      PYTEST_REQPASS: 834
     steps:
       - uses: actions/checkout@v4
         with:

examples/roles/role_with_handler/handlers/main.yml

@@ -0,0 +1,8 @@
+---
+- name: Debug
+  loop: "{{ _something_done.results }}"
+  loop_control:
+    label: "{{ item.item.name }}"
+  when: item.changed
+  ansible.builtin.debug:
+    msg: "{{ item.item.name }} changed"

examples/roles/role_with_handler/tasks/main.yml

@@ -0,0 +1,17 @@
+---
+- name: Get info
+  delegate_to: localhost
+  register: collected_info
+  ansible.builtin.debug:
+    msg: test
+
+- name: Do something
+  delegate_to: localhost
+  loop: "{{ collected_info['some_list'] }}"
+  loop_control:
+    label: "{{ item.name }}"
+  notify:
+    - Debug
+  register: _something_done
+  ansible.builtin.debug:
+    msg: test2

src/ansiblelint/rules/no_handler.py

@@ -89,6 +89,7 @@ def matchtask(
     # pylint: disable=ungrouped-imports
     from ansiblelint.rules import RulesCollection
     from ansiblelint.runner import Runner
+    from ansiblelint.testing import run_ansible_lint
 
     @pytest.mark.parametrize(
         ("test_file", "failures"),
@@ -107,3 +108,10 @@ def test_no_handler(
         assert len(results) == failures
         for result in results:
             assert result.tag == "no-handler"
+
+    def test_role_with_handler() -> None:
+        """Test role with handler."""
+        role_path = "examples/roles/role_with_handler"
+
+        results = run_ansible_lint("-v", role_path)
+        assert "no-handler" not in results.stdout

src/ansiblelint/utils.py

@@ -755,7 +755,12 @@ def skip_tags(self) -> list[str]:
 
     def is_handler(self) -> bool:
         """Return true for tasks that are handlers."""
-        return ".handlers[" in self.position
+        is_handler_file = False
+        if isinstance(self._normalized_task, dict):
+            file_name = str(self._normalized_task["action"].get(FILENAME_KEY, None))
+            if file_name:
+                is_handler_file = "handlers" in str(file_name)
+        return is_handler_file if is_handler_file else ".handlers[" in self.position
 
     def __repr__(self) -> str:
         """Return a string representation of the task."""