ansible tests bug fixes (#122)

* v5.1.3

* ansible-lint bug fixes

* ansible-lint bug fixes

* ansible-lint bug fixes

* ansible test bug-fixes

Author: chkp-edenbr

CHANGELOG.rst

@@ -4,6 +4,14 @@ Check_Point.Mgmt Release Notes
 
 .. contents:: Topics
 
+v5.2.2
+======
+
+Release Summary
+---------------
+
+This is release 5.2.2 of ``check_point.mgmt``, released on 2024-01-28.
+
 v5.2.1
 ======
 

changelogs/changelog.yaml

@@ -956,3 +956,10 @@ releases:
     fragments:
       - 5.2.1.yml
     release_date: "2024-01-16"
+  5.2.2:
+    changes:
+      release_summary: This is release 5.2.2 of ``check_point.mgmt``, released on
+        2024-01-28.
+    fragments:
+      - 5.2.2.yml
+    release_date: '2024-01-28'

galaxy.yml

@@ -9,17 +9,17 @@ namespace: check_point
 name: mgmt
 
 # The version of the collection. Must be compatible with semantic versioning
-version: 5.2.1
+version: 5.2.2
 
 # The path to the Markdown (.md) readme file. This path is relative to the root of the collection
 readme: README.md
 
 # A list of the collection's content authors. Can be just the name or in the format 'Full Name <email> (url)
 # @nicks:irc/im.site#channel'
 authors:
-- Or Soffer <orso@checkpoint.com>
-- Shiran Golzar <shirango@checkpoint.com>
-- Eden Brillant <edenbr@checkpoint.com>
+ - Or Soffer <orso@checkpoint.com>
+ - Shiran Golzar <shirango@checkpoint.com>
+ - Eden Brillant <edenbr@checkpoint.com>
 
 
 ### OPTIONAL but strongly recommended
@@ -30,7 +30,7 @@ description: Check Point collection for the Management Server
 # Either a single license or a list of licenses for content inside of a collection. Ansible Galaxy currently only
 # accepts L(SPDX,https://spdx.org/licenses/) licenses. This key is mutually exclusive with 'license_file'
 license:
-- GPL-2.0-or-later
+ - GPL-2.0-or-later
 
 # The path to the license file for the collection. This path is relative to the root of the collection. This key is
 # mutually exclusive with 'license'

meta/runtime.yml

@@ -1,2 +1,52 @@
 ---
 requires_ansible: ">=2.14.0"
+plugin_routing:
+  modules:
+    cp_mgmt_add_nat_rule:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_add_nat_rule will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_nat_rule with more functionality instead.
+    cp_mgmt_set_nat_rule:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_set_nat_rule will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_nat_rule with more functionality instead.
+    cp_mgmt_delete_nat_rule:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_delete_nat_rule will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_nat_rule with more functionality instead.
+    cp_mgmt_access_layer:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_access_layer will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_access_layers with more functionality instead.
+    cp_mgmt_show_task:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_show_task will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_task_facts instead.
+    cp_mgmt_show_tasks:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_show_tasks will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_task_facts instead.
+    cp_mgmt_host:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_host will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_hosts with more functionality instead.
+    cp_mgmt_threat_layer:
+      deprecation:
+        removal_date: "2024-11-01"
+        warning_text: >
+          cp_mgmt_threat_layer will be removed in a future release of this collection.
+          Use newer and updated module cp_mgmt_threat_layers with more functionality instead.

plugins/doc_fragments/checkpoint_commands.py

@@ -27,4 +27,9 @@ class ModuleDocFragment(object):
     description:
       - Version of checkpoint. If not given one, the latest version taken.
     type: str
+  auto_publish_session:
+    description:
+      - Publish the current session if changes have been performed after task completes.
+    type: bool
+    default: False
 """

plugins/doc_fragments/checkpoint_objects.py

@@ -15,17 +15,15 @@ class ModuleDocFragment(object):
 options:
   state:
     description:
-      - State of the access rule (present or absent). Defaults to present.
+      - State of the access rule (present or absent).
     type: str
     default: present
-    choices:
-      - 'present'
-      - 'absent'
+    choices: ['present', 'absent']
   auto_publish_session:
     description:
-      - Publish the current session if changes have been performed
-        after task completes.
+      - Publish the current session if changes have been performed after task completes.
     type: bool
+    default: False
   wait_for_task:
     description:
       - Wait for the task to end. Such as publish task.

plugins/doc_fragments/checkpoint_objects_action_module.py

@@ -18,6 +18,7 @@ class ModuleDocFragment(object):
       - Publish the current session if changes have been performed
         after task completes.
     type: bool
+    default: False
   wait_for_task_timeout:
     description:
       - How many minutes to wait until throwing a timeout error.

plugins/module_utils/checkpoint.py

@@ -46,13 +46,13 @@
 }
 
 checkpoint_argument_spec_for_action_module = dict(
-    auto_publish_session=dict(type="bool"),
+    auto_publish_session=dict(type="bool", default=False),
     wait_for_task_timeout=dict(type="int", default=30),
     version=dict(type="str"),
 )
 
 checkpoint_argument_spec_for_objects = dict(
-    auto_publish_session=dict(type="bool"),
+    auto_publish_session=dict(type="bool", default=False),
     wait_for_task=dict(type="bool", default=True),
     wait_for_task_timeout=dict(type="int", default=30),
     state=dict(type="str", choices=["present", "absent"], default="present"),
@@ -65,7 +65,7 @@
     wait_for_task=dict(type="bool", default=True),
     wait_for_task_timeout=dict(type="int", default=30),
     version=dict(type="str"),
-    auto_publish_session=dict(type="bool"),
+    auto_publish_session=dict(type="bool", default=False),
 )
 
 delete_params = [
@@ -76,7 +76,8 @@
     "rule-name",
     "package",
     "ignore-errors",
-    "ignore-warnings"
+    "ignore-warnings",
+    "gateway-uid"
 ]
 
 remove_from_set_payload = {
@@ -304,6 +305,10 @@ def get_payload_from_parameters(params):
                 ):
                     parameter = "version"
 
+                # message & syslog_facility are internally used by Ansible, so need to avoid param duplicity
+                elif parameter == "login_message":
+                    parameter = "message"
+
                 payload[parameter.replace("_", "-")] = parameter_value
 
     return payload
@@ -579,7 +584,7 @@ def api_call_facts(module, api_call_object, api_call_object_plural_version):
         False,
         False,
     )
-    result = {api_call_object: response}
+    result = {api_call_object.replace("-", "_"): response}
     return result
 
 
@@ -1431,12 +1436,12 @@ def api_call_facts_for_rule(
 
 
 # The code from here till EOF will be deprecated when Rikis' modules will be deprecated
-checkpoint_argument_spec = dict(
-    auto_publish_session=dict(type="bool", default=True),
-    policy_package=dict(type="str", default="standard"),
-    auto_install_policy=dict(type="bool", default=True),
-    targets=dict(type="list"),
-)
+# checkpoint_argument_spec = dict(
+#     auto_publish_session=dict(type="bool", default=True),
+#     policy_package=dict(type="str", default="standard"),
+#     auto_install_policy=dict(type="bool", default=True),
+#     targets=dict(type="list"),
+# )
 
 
 def publish(connection, uid=None):

plugins/modules/cp_mgmt_access_layer.py

@@ -23,7 +23,7 @@
 
 ANSIBLE_METADATA = {
     "metadata_version": "1.1",
-    "status": ["preview"],
+    "status": ["deprecated"],
     "supported_by": "community",
 }
 

plugins/modules/cp_mgmt_access_point_name_facts.py

@@ -131,7 +131,7 @@ def main():
     )
     argument_spec.update(checkpoint_argument_spec_for_facts)
 
-    module = AnsibleModule(argument_spec=argument_spec)
+    module = AnsibleModule(argument_spec=argument_spec, supports_check_mode=True)
 
     api_call_object = "access-point-name"
     api_call_object_plural_version = "access-point-names"