[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
ansible-collections · Mar 27, 2024 · ddd53d9 · ddd53d9
1 parent 9d23577
commit ddd53d9
Show file tree

Hide file tree

Showing 36 changed files with 182 additions and 211 deletions.
diff --git a/tests/unit/mock/path.py b/tests/unit/mock/path.py
@@ -8,5 +8,6 @@
 
 
 mock_unfrackpath_noop = MagicMock(
-    spec_set=unfrackpath, side_effect=lambda x, *args, **kwargs: x
+    spec_set=unfrackpath,
+    side_effect=lambda x, *args, **kwargs: x,
 )
diff --git a/tests/unit/modules/network/ios/test_ios_acl_interfaces.py b/tests/unit/modules/network/ios/test_ios_acl_interfaces.py
@@ -7,11 +7,10 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 from textwrap import dedent
+from unittest.mock import patch
 
 from ansible_collections.cisco.ios.plugins.modules import ios_acl_interfaces
-
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 
 from .ios_module import TestIosModule
@@ -27,9 +26,7 @@ def setUp(self):
             "ansible_collections.ansible.netcommon.plugins.module_utils.network.common.rm_base.resource_module_base."
             "get_resource_connection",
         )
-        self.get_resource_connection_facts = (
-            self.mock_get_resource_connection_facts.start()
-        )
+        self.get_resource_connection_facts = self.mock_get_resource_connection_facts.start()
 
         self.mock_execute_show_command = patch(
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.acl_interfaces.acl_interfaces."

diff --git a/tests/unit/modules/network/ios/test_ios_acls.py b/tests/unit/modules/network/ios/test_ios_acls.py
@@ -7,11 +7,10 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 from textwrap import dedent
+from unittest.mock import patch
 
 from ansible_collections.cisco.ios.plugins.modules import ios_acls
-
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 
 from .ios_module import TestIosModule
@@ -27,9 +26,7 @@ def setUp(self):
             "ansible_collections.ansible.netcommon.plugins.module_utils.network.common.rm_base.resource_module_base."
             "get_resource_connection",
         )
-        self.get_resource_connection_facts = (
-            self.mock_get_resource_connection_facts.start()
-        )
+        self.get_resource_connection_facts = self.mock_get_resource_connection_facts.start()
 
         self.mock_execute_show_command = patch(
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.acls.acls."
@@ -40,9 +37,7 @@ def setUp(self):
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.acls.acls."
             "AclsFacts.get_acl_names",
         )
-        self.execute_show_command_name = (
-            self.mock_execute_show_command_name_specific.start()
-        )
+        self.execute_show_command_name = self.mock_execute_show_command_name_specific.start()
 
     def tearDown(self):
         super(TestIosAclsModule, self).tearDown()

diff --git a/tests/unit/modules/network/ios/test_ios_banner.py b/tests/unit/modules/network/ios/test_ios_banner.py
@@ -19,8 +19,8 @@
 
 __metaclass__ = type
 from unittest.mock import patch
-from ansible_collections.cisco.ios.plugins.modules import ios_banner
 
+from ansible_collections.cisco.ios.plugins.modules import ios_banner
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 
 from .ios_module import TestIosModule, load_fixture

diff --git a/tests/unit/modules/network/ios/test_ios_bgp_address_family.py b/tests/unit/modules/network/ios/test_ios_bgp_address_family.py
@@ -7,11 +7,10 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 from textwrap import dedent
+from unittest.mock import patch
 
 from ansible_collections.cisco.ios.plugins.modules import ios_bgp_address_family
-
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 
 from .ios_module import TestIosModule
@@ -27,9 +26,7 @@ def setUp(self):
             "ansible_collections.ansible.netcommon.plugins.module_utils.network.common.rm_base.resource_module_base."
             "get_resource_connection",
         )
-        self.get_resource_connection_facts = (
-            self.mock_get_resource_connection_facts.start()
-        )
+        self.get_resource_connection_facts = self.mock_get_resource_connection_facts.start()
 
         self.mock_execute_show_command = patch(
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.bgp_address_family.bgp_address_family."
@@ -122,10 +119,10 @@ def test_ios_bgp_address_family_merged(self):
                                     neighbor_address="198.51.100.1",
                                     remote_as="65.11",
                                     route_maps=[
-                                        dict(name="test-route-out", out="true")
+                                        dict(name="test-route-out", out="true"),
                                     ],
                                     prefix_lists=[
-                                        dict(name="AS65100-PREFIX-OUT", out="true")
+                                        dict(name="AS65100-PREFIX-OUT", out="true"),
                                     ],
                                 ),
                             ],
