Fixes needed for sanity (#40)

Fixes needed for sanity

Reviewed-by: https://github.com/apps/ansible-zuul

6 files changed, 67 insertions, 130 deletions

diff --git a/tests/sanity/ignore-2.10.txt b/tests/sanity/ignore-2.10.txt
deleted file mode 100644
index a62f497..0000000
--- a/tests/sanity/ignore-2.10.txt
+++ /dev/null
@@ -1,83 +0,0 @@
-plugins/module_utils/network/vyos/vyos.py future-import-boilerplate
-plugins/module_utils/network/vyos/vyos.py metaclass-boilerplate
-plugins/modules/vyos_banner.py future-import-boilerplate
-plugins/modules/vyos_banner.py metaclass-boilerplate
-plugins/modules/vyos_banner.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_banner.py validate-modules:doc-missing-type
-plugins/modules/vyos_banner.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_command.py future-import-boilerplate
-plugins/modules/vyos_command.py metaclass-boilerplate
-plugins/modules/vyos_command.py pylint:blacklisted-name
-plugins/modules/vyos_command.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_command.py validate-modules:doc-missing-type
-plugins/modules/vyos_command.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_command.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_command.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_config.py future-import-boilerplate
-plugins/modules/vyos_config.py metaclass-boilerplate
-plugins/modules/vyos_config.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_config.py validate-modules:doc-missing-type
-plugins/modules/vyos_config.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_config.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_config.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_facts.py future-import-boilerplate
-plugins/modules/vyos_facts.py metaclass-boilerplate
-plugins/modules/vyos_facts.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_facts.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_facts.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_facts.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lag_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lag_interfaces.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_lldp_global.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_lldp_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lldp_interfaces.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_logging.py future-import-boilerplate
-plugins/modules/vyos_logging.py metaclass-boilerplate
-plugins/modules/vyos_logging.py validate-modules:doc-choices-do-not-match-spec
-plugins/modules/vyos_logging.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_logging.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_logging.py validate-modules:doc-missing-type
-plugins/modules/vyos_logging.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_logging.py validate-modules:missing-suboption-docs
-plugins/modules/vyos_logging.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_logging.py validate-modules:undocumented-parameter
-plugins/modules/vyos_ping.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_ping.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_ping.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_system.py future-import-boilerplate
-plugins/modules/vyos_system.py metaclass-boilerplate
-plugins/modules/vyos_system.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_system.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_system.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_system.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_user.py future-import-boilerplate
-plugins/modules/vyos_user.py metaclass-boilerplate
-plugins/modules/vyos_user.py validate-modules:doc-choices-do-not-match-spec
-plugins/modules/vyos_user.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_user.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_user.py validate-modules:doc-missing-type
-plugins/modules/vyos_user.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_user.py validate-modules:missing-suboption-docs
-plugins/modules/vyos_user.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_user.py validate-modules:undocumented-parameter
-plugins/modules/vyos_vlan.py validate-modules:doc-choices-do-not-match-spec
-plugins/modules/vyos_vlan.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_vlan.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_vlan.py validate-modules:doc-missing-type
-plugins/modules/vyos_vlan.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_vlan.py validate-modules:missing-suboption-docs
-plugins/modules/vyos_vlan.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_vlan.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_vlan.py validate-modules:undocumented-parameter
-plugins/action/vyos.py action-plugin-docs # base class for deprecated network platform modules using `connection: local`
-plugins/doc_fragments/vyos.py future-import-boilerplate
-plugins/doc_fragments/vyos.py metaclass-boilerplate
-tests/unit/mock/path.py future-import-boilerplate
-tests/unit/mock/path.py metaclass-boilerplate
-tests/unit/mock/yaml_helper.py future-import-boilerplate
-tests/unit/mock/yaml_helper.py metaclass-boilerplate
-tests/unit/modules/conftest.py future-import-boilerplate
-tests/unit/modules/conftest.py metaclass-boilerplate
-tests/unit/modules/utils.py future-import-boilerplate
-tests/unit/modules/utils.py metaclass-boilerplate
\ No newline at end of file
diff --git a/tests/sanity/ignore-2.9.txt b/tests/sanity/ignore-2.9.txt
index a62f497..68c5fc2 100644
--- a/tests/sanity/ignore-2.9.txt
+++ b/tests/sanity/ignore-2.9.txt
@@ -1,78 +1,100 @@
-plugins/module_utils/network/vyos/vyos.py future-import-boilerplate
-plugins/module_utils/network/vyos/vyos.py metaclass-boilerplate
+plugins/action/vyos.py action-plugin-docs # base class for deprecated network platform modules using `connection: local`
+plugins/doc_fragments/vyos.py future-import-boilerplate
+plugins/doc_fragments/vyos.py metaclass-boilerplate
 plugins/modules/vyos_banner.py future-import-boilerplate
 plugins/modules/vyos_banner.py metaclass-boilerplate
 plugins/modules/vyos_banner.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/vyos_banner.py validate-modules:doc-missing-type
-plugins/modules/vyos_banner.py validate-modules:doc-required-mismatch
 plugins/modules/vyos_command.py future-import-boilerplate
 plugins/modules/vyos_command.py metaclass-boilerplate
 plugins/modules/vyos_command.py pylint:blacklisted-name
 plugins/modules/vyos_command.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/vyos_command.py validate-modules:doc-missing-type
-plugins/modules/vyos_command.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_command.py validate-modules:parameter-list-no-elements
 plugins/modules/vyos_command.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_config.py future-import-boilerplate
 plugins/modules/vyos_config.py metaclass-boilerplate
 plugins/modules/vyos_config.py validate-modules:doc-default-does-not-match-spec
 plugins/modules/vyos_config.py validate-modules:doc-missing-type
-plugins/modules/vyos_config.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_config.py validate-modules:parameter-list-no-elements
 plugins/modules/vyos_config.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_facts.py future-import-boilerplate
 plugins/modules/vyos_facts.py metaclass-boilerplate
 plugins/modules/vyos_facts.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_facts.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_facts.py validate-modules:parameter-list-no-elements
 plugins/modules/vyos_facts.py validate-modules:parameter-type-not-in-doc
-plugins/modules/vyos_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lag_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lag_interfaces.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_lldp_global.py validate-modules:parameter-list-no-elements
-plugins/modules/vyos_lldp_interfaces.py validate-modules:doc-elements-mismatch
-plugins/modules/vyos_lldp_interfaces.py validate-modules:doc-required-mismatch
+plugins/modules/vyos_firewall_global.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_interface.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_interface.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_interface.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_interface.py validate-modules:doc-missing-type
+plugins/modules/vyos_interface.py validate-modules:invalid-documentation
+plugins/modules/vyos_interface.py validate-modules:missing-suboption-docs
+plugins/modules/vyos_interface.py validate-modules:parameter-type-not-in-doc
+plugins/modules/vyos_interface.py validate-modules:undocumented-parameter
+plugins/modules/vyos_l3_interface.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_l3_interface.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_l3_interface.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_l3_interface.py validate-modules:doc-missing-type
+plugins/modules/vyos_l3_interface.py validate-modules:invalid-documentation
+plugins/modules/vyos_l3_interface.py validate-modules:missing-suboption-docs
+plugins/modules/vyos_l3_interface.py validate-modules:parameter-type-not-in-doc
+plugins/modules/vyos_l3_interface.py validate-modules:undocumented-parameter
+plugins/modules/vyos_linkagg.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_linkagg.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_linkagg.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_linkagg.py validate-modules:doc-missing-type
+plugins/modules/vyos_linkagg.py validate-modules:invalid-documentation
+plugins/modules/vyos_linkagg.py validate-modules:missing-suboption-docs
+plugins/modules/vyos_linkagg.py validate-modules:parameter-type-not-in-doc
+plugins/modules/vyos_linkagg.py validate-modules:undocumented-parameter
+plugins/modules/vyos_lldp_interface.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_lldp_interface.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_lldp_interface.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_lldp_interface.py validate-modules:doc-missing-type
+plugins/modules/vyos_lldp_interface.py validate-modules:invalid-documentation
+plugins/modules/vyos_lldp_interface.py validate-modules:missing-suboption-docs
+plugins/modules/vyos_lldp_interface.py validate-modules:undocumented-parameter
+plugins/modules/vyos_lldp.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_lldp.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_lldp.py validate-modules:invalid-documentation
 plugins/modules/vyos_logging.py future-import-boilerplate
 plugins/modules/vyos_logging.py metaclass-boilerplate
 plugins/modules/vyos_logging.py validate-modules:doc-choices-do-not-match-spec
 plugins/modules/vyos_logging.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_logging.py validate-modules:doc-elements-mismatch
 plugins/modules/vyos_logging.py validate-modules:doc-missing-type
-plugins/modules/vyos_logging.py validate-modules:doc-required-mismatch
 plugins/modules/vyos_logging.py validate-modules:missing-suboption-docs
 plugins/modules/vyos_logging.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_logging.py validate-modules:undocumented-parameter
+plugins/modules/vyos_ospfv2.py validate-modules:invalid-documentation
+plugins/modules/vyos_ospfv3.py validate-modules:invalid-documentation
 plugins/modules/vyos_ping.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_ping.py validate-modules:doc-required-mismatch
 plugins/modules/vyos_ping.py validate-modules:parameter-type-not-in-doc
+plugins/modules/vyos_static_route.py validate-modules:deprecation-mismatch
+plugins/modules/vyos_static_route.py validate-modules:doc-choices-do-not-match-spec
+plugins/modules/vyos_static_route.py validate-modules:doc-default-does-not-match-spec
+plugins/modules/vyos_static_route.py validate-modules:doc-missing-type
+plugins/modules/vyos_static_route.py validate-modules:invalid-documentation
+plugins/modules/vyos_static_route.py validate-modules:missing-suboption-docs
+plugins/modules/vyos_static_route.py validate-modules:parameter-type-not-in-doc
+plugins/modules/vyos_static_route.py validate-modules:undocumented-parameter
 plugins/modules/vyos_system.py future-import-boilerplate
 plugins/modules/vyos_system.py metaclass-boilerplate
 plugins/modules/vyos_system.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_system.py validate-modules:doc-required-mismatch
-plugins/modules/vyos_system.py validate-modules:parameter-list-no-elements
 plugins/modules/vyos_system.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_user.py future-import-boilerplate
 plugins/modules/vyos_user.py metaclass-boilerplate
 plugins/modules/vyos_user.py validate-modules:doc-choices-do-not-match-spec
 plugins/modules/vyos_user.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_user.py validate-modules:doc-elements-mismatch
 plugins/modules/vyos_user.py validate-modules:doc-missing-type
-plugins/modules/vyos_user.py validate-modules:doc-required-mismatch
 plugins/modules/vyos_user.py validate-modules:missing-suboption-docs
 plugins/modules/vyos_user.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_user.py validate-modules:undocumented-parameter
 plugins/modules/vyos_vlan.py validate-modules:doc-choices-do-not-match-spec
 plugins/modules/vyos_vlan.py validate-modules:doc-default-does-not-match-spec
-plugins/modules/vyos_vlan.py validate-modules:doc-elements-mismatch
 plugins/modules/vyos_vlan.py validate-modules:doc-missing-type
-plugins/modules/vyos_vlan.py validate-modules:doc-required-mismatch
 plugins/modules/vyos_vlan.py validate-modules:missing-suboption-docs
-plugins/modules/vyos_vlan.py validate-modules:parameter-list-no-elements
 plugins/modules/vyos_vlan.py validate-modules:parameter-type-not-in-doc
 plugins/modules/vyos_vlan.py validate-modules:undocumented-parameter
-plugins/action/vyos.py action-plugin-docs # base class for deprecated network platform modules using `connection: local`
-plugins/doc_fragments/vyos.py future-import-boilerplate
-plugins/doc_fragments/vyos.py metaclass-boilerplate
+plugins/module_utils/network/vyos/vyos.py future-import-boilerplate
+plugins/module_utils/network/vyos/vyos.py metaclass-boilerplate
 tests/unit/mock/path.py future-import-boilerplate
 tests/unit/mock/path.py metaclass-boilerplate
 tests/unit/mock/yaml_helper.py future-import-boilerplate
@@ -80,4 +102,4 @@ tests/unit/mock/yaml_helper.py metaclass-boilerplate
 tests/unit/modules/conftest.py future-import-boilerplate
 tests/unit/modules/conftest.py metaclass-boilerplate
 tests/unit/modules/utils.py future-import-boilerplate
-tests/unit/modules/utils.py metaclass-boilerplate
\ No newline at end of file
+tests/unit/modules/utils.py metaclass-boilerplate
diff --git a/tests/unit/modules/network/vyos/test_vyos_firewall_interfaces.py b/tests/unit/modules/network/vyos/test_vyos_firewall_interfaces.py
index 92d2a4f..2a9a81c 100644
--- a/tests/unit/modules/network/vyos/test_vyos_firewall_interfaces.py
+++ b/tests/unit/modules/network/vyos/test_vyos_firewall_interfaces.py
@@ -303,7 +303,7 @@ class TestVyosFirewallInterfacesModule(TestVyosModule):
                         access_rules=[
                             dict(
                                 afi="ipv4",
-                                rules=[dict(name="INBOUND", direction="in"),],
+                                rules=[dict(name="INBOUND", direction="in")],
                             ),
                             dict(
                                 afi="ipv6",
diff --git a/tests/unit/modules/network/vyos/test_vyos_firewall_rules.py b/tests/unit/modules/network/vyos/test_vyos_firewall_rules.py
index 86fcc65..8d59e19 100644
--- a/tests/unit/modules/network/vyos/test_vyos_firewall_rules.py
+++ b/tests/unit/modules/network/vyos/test_vyos_firewall_rules.py
@@ -686,9 +686,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
     def test_vyos_firewall_v4_rule_sets_del_01(self):
         set_module_args(
             dict(
-                config=[
-                    dict(afi="ipv4", rule_sets=[dict(name="V4-INGRESS"),])
-                ],
+                config=[dict(afi="ipv4", rule_sets=[dict(name="V4-INGRESS")])],
                 state="deleted",
             )
         )
@@ -699,8 +697,8 @@ class TestVyosFirewallRulesModule(TestVyosModule):
         set_module_args(
             dict(
                 config=[
-                    dict(afi="ipv4", rule_sets=[dict(name="V4-INGRESS"),]),
-                    dict(afi="ipv6", rule_sets=[dict(name="V6-INGRESS"),]),
+                    dict(afi="ipv4", rule_sets=[dict(name="V4-INGRESS")]),
+                    dict(afi="ipv6", rule_sets=[dict(name="V6-INGRESS")]),
                 ],
                 state="deleted",
             )
@@ -720,8 +718,8 @@ class TestVyosFirewallRulesModule(TestVyosModule):
         set_module_args(
             dict(
                 config=[
-                    dict(afi="ipv4", rule_sets=[dict(name="V4-ING"),]),
-                    dict(afi="ipv6", rule_sets=[dict(name="V6-ING"),]),
+                    dict(afi="ipv4", rule_sets=[dict(name="V4-ING")]),
+                    dict(afi="ipv6", rule_sets=[dict(name="V6-ING")]),
                 ],
                 state="deleted",
             )
diff --git a/tests/unit/modules/network/vyos/test_vyos_ospfv2.py b/tests/unit/modules/network/vyos/test_vyos_ospfv2.py
index b825066..3d558b8 100644
--- a/tests/unit/modules/network/vyos/test_vyos_ospfv2.py
+++ b/tests/unit/modules/network/vyos/test_vyos_ospfv2.py
@@ -99,7 +99,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         dict(
                             area_id="4",
                             area_type=dict(stub=dict(default_cost=10)),
-                            network=[dict(address="192.0.2.0/24"),],
+                            network=[dict(address="192.0.2.0/24")],
                             range=[
                                 dict(address="192.0.3.0/24", cost=10),
                                 dict(address="192.0.4.0/24", cost=12),
@@ -143,7 +143,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         dict(
                             area_id="14",
                             area_type=dict(stub=dict(default_cost=20)),
-                            network=[dict(address="192.0.12.0/24"),],
+                            network=[dict(address="192.0.12.0/24")],
                             range=[
                                 dict(address="192.0.13.0/24", cost=10),
                                 dict(address="192.0.14.0/24", cost=12),
@@ -207,7 +207,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         dict(
                             area_id="15",
                             area_type=dict(stub=dict(default_cost=10)),
-                            network=[dict(address="192.0.12.0/24"),],
+                            network=[dict(address="192.0.12.0/24")],
                             range=[
                                 dict(address="192.0.13.0/24", cost=10),
                                 dict(address="192.0.14.0/24", cost=12),
@@ -251,7 +251,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         dict(
                             area_id="14",
                             area_type=dict(stub=dict(default_cost=20)),
-                            network=[dict(address="192.0.12.0/24"),],
+                            network=[dict(address="192.0.12.0/24")],
                             range=[
                                 dict(address="192.0.13.0/24", cost=10),
                                 dict(address="192.0.14.0/24", cost=12),
@@ -401,7 +401,7 @@ set protocols ospf redistribute bgp metric-type '2'"""
                         dict(
                             area_id="4",
                             area_type=dict(stub=dict(default_cost=10)),
-                            network=[dict(address="192.0.2.0/24"),],
+                            network=[dict(address="192.0.2.0/24")],
                             range=[
                                 dict(address="192.0.3.0/24", cost=10),
                                 dict(address="192.0.4.0/24", cost=12),
diff --git a/tests/unit/modules/network/vyos/test_vyos_ospfv3.py b/tests/unit/modules/network/vyos/test_vyos_ospfv3.py
index 1d9cb3a..9c016da 100644
--- a/tests/unit/modules/network/vyos/test_vyos_ospfv3.py
+++ b/tests/unit/modules/network/vyos/test_vyos_ospfv3.py
@@ -101,7 +101,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         ),
                         dict(
                             area_id="3",
-                            range=[dict(address="2001:db40::/32"),],
+                            range=[dict(address="2001:db40::/32")],
                         ),
                     ],
                 ),
@@ -139,7 +139,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         ),
                         dict(
                             area_id="13",
-                            range=[dict(address="2001:db44::/32"),],
+                            range=[dict(address="2001:db44::/32")],
                         ),
                     ],
                 ),
@@ -203,7 +203,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         ),
                         dict(
                             area_id="14",
-                            range=[dict(address="2001:db40::/32"),],
+                            range=[dict(address="2001:db40::/32")],
                         ),
                     ],
                 ),
@@ -238,7 +238,7 @@ class TestVyosFirewallRulesModule(TestVyosModule):
                         ),
                         dict(
                             area_id="13",
-                            range=[dict(address="2001:db44::/32"),],
+                            range=[dict(address="2001:db44::/32")],
                         ),
                     ],
                 ),
@@ -323,7 +323,7 @@ set protocols ospfv3 redistribute 'bgp'"""
                         ),
                         dict(
                             area_id="3",
-                            range=[dict(address="2001:db40::/32"),],
+                            range=[dict(address="2001:db40::/32")],
                         ),
                     ],
                 ),