Skip to content

Commit

Permalink
Remove py27 compat files & update imports for patch & magicmock (#480)
Browse files Browse the repository at this point in the history
* remove compat files & imports

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

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
  • Loading branch information
Ruchip16 and pre-commit-ci[bot] authored Mar 27, 2024
1 parent 2429e30 commit cbb648c
Show file tree
Hide file tree
Showing 42 changed files with 70 additions and 215 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,5 +45,5 @@ jobs:
'${{ needs.sanity.result }}',
'${{ needs.unit-galaxy.result }}',
'${{ needs.ansible-lint.result }}',
'${{ needs.unit-source.outputs.result }}'
'${{ needs.unit-source.result }}'
])"
4 changes: 4 additions & 0 deletions changelogs/fragments/trivial_tests_updates.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
---
trivial:
- remove compat files as we stopped testing against python 2.7.
- update direct imports for patch and MagicMock from unittest.
Empty file removed tests/unit/compat/__init__.py
Empty file.
129 changes: 0 additions & 129 deletions tests/unit/compat/mock.py

This file was deleted.

41 changes: 0 additions & 41 deletions tests/unit/compat/unittest.py

This file was deleted.

4 changes: 2 additions & 2 deletions tests/unit/mock/path.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@


__metaclass__ = type
from ansible.utils.path import unfrackpath
from unittest.mock import MagicMock

from ansible_collections.cisco.iosxr.tests.unit.compat.mock import MagicMock
from ansible.utils.path import unfrackpath


mock_unfrackpath_noop = MagicMock(
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/modules/network/iosxr/test_hostname.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
__metaclass__ = type

from textwrap import dedent
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_hostname
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule
Expand Down
4 changes: 2 additions & 2 deletions tests/unit/modules/network/iosxr/test_iosxr.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,15 +22,15 @@
__metaclass__ = type

from os import path
from unittest import TestCase

from ansible.module_utils._text import to_bytes, to_text
from mock import MagicMock

from ansible_collections.cisco.iosxr.plugins.cliconf import iosxr
from ansible_collections.cisco.iosxr.tests.unit.compat import unittest


class TestPluginCLIConfIOSXR(unittest.TestCase):
class TestPluginCLIConfIOSXR(TestCase):
"""Test class for IOSXR CLI Conf Methods"""

def setUp(self):
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_acl_interfaces
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_acls.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_acls
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_banner.py
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,9 @@


__metaclass__ = type
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_banner
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
__metaclass__ = type

from textwrap import dedent
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_bgp_address_family
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_bgp_global.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
__metaclass__ = type

from textwrap import dedent
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_bgp_global
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
__metaclass__ = type

from textwrap import dedent
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_bgp_neighbor_address_family
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,9 +23,9 @@
__metaclass__ = type

from textwrap import dedent
from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_bgp_templates
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_command.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_command
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
2 changes: 1 addition & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@


__metaclass__ = type
from unittest.mock import MagicMock, patch

from ansible_collections.cisco.iosxr.plugins.cliconf.iosxr import Cliconf
from ansible_collections.cisco.iosxr.plugins.modules import iosxr_config
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import MagicMock, patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_facts.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,9 @@

import json

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_facts
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_interfaces
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_l2_interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_l2_interfaces
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_l3_interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_l3_interfaces
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
3 changes: 2 additions & 1 deletion tests/unit/modules/network/iosxr/test_iosxr_lacp.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_lacp
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,9 @@

__metaclass__ = type

from unittest.mock import patch

from ansible_collections.cisco.iosxr.plugins.modules import iosxr_lacp_interfaces
from ansible_collections.cisco.iosxr.tests.unit.compat.mock import patch
from ansible_collections.cisco.iosxr.tests.unit.modules.utils import set_module_args

from .iosxr_module import TestIosxrModule, load_fixture
Expand Down
Loading

0 comments on commit cbb648c

Please sign in to comment.