Disable some sentiel tests in Python 3.6 - it doesn't play nice with …

…deepcopy

plugins/modules/ec2_security_group.py

@@ -836,7 +836,8 @@ def get_target_from_rule(module, client, rule, name, group, groups, vpc_id):
 
 def _strip_rule(rule):
     """
-    Strips the Target/Source and Port information from a rule
+    Returns a copy of the rule with the Target/Source and Port information
+    from a rule stripped out.
     This can then be combined with the expanded information
     """
     stripped_rule = deepcopy(rule)

tests/unit/plugins/modules/ec2_security_group/test_expand_rules.py

@@ -4,6 +4,7 @@
 # GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
 
 import pytest
+import sys
 from unittest.mock import sentinel
 
 import ansible_collections.amazon.aws.plugins.modules.ec2_security_group as ec2_security_group_module
@@ -173,6 +174,9 @@ def test_expand_ports_list(rule, expected):
     assert ec2_security_group_module.expand_ports_list(rule) == expected
 
 
+@pytest.mark.skipif(
+    sys.version_info < (3, 7), reason="requires Python 3.7 or higher - sentinel doesn't behave well with deepcopy in Python 3.6"
+)
 @pytest.mark.parametrize("source_type", sorted(ec2_security_group_module.SOURCE_TYPES_ALL))
 def test_strip_rule_source(source_type):
     rule = {source_type: sentinel.SOURCE_VALUE}
@@ -184,8 +188,11 @@ def test_strip_rule_source(source_type):
     assert rule == {source_type: sentinel.SOURCE_VALUE, "sentinel": sentinel.SENTINEL_VALUE}
 
 
+@pytest.mark.skipif(
+    sys.version_info < (3, 7), reason="requires Python 3.7 or higher - sentinel doesn't behave well with deepcopy in Python 3.6"
+)
 @pytest.mark.parametrize("port_type", sorted(ec2_security_group_module.PORT_TYPES_ALL))
-def test_strip_rule_source(port_type):
+def test_strip_rule_port(port_type):
     rule = {port_type: sentinel.PORT_VALUE}
     assert ec2_security_group_module._strip_rule(rule) == {}
     assert rule == {port_type: sentinel.PORT_VALUE}
@@ -195,6 +202,9 @@ def test_strip_rule_source(port_type):
     assert rule == {port_type: sentinel.PORT_VALUE, "sentinel": sentinel.SENTINEL_VALUE}
 
 
+@pytest.mark.skipif(
+    sys.version_info < (3, 7), reason="requires Python 3.7 or higher - sentinel doesn't behave well with deepcopy in Python 3.6"
+)
 @pytest.mark.parametrize("rule, expected", RULE_EXPANSION)
 def test_rule_expand(rule, expected):
     assert ec2_security_group_module.expand_rule(rule) == expected