Fix sanity

Signed-off-by: Alina Buzachis <abuzachis@redhat.com>
ansible-collections · Oct 14, 2024 · e15fc5a · e15fc5a
1 parent 2443d08
commit e15fc5a
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/plugins/modules/ec2_vpc_vpn.py b/plugins/modules/ec2_vpc_vpn.py
@@ -904,7 +904,7 @@ def main():
             options=dict(
                 TunnelInsideCidr=dict(type="str"),
                 TunnelInsideIpv6Cidr=dict(type="str"),
-                PreSharedKey=dict(type="str"),
+                PreSharedKey=dict(type="str", no_log=True),
             ),
         ),
         static_only=dict(default=False, type="bool"),

diff --git a/plugins/modules/ec2_vpc_vpn_info.py b/plugins/modules/ec2_vpc_vpn_info.py
@@ -57,6 +57,7 @@
     description: List of one or more EC2 VPN Connections.
     type: list
     elements: dict
+    returned: always
     contains:
       category:
         description: The category of the VPN connection.

diff --git a/tests/unit/plugins/modules/test_ec2_vpc_vpn.py b/tests/unit/plugins/modules/test_ec2_vpc_vpn.py
@@ -149,13 +149,15 @@ def test_create_filter(ansible_module, provided_filters, expected_result, expect
         (
             {"customer_gateway_id": None, "vpn_gateway_id": "vgw-123", "static_only": True},
             None,
-            "No matching connection was found. To create a new connection you must provide customer_gateway_id and one of either transit_gateway_id or vpn_gateway_id.",
+            "No matching connection was found. To create a new connection you must provide customer_gateway_id"
+            + " and one of either transit_gateway_id or vpn_gateway_id.",
         ),
         # Case 4: Both customer gateway and VPN gateway are None
         (
             {"customer_gateway_id": None, "vpn_gateway_id": None, "static_only": False},
             None,
-            "No matching connection was found. To create a new connection you must provide customer_gateway_id and one of either transit_gateway_id or vpn_gateway_id.",
+            "No matching connection was found. To create a new connection you must provide customer_gateway_id"
+            + " and one of either transit_gateway_id or vpn_gateway_id.",
         ),
         # Case 5: Optional parameters passed (e.g., static routes)
         (