Fix iosxr copy and delete tasks (#44083)

* Fix iosxr copy and delete tasks

* Drop local from non-smoke tasks

test/integration/targets/iosxr_banner/tasks/cli.yaml

@@ -14,11 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_command/tasks/cli.yaml

@@ -14,9 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_command/tests/cli/cli_command.yaml

@@ -23,31 +23,23 @@
       - "result.failed == true"
       - "result.msg is defined"
 
-- name: Remove existing file if any
+- name: delete config file on disk to prevent failure of copy task for duplicate
   cli_command:
     command: 'delete harddisk:ansible_tmp.txt'
-    prompt: 'Delete harddisk\:ansible_tmp\.txt\[confirm\]'
-    answer: "\r\n"
+    prompt: 'Delete harddisk\:/?ansible_tmp\.txt\[confirm\]'
+    answer: ""
   ignore_errors: yes
 
-- block:
-    - name: Run command with prompt
-      cli_command:
-        command: 'copy running-config harddisk:ansible_tmp.txt'
-        prompt: 'Destination file name \(control-c to abort\)\: \[\/ansible_tmp.txt\]\?'
-        answer: 'ansible_tmp.txt'
-      register: result
+- name: Run command with prompt
+  cli_command:
+    command: 'copy running-config harddisk:ansible_tmp.txt'
+    prompt: 'Destination file name \(control-c to abort\)\: \[\/ansible_tmp.txt\]\?'
+    answer: 'ansible_tmp.txt'
+  register: result
 
-    - assert:
-        that:
-          - "result.stdout is defined"
-          - "'ansible_tmp' in result.stdout"
-  always:
-    - name: Remove copied file
-      cli_command:
-        command: 'delete harddisk:ansible_tmp.txt'
-        prompt: 'Delete harddisk\:ansible_tmp\.txt\[confirm\]'
-        answer: "\r\n"
-      ignore_errors: yes
+- assert:
+    that:
+      - "result.stdout is defined"
+      - "'ansible_tmp' in result.stdout"
 
 - debug: msg="END cli/cli_command.yaml on connection={{ ansible_connection }}"

test/integration/targets/iosxr_command/tests/cli/prompt.yaml

@@ -1,12 +1,13 @@
 ---
-- debug: msg="START cli/prompt.yaml on connection={{ ansible_connection }}"
+- debug:
+    msg: "START cli/prompt.yaml on connection={{ ansible_connection }}"
 
 - name: delete config file on disk to prevent failure of copy task for duplicate
   iosxr_command:
     commands:
         - command: 'delete harddisk:ansible_tmp.txt'
-          prompt: 'Delete harddisk\:ansible_tmp\.txt\[confirm\]'
-          answer: "\r\n"
+          prompt: 'Delete harddisk\:/?ansible_tmp\.txt\[confirm\]'
+          answer: ""
   ignore_errors: yes
 
 - name: copy

test/integration/targets/iosxr_config/tasks/cli.yaml

@@ -14,9 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_facts/tasks/cli.yaml

@@ -14,9 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_interface/tasks/cli.yaml

@@ -14,11 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_logging/tasks/cli.yaml

@@ -15,13 +15,5 @@
   loop_control:
     loop_var: test_case_to_run
 
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run
-
 - name: reset connection
   meta: reset_connection

test/integration/targets/iosxr_netconf/tasks/cli.yaml

@@ -14,9 +14,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run

test/integration/targets/iosxr_smoke/tasks/cli.yaml

@@ -15,10 +15,8 @@
   loop_control:
     loop_var: test_case_to_run
 
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
 - name: run test case (connection=local)
   include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
+  with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run

test/integration/targets/iosxr_smoke/tasks/netconf.yaml

@@ -15,10 +15,8 @@
   loop_control:
     loop_var: test_case_to_run
 
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
 - name: run test case (connection=local)
   include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
+  with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run

test/integration/targets/iosxr_user/tasks/cli.yaml

@@ -25,11 +25,3 @@
   with_items: "{{ test_items }}"
   loop_control:
     loop_var: test_case_to_run
-
-# Only one of the Testcase would be run to check if `connection: local`
-# is established. Full suite is run with `connection:network_cli` or `connection:netconf`
-- name: run test case (connection=local)
-  include: "{{ test_case_to_run }} ansible_connection=local"
-  with_first_found: "{{ test_items }}"
-  loop_control:
-    loop_var: test_case_to_run