@@ -135,7 +132,7 @@ def test_ios_bgp_address_family_merged(self):
                             bgp=dict(aggregate_timer=20, dmzlink_bw=True, scan_time=10),
                             default_metric=10,
                             networks=[
-                                dict(address="192.0.1.1", route_map="test_route")
+                                dict(address="192.0.1.1", route_map="test_route"),
                             ],
                         ),
                     ],
@@ -336,13 +333,13 @@ def test_ios_bgp_address_family_merged_idempotent(self):
                                     "neighbor_address": "198.51.100.1",
                                     "nexthop_self": {"all": True},
                                     "prefix_lists": [
-                                        {"name": "AS65100-PREFIX-OUT", "out": True}
+                                        {"name": "AS65100-PREFIX-OUT", "out": True},
                                     ],
                                     "remote_as": 10,
                                     "route_maps": [{"name": "test-out", "out": True}],
                                     "route_server_client": True,
                                     "slow_peer_options": {
-                                        "detection": {"threshold": 150}
+                                        "detection": {"threshold": 150},
                                     },
                                 },
                             ],
@@ -442,7 +439,7 @@ def test_ios_bgp_address_family_replaced(self):
                                     activate=True,
                                     remote_as=200,
                                     route_maps=[
-                                        dict(name="test-replaced-route", out=True)
+                                        dict(name="test-replaced-route", out=True),
                                     ],
                                 ),
                             ],
@@ -552,14 +549,15 @@ def test_ios_bgp_address_family_replaced_idempotent(self):
                                             cost_community=dict(
                                                 id=100,
                                                 poi=dict(
-                                                    igp_cost=True, transitive=True
+                                                    igp_cost=True,
+                                                    transitive=True,
                                                 ),
                                             ),
                                         ),
                                     ),
                                     nexthop_self=dict(all=True),
                                     prefix_lists=[
-                                        dict(name="AS65100-PREFIX-OUT", out="true")
+                                        dict(name="AS65100-PREFIX-OUT", out="true"),
                                     ],
                                     slow_peer=[dict(detection=dict(threshold=150))],
                                     remote_as=10,
@@ -731,14 +729,15 @@ def test_ios_bgp_address_family_overridden_idempotent(self):
                                             cost_community=dict(
                                                 id=100,
                                                 poi=dict(
-                                                    igp_cost=True, transitive=True
+                                                    igp_cost=True,
+                                                    transitive=True,
                                                 ),
                                             ),
                                         ),
                                     ),
                                     nexthop_self=dict(all=True),
                                     prefix_lists=[
-                                        dict(name="AS65100-PREFIX-OUT", out="true")
+                                        dict(name="AS65100-PREFIX-OUT", out="true"),
                                     ],
                                     slow_peer=[dict(detection=dict(threshold=150))],
                                     remote_as=10,
@@ -963,7 +962,8 @@ def test_ios_bgp_address_family_rendered(self):
                                             cost_community=dict(
                                                 id=100,
                                                 poi=dict(
-                                                    igp_cost=True, transitive=True
+                                                    igp_cost=True,
+                                                    transitive=True,
                                                 ),
                                             ),
                                         ),
@@ -1209,7 +1209,7 @@ def test_ios_bgp_address_family_parsed(self):
                             },
                             "route_server_client": True,
                             "prefix_lists": [
-                                {"name": "AS65100-PREFIX-OUT", "out": True}
+                                {"name": "AS65100-PREFIX-OUT", "out": True},
                             ],
                             "slow_peer_options": {"detection": {"threshold": 150}},
                             "route_maps": [{"name": "test-out", "out": True}],

diff --git a/tests/unit/modules/network/ios/test_ios_bgp_global.py b/tests/unit/modules/network/ios/test_ios_bgp_global.py
@@ -7,14 +7,11 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 from textwrap import dedent
+from unittest.mock import patch
 
 from ansible_collections.cisco.ios.plugins.modules import ios_bgp_global
-from ansible_collections.cisco.ios.tests.unit.modules.utils import (
-    AnsibleFailJson,
-    set_module_args,
-)
+from ansible_collections.cisco.ios.tests.unit.modules.utils import AnsibleFailJson, set_module_args
 
 from .ios_module import TestIosModule
 
