Low-hanging unit fixes (#515)

plugins/sub_plugins/cli_parser/pyats_parser.py

@@ -42,7 +42,6 @@
 
 from ansible.module_utils._text import to_native
 from ansible.module_utils.basic import missing_required_lib
-from ansible.module_utils.six import PY3
 from ansible_collections.ansible.utils.plugins.plugin_utils.base.cli_parser import (
     CliParserBase,
 )
@@ -77,8 +76,6 @@ def _check_reqs():
         :return dict: A dict with a list of errors
         """
         errors = []
-        if not PY3:
-            errors.append("Pyats and Genie require Python 3")
         if not HAS_GENIE:
             errors.append(missing_required_lib("genie"))
         if not HAS_PYATS:

tests/unit/module_utils/network/common/test_parsing.py

@@ -31,7 +31,3 @@ def test_key_value_instance_variable_assignment(self):
 
     def test_conditionals_w_not_keyword(self):
         assert c1(test_results) and c2(test_results) and c3(test_results)
-
-
-if __name__ == "__main__":
-    unittest.main()

tests/unit/plugins/connection/test_netconf.py

@@ -8,7 +8,6 @@
 
 __metaclass__ = type
 
-import sys
 from unittest.mock import MagicMock, PropertyMock, patch
 
 import pytest
@@ -28,19 +27,11 @@ def import_mock(name, *args):
     return builtin_import(name, *args)
 
 
-PY3 = sys.version_info[0] == 3
-if PY3:
-    with patch("builtins.__import__", side_effect=import_mock):
-        from ansible.plugins.loader import connection_loader
-        from ansible_collections.ansible.netcommon.plugins.connection import (
-            netconf,
-        )
-else:
-    with patch("__builtin__.__import__", side_effect=import_mock):
-        from ansible.plugins.loader import connection_loader
-        from ansible_collections.ansible.netcommon.plugins.connection import (
-            netconf,
-        )
+with patch("builtins.__import__", side_effect=import_mock):
+    from ansible.plugins.loader import connection_loader
+    from ansible_collections.ansible.netcommon.plugins.connection import (
+        netconf,
+    )
 
 
 def test_netconf_init():

tests/unit/plugins/plugin_utils/test_version.py

@@ -0,0 +1,31 @@
+# (c) 2023 Ansible Project
+# GNU General Public License v3.0+ (see LICENSES/GPL-3.0-or-later.txt or https://www.gnu.org/licenses/gpl-3.0.txt)
+# SPDX-License-Identifier: GPL-3.0-or-later
+
+from __future__ import absolute_import, division, print_function
+
+__metaclass__ = type
+
+import pytest
+from ansible_collections.ansible.netcommon.plugins.plugin_utils.version import (
+    Version,
+)
+
+
+@pytest.mark.parametrize("left", ["6.0.0", 6, 6.0])
+@pytest.mark.parametrize("right", ["4.0.0", 4, 4.0])
+def test_versions_different(left, right):
+    assert Version(str(left)) > right
+
+
+@pytest.mark.parametrize("value", ["6.0.0", 6, 6.0])
+def test_versions_same(value):
+    assert Version(str(value)) == value
+
+
+def test_version_error():
+    with pytest.raises(TypeError):
+        Version("1.2.3") < [1, 2, 3]
+
+    # with pytest.raises(TypeError):
+    Version("1.2.3") == [1, 2, 3]