`vyos.vyos` is no longer deprecated! 🎉 (#348)

* Remove deprecation notice for vyos.vyos

Signed-off-by: NilashishC <nilashishchakraborty8@gmail.com>

* Add changelog

Signed-off-by: NilashishC <nilashishchakraborty8@gmail.com>

* fix review comments

Signed-off-by: NilashishC <nilashishchakraborty8@gmail.com>

* chore: auto fixes from pre-commit.com hooks

---------

Signed-off-by: NilashishC <nilashishchakraborty8@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

17 files changed, 61 insertions, 134 deletions

diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate.yaml
index 3dee476e..c5e2f4f7 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate.yaml
@@ -3,13 +3,14 @@
 
 - name: Setup
   vars:
-    lines:
-      "set interfaces ethernet eth1 firewall in name 'INBOUND' \nset interfaces\
-      \ ethernet eth1 firewall out name 'OUTBOUND' \nset interfaces ethernet eth1\
-      \ firewall local name 'LOCAL' \nset interfaces ethernet eth1 firewall local\
-      \ ipv6-name 'V6-LOCAL'\nset interfaces ethernet eth2 firewall in name 'INBOUND'\n\
-      set interfaces ethernet eth2 firewall out name 'OUTBOUND'\nset interfaces\
-      \ ethernet eth2 firewall local name 'LOCAL' \nset interfaces ethernet eth2\
-      \ firewall local ipv6-name 'V6-LOCAL'\n"
+    lines: |-
+      set interfaces ethernet eth1 firewall in name 'INBOUND'
+      set interfaces ethernet eth1 firewall out name 'OUTBOUND'
+      set interfaces ethernet eth1 firewall local name 'LOCAL'
+      set interfaces ethernet eth1 firewall local ipv6-name 'V6-LOCAL'
+      set interfaces ethernet eth2 firewall in name 'INBOUND'
+      set interfaces ethernet eth2 firewall out name 'OUTBOUND'
+      set interfaces ethernet eth2 firewall local name 'LOCAL'
+      set interfaces ethernet eth2 firewall local ipv6-name 'V6-LOCAL'
   ansible.netcommon.cli_config:
     config: "{{ lines }}"
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate_rule_sets.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate_rule_sets.yaml
index d2fd3ee7..b1e9425d 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate_rule_sets.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_populate_rule_sets.yaml
@@ -1,8 +1,6 @@
 ---
 - name: Setup
   vars:
-    lines:
-      "set firewall name 'INBOUND'\nset firewall name 'OUTBOUND'\nset firewall\
-      \ name 'LOCAL'\nset firewall ipv6-name 'V6-LOCAL'\n"
+    lines: "set firewall name 'INBOUND'\nset firewall name 'OUTBOUND'\nset firewall name 'LOCAL'\nset firewall ipv6-name 'V6-LOCAL'\n"
   ansible.netcommon.cli_config:
     config: "{{ lines }}"
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_config.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_config.yaml
index c95fc004..6074960a 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_config.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_config.yaml
@@ -1,8 +1,6 @@
 ---
 - name: Remove Config
   vars:
-    lines:
-      "delete interfaces ethernet eth1 firewall\ndelete interfaces ethernet\
-      \ eth2 firewall\n"
+    lines: "delete interfaces ethernet eth1 firewall\ndelete interfaces ethernet eth2 firewall\n"
   ansible.netcommon.cli_config:
     config: "{{ lines }}"
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_firewall_config.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_firewall_config.yaml
index da1b02ca..f77e6b74 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_firewall_config.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/_remove_firewall_config.yaml
@@ -1,7 +1,6 @@
 ---
 - name: Remove Config
   vars:
-    lines: "delete firewall name INBOUND\ndelete firewall name OUTBOUND\ndelete\
-      \ firewall name LOCAL\ndelete firewall ipv6-name V6-LOCAL\n"
+    lines: "delete firewall name INBOUND\ndelete firewall name OUTBOUND\ndelete firewall name LOCAL\ndelete firewall ipv6-name V6-LOCAL\n"
   ansible.netcommon.cli_config:
     config: "{{ lines }}"
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted.yaml
index eb9bded5..2b94c08b 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces deleted integration tests ansible_connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces deleted integration tests ansible_connection={{ ansible_connection }}
 
 - include_tasks: _populate_rule_sets.yaml
 
@@ -26,14 +24,12 @@
     - name: Assert that the correct set of commands were generated
       assert:
         that:
-          - "{{ deleted['commands'] | symmetric_difference(result['commands']) |length\
-            \ == 0 }}"
+          - "{{ deleted['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that the after dicts were correctly generated
       assert:
         that:
-          - "{{ deleted['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ deleted['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Delete attributes of given interfaces (IDEMPOTENT)
       register: result
@@ -48,8 +44,7 @@
     - name: Assert that the before dicts were correctly generated
       assert:
         that:
-          - "{{ deleted['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ deleted['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_afi.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_afi.yaml
index 8a31b2ac..065fcf22 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_afi.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_afi.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces deleted integration tests ansible_connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces deleted integration tests ansible_connection={{ ansible_connection }}
 
 - include_tasks: _populate_rule_sets.yaml
 
@@ -34,14 +32,12 @@
     - name: Assert that the correct set of commands were generated
       assert:
         that:
-          - "{{ deleted_afi['commands'] | symmetric_difference(result['commands'])\
-            \ |length == 0 }}"
+          - "{{ deleted_afi['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that the after dicts were correctly generated
       assert:
         that:
-          - "{{ deleted_afi['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ deleted_afi['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Delete attributes of given interfaces (IDEMPOTENT)
       register: result
@@ -56,8 +52,7 @@
     - name: Assert that the before dicts were correctly generated
       assert:
         that:
-          - "{{ deleted_afi['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ deleted_afi['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_all.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_all.yaml
index a3515b0f..f6cdae4e 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_all.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_all.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces deleted integration tests ansible_connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces deleted integration tests ansible_connection={{ ansible_connection }}
 
 - include_tasks: _populate_rule_sets.yaml
 
@@ -23,14 +21,12 @@
     - name: Assert that the correct set of commands were generated
       assert:
         that:
-          - "{{ deleted['commands'] | symmetric_difference(result['commands']) |length\
-            \ == 0 }}"
+          - "{{ deleted['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that the after dicts were correctly generated
       assert:
         that:
-          - "{{ deleted['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ deleted['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Delete attributes of given interfaces (IDEMPOTENT)
       register: result
@@ -45,8 +41,7 @@
     - name: Assert that the before dicts were correctly generated
       assert:
         that:
-          - "{{ deleted['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ deleted['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_single.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_single.yaml
index d62e27fd..6aa50324 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_single.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/deleted_single.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces deleted integration tests ansible_connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces deleted integration tests ansible_connection={{ ansible_connection }}
 
 - include_tasks: _populate_rule_sets.yaml
 
@@ -29,14 +27,12 @@
     - name: Assert that the correct set of commands were generated
       assert:
         that:
-          - "{{ deleted_single['commands'] | symmetric_difference(result['commands'])\
-            \ |length == 0 }}"
+          - "{{ deleted_single['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that the after dicts were correctly generated
       assert:
         that:
-          - "{{ deleted_single['after'] | symmetric_difference(result['after'])\
-            \ |length == 0 }}"
+          - "{{ deleted_single['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Delete attributes of given interfaces (IDEMPOTENT)
       register: result
@@ -51,8 +47,7 @@
     - name: Assert that the before dicts were correctly generated
       assert:
         that:
-          - "{{ deleted_single['after'] | symmetric_difference(result['before'])\
-            \ |length == 0 }}"
+          - "{{ deleted_single['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/empty_config.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/empty_config.yaml
index 1b9eb349..0799a676 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/empty_config.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/empty_config.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces empty_config integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces empty_config integration tests on connection={{ ansible_connection }}
 
 - name: Merged with empty config should give appropriate error message
   register: result
@@ -46,8 +44,7 @@
 
 - assert:
     that:
-      - result.msg == 'value of running_config parameter must not be empty for state
-        parsed'
+      - result.msg == 'value of running_config parameter must not be empty for state parsed'
 
 - name: Rendered with empty config should give appropriate error message
   register: result
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/gathered.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/gathered.yaml
index 2593a67d..145ad737 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/gathered.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/gathered.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces gathered integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces gathered integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -22,8 +20,7 @@
     - name: Assert that gathered dicts was correctly generated
       assert:
         that:
-          - "{{ populate | symmetric_difference(result['gathered']) |length == 0\
-            \ }}"
+          - "{{ populate | symmetric_difference(result['gathered']) |length == 0 }}"
 
     - name: Gather the existing running configuration (IDEMPOTENT)
       register: result
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged.yaml
index 10c8f01c..145fcefd 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces merged integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces merged integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _populate_rule_sets.yaml
 
@@ -52,25 +50,19 @@
 
     - name: Assert that before dicts were correctly generated
       assert:
-        that:
-          "{{ merged['before'] | symmetric_difference(result['before']) |length\
-          \ == 0 }}"
+        that: "{{ merged['before'] | symmetric_difference(result['before']) |length == 0 }}"
 
     - name: Assert that correct set of commands were generated
       assert:
         that:
-          - "{{ merged['commands'] | symmetric_difference(result['commands']) |length\
-            \ == 0 }}"
+          - "{{ merged['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that after dicts was correctly generated
       assert:
         that:
-          - "{{ merged['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ merged['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
-    - name:
-        Merge the provided configuration with the existing running configuration
-        (IDEMPOTENT)
+    - name: Merge the provided configuration with the existing running configuration (IDEMPOTENT)
       register: result
       vyos.vyos.vyos_firewall_interfaces: *id001
 
@@ -82,8 +74,7 @@
     - name: Assert that before dicts were correctly generated
       assert:
         that:
-          - "{{ merged['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ merged['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged_edit.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged_edit.yaml
index 1a53cabe..fbbb81ae 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged_edit.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/merged_edit.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces merged integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces merged integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -30,25 +28,19 @@
 
     - name: Assert that before dicts were correctly generated
       assert:
-        that:
-          "{{ populate | symmetric_difference(result['before']) |length == 0\
-          \ }}"
+        that: "{{ populate | symmetric_difference(result['before']) |length == 0 }}"
 
     - name: Assert that correct set of commands were generated
       assert:
         that:
-          - "{{ merged_edit['commands'] | symmetric_difference(result['commands'])\
-            \ |length == 0 }}"
+          - "{{ merged_edit['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that after dicts was correctly generated
       assert:
         that:
-          - "{{ merged_edit['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ merged_edit['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
-    - name:
-        Merge the provided configuration with the existing running configuration
-        (IDEMPOTENT)
+    - name: Merge the provided configuration with the existing running configuration (IDEMPOTENT)
       register: result
       vyos.vyos.vyos_firewall_interfaces: *id001
 
@@ -60,8 +52,7 @@
     - name: Assert that before dicts were correctly generated
       assert:
         that:
-          - "{{ merged_edit['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ merged_edit['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/overridden.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/overridden.yaml
index 5776718e..82508ba5 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/overridden.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/overridden.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces overridden integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces overridden integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -33,14 +31,12 @@
     - name: Assert that correct commands were generated
       assert:
         that:
-          - "{{ overridden['commands'] | symmetric_difference(result['commands'])\
-            \ |length == 0 }}"
+          - "{{ overridden['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that after dicts were correctly generated
       assert:
         that:
-          - "{{ overridden['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ overridden['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Overrides all device configuration with provided configurations (IDEMPOTENT)
       register: result
@@ -54,8 +50,7 @@
     - name: Assert that before dicts were correctly generated
       assert:
         that:
-          - "{{ overridden['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ overridden['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/parsed.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/parsed.yaml
index 105c647b..339e64ed 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/parsed.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/parsed.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces parsed integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces parsed integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -29,9 +27,7 @@
 
     - name: Assert that correct parsing done
       assert:
-        that:
-          "{{ ansible_facts['network_resources']['firewall_interfaces'] | symmetric_difference(result['parsed'])\
-          \ |length == 0 }}"
+        that: "{{ ansible_facts['network_resources']['firewall_interfaces'] | symmetric_difference(result['parsed']) |length == 0 }}"
 
     - name: Gather the existing running configuration (IDEMPOTENT)
       register: result
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rendered.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rendered.yaml
index 3283ec6d..3b815eb0 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rendered.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rendered.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces rendered integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces rendered integration tests on connection={{ ansible_connection }}
 
 - block:
     - name: Structure provided configuration into device specific commands
@@ -49,8 +47,7 @@
     - name: Assert that correct set of commands were generated
       assert:
         that:
-          - "{{ rendered['commands'] | symmetric_difference(result['rendered'])\
-            \ |length == 0 }}"
+          - "{{ rendered['commands'] | symmetric_difference(result['rendered']) |length == 0 }}"
 
     - name: Structure provided configuration into device specific commands (IDEMPOTENT)
       register: result
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/replaced.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/replaced.yaml
index d82a0f6a..bca14cfb 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/replaced.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/replaced.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces replaced integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces replaced integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -13,8 +11,7 @@
 - include_tasks: _populate.yaml
 
 - block:
-    - name: Replace device configurations of listed firewall rules with provided
-        configurations
+    - name: Replace device configurations of listed firewall rules with provided configurations
       register: result
       vyos.vyos.vyos_firewall_interfaces: &id001
         config:
@@ -41,8 +38,7 @@
     - name: Assert that correct set of commands were generated
       assert:
         that:
-          - "{{ replaced['commands'] | symmetric_difference(result['commands'])\
-            \ |length == 0 }}"
+          - "{{ replaced['commands'] | symmetric_difference(result['commands']) |length == 0 }}"
 
     - name: Assert that before dicts are correctly generated
       assert:
@@ -52,11 +48,9 @@
     - name: Assert that after dict is correctly generated
       assert:
         that:
-          - "{{ replaced['after'] | symmetric_difference(result['after']) |length\
-            \ == 0 }}"
+          - "{{ replaced['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
-    - name: Replace device configurations of listed firewall rules with provided
-        configurarions (IDEMPOTENT)
+    - name: Replace device configurations of listed firewall rules with provided configurarions (IDEMPOTENT)
       register: result
       vyos.vyos.vyos_firewall_interfaces: *id001
 
@@ -68,8 +62,7 @@
     - name: Assert that before dict is correctly generated
       assert:
         that:
-          - "{{ replaced['after'] | symmetric_difference(result['before']) |length\
-            \ == 0 }}"
+          - "{{ replaced['after'] | symmetric_difference(result['before']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml
 
diff --git a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rtt.yaml b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rtt.yaml
index ff3abb19..d64a987c 100644
--- a/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rtt.yaml
+++ b/tests/integration/targets/vyos_firewall_interfaces/tests/cli/rtt.yaml
@@ -1,8 +1,6 @@
 ---
 - debug:
-    msg:
-      START vyos_firewall_interfaces round trip integration tests on connection={{
-      ansible_connection }}
+    msg: START vyos_firewall_interfaces round trip integration tests on connection={{ ansible_connection }}
 
 - include_tasks: _remove_config.yaml
 
@@ -66,9 +64,7 @@
 
     - name: Assert that changes were applied
       assert:
-        that:
-          "{{ round_trip['after'] | symmetric_difference(result['after']) |length\
-          \ == 0 }}"
+        that: "{{ round_trip['after'] | symmetric_difference(result['after']) |length == 0 }}"
 
     - name: Revert back to base config using facts round trip
       register: revert
@@ -78,9 +74,7 @@
 
     - name: Assert that config was reverted
       assert:
-        that:
-          "{{ base_config['after'] | symmetric_difference(revert['after']) |length\
-          \ == 0 }}"
+        that: "{{ base_config['after'] | symmetric_difference(revert['after']) |length == 0 }}"
   always:
     - include_tasks: _remove_config.yaml