Merge pull request #91 from tyll/name_other_provider_tasks

Tests: Add names for other provider tasks/playbooks

tests/ensure_non_running_provider.py

@@ -18,13 +18,16 @@ IGNORE = ["tests_unit.yml", "tests_helpers-and-asserts.yml"]
 OTHER_PLAYBOOK = """
 # SPDX-License-Identifier: BSD-3-Clause
 ---
-- hosts: all
+- name: Run playbook '{tests_playbook}' with non-default provider
+  hosts: all
   vars:
       network_provider_current:
   tasks:
   # required for the code to set network_provider_current
-  - service_facts:
-  - set_fact:
+  - name: Get service facts
+    service_facts:
+  - name: Set network provider
+    set_fact:
       network_provider: "{{{{ 'initscripts' if network_provider_current == 'nm' else 'nm' }}}}"
 
 - import_playbook: "{tests_playbook}"

tests/tests_bridge_other_provider.yml

@@ -1,7 +1,10 @@
 - hosts: all
+  name: Run playbook 'tests_bridge.yml' with non-default provider
   tasks:
-  - service_facts: null
-  - set_fact:
+  - name: Get service facts
+    service_facts: null
+  - name: Set network provider
+    set_fact:
       network_provider: '{{ ''initscripts'' if network_provider_current == ''nm''
         else ''nm'' }}'
   vars:

tests/tests_default_other_provider.yml

@@ -1,7 +1,10 @@
 - hosts: all
+  name: Run playbook 'tests_default.yml' with non-default provider
   tasks:
-  - service_facts: null
-  - set_fact:
+  - name: Get service facts
+    service_facts: null
+  - name: Set network provider
+    set_fact:
       network_provider: '{{ ''initscripts'' if network_provider_current == ''nm''
         else ''nm'' }}'
   vars:

tests/tests_ethernet_other_provider.yml

@@ -1,7 +1,10 @@
 - hosts: all
+  name: Run playbook 'tests_ethernet.yml' with non-default provider
   tasks:
-  - service_facts: null
-  - set_fact:
+  - name: Get service facts
+    service_facts: null
+  - name: Set network provider
+    set_fact:
       network_provider: '{{ ''initscripts'' if network_provider_current == ''nm''
         else ''nm'' }}'
   vars: