Refactored import_playbook tests

test/TestImportPlaybook.py

@@ -0,0 +1,17 @@
+"""Test ability to import playbooks."""
+from ansiblelint.runner import Runner
+
+
+def test_task_hook_import_playbook(default_rules_collection):
+    """Assures import_playbook includes are recognized."""
+    playbook_path = 'test/playbook-import/playbook_parent.yml'
+    runner = Runner(default_rules_collection, playbook_path, [], [], [])
+    results = runner.run()
+
+    results_text = str(results)
+    assert len(runner.playbooks) == 2
+    assert len(results) == 2
+    # Assures we detected the issues from imported playbook
+    assert 'Commands should not change things' in results_text
+    assert '502' in results_text
+    assert 'All tasks should be named' in results_text

test/playbook-include/playbook_included.yml → test/playbook-import/playbook_imported.yml

@@ -3,7 +3,7 @@
   connection: local
   gather_facts: no
   tasks:
-  - command: echo "no name"  # noqa 502
+  - command: echo "no name"  # should generate 502
   - name: Another task
     debug:
       msg: debug message

test/playbook-import/playbook_parent.yml

@@ -0,0 +1,3 @@
+---
+- name: Importing another playbook
+  import_playbook: playbook_imported.yml