Skip to content

Commit

Permalink
[pre-commit.ci] auto fixes from pre-commit.com hooks
Browse files Browse the repository at this point in the history
for more information, see https://pre-commit.ci
  • Loading branch information
pre-commit-ci[bot] committed Jan 3, 2025
1 parent 503ed1e commit 3641e99
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 27 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,4 +4,3 @@ bugfixes:
- Cleaned up unit tests that were passing for the wrong reasons. The updated tests now ensure the right config sections are verified for VLAN configurations.
- Added a test to validate the gathered state for VLAN configuration context, improving reliability.
- Made improvements to ensure VLAN facts are gathered properly, both for specific configurations and general VLAN settings.

52 changes: 26 additions & 26 deletions tests/unit/modules/network/ios/test_ios_vlans.py
Original file line number Diff line number Diff line change
Expand Up @@ -511,32 +511,32 @@ def test_ios_vlans_overridden(self):
)
result = self.execute_module(changed=True)
commands = [
'no vlan 1',
'no vlan 150',
'no vlan 888',
'no vlan 1002',
'no vlan 1003',
'no vlan 1004',
'no vlan 1005',
'vlan 200',
'name test_vlan_200',
'state active',
'remote-span',
'no shutdown',
'vlan 123',
'name Override_RemoteIsInMyName',
'no state active',
'no mtu 610',
'remote-span',
'shutdown',
'no vlan configuration 1',
'no vlan configuration 150',
'no vlan configuration 888',
'no vlan configuration 1002',
'no vlan configuration 1003',
'no vlan configuration 1004',
'no vlan configuration 1005'
]
"no vlan 1",
"no vlan 150",
"no vlan 888",
"no vlan 1002",
"no vlan 1003",
"no vlan 1004",
"no vlan 1005",
"vlan 200",
"name test_vlan_200",
"state active",
"remote-span",
"no shutdown",
"vlan 123",
"name Override_RemoteIsInMyName",
"no state active",
"no mtu 610",
"remote-span",
"shutdown",
"no vlan configuration 1",
"no vlan configuration 150",
"no vlan configuration 888",
"no vlan configuration 1002",
"no vlan configuration 1003",
"no vlan configuration 1004",
"no vlan configuration 1005",
]

self.assertEqual(result["commands"], commands)

Expand Down

0 comments on commit 3641e99

Please sign in to comment.