@@ -29,9 +26,7 @@ def setUp(self):
             "ansible_collections.ansible.netcommon.plugins.module_utils.network.common.rm_base.resource_module_base."
             "get_resource_connection",
         )
-        self.get_resource_connection_facts = (
-            self.mock_get_resource_connection_facts.start()
-        )
+        self.get_resource_connection_facts = self.mock_get_resource_connection_facts.start()
 
         self.mock_execute_show_command = patch(
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.bgp_global.bgp_global."

diff --git a/tests/unit/modules/network/ios/test_ios_command.py b/tests/unit/modules/network/ios/test_ios_command.py
@@ -20,9 +20,10 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 import json
 
+from unittest.mock import patch
+
 from ansible_collections.cisco.ios.plugins.modules import ios_command
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 

diff --git a/tests/unit/modules/network/ios/test_ios_config.py b/tests/unit/modules/network/ios/test_ios_config.py
@@ -22,9 +22,9 @@
 
 __metaclass__ = type
 from unittest.mock import MagicMock, patch
+
 from ansible_collections.cisco.ios.plugins.cliconf.ios import Cliconf
 from ansible_collections.cisco.ios.plugins.modules import ios_config
-
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
 
 from .ios_module import TestIosModule, load_fixture
@@ -124,7 +124,8 @@ def test_ios_config_lines_wo_parents(self):
         set_module_args(dict(lines=lines))
         self.conn.get_diff = MagicMock(
             return_value=self.cliconf_obj.get_diff(
-                "\n".join(lines), self.running_config
+                "\n".join(lines),
+                self.running_config,
             ),
         )
         commands = ["hostname foo"]
@@ -140,7 +141,8 @@ def test_ios_config_lines_w_parents(self):
 
         self.conn.get_diff = MagicMock(
             return_value=self.cliconf_obj.get_diff(
-                candidate_config, self.running_config
+                candidate_config,
+                self.running_config,
             ),
         )
 
@@ -152,7 +154,8 @@ def test_ios_config_before(self):
         set_module_args(dict(lines=lines, before=["test1", "test2"]))
         self.conn.get_diff = MagicMock(
             return_value=self.cliconf_obj.get_diff(
-                "\n".join(lines), self.running_config
+                "\n".join(lines),
+                self.running_config,
             ),
         )
         commands = ["test1", "test2", "hostname foo"]
@@ -163,7 +166,8 @@ def test_ios_config_after(self):
         set_module_args(dict(lines=lines, after=["test1", "test2"]))
         self.conn.get_diff = MagicMock(
             return_value=self.cliconf_obj.get_diff(
-                "\n".join(lines), self.running_config
+                "\n".join(lines),
+                self.running_config,
             ),
         )
         commands = ["hostname foo", "test1", "test2"]
@@ -172,11 +176,12 @@ def test_ios_config_after(self):
     def test_ios_config_before_after_no_change(self):
         lines = ["hostname router"]
         set_module_args(
-            dict(lines=lines, before=["test1", "test2"], after=["test3", "test4"])
+            dict(lines=lines, before=["test1", "test2"], after=["test3", "test4"]),
         )
         self.conn.get_diff = MagicMock(
             return_value=self.cliconf_obj.get_diff(
-                "\n".join(lines), self.running_config
+                "\n".join(lines),
+                self.running_config,
             ),
         )
         self.execute_module()

diff --git a/tests/unit/modules/network/ios/test_ios_evpn_global.py b/tests/unit/modules/network/ios/test_ios_evpn_global.py
@@ -7,8 +7,8 @@
 
 
 __metaclass__ = type
-from unittest.mock import patch
 from textwrap import dedent
+from unittest.mock import patch
 
 from ansible_collections.cisco.ios.plugins.modules import ios_evpn_global
 from ansible_collections.cisco.ios.tests.unit.modules.utils import set_module_args
@@ -26,9 +26,7 @@ def setUp(self):
             "ansible_collections.ansible.netcommon.plugins.module_utils.network.common.rm_base.resource_module_base."
             "get_resource_connection",
         )
-        self.get_resource_connection_facts = (
-            self.mock_get_resource_connection_facts.start()
-        )
+        self.get_resource_connection_facts = self.mock_get_resource_connection_facts.start()
 
         self.mock_execute_show_command = patch(
             "ansible_collections.cisco.ios.plugins.module_utils.network.ios.facts.evpn_global.evpn_global."