CFn: handle secretsmanager policy BlockPublicPolicy

[simonrw]

Motivation

When deploying an AWS::SecretsManager::ResourcePolicy resource with CloudFormation, if BlockPublicPolicy is not specified, it should default to True however in LocalStack it is represented as None which is not a valid type for this property.

Changes

• Default the property to True if not specified
• Update existing test that creates a ResourcePolicy to either specify the value, or use AWS::NoValue to not provide a value

[github-actions]

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 40m 41s ⏱️ + 1m 16s
2 980 tests +1  2 675 ✅ +1  305 💤 ±0  0 ❌ ±0 
2 982 runs  +1  2 675 ✅ +1  307 💤 ±0  0 ❌ ±0 

Results for commit c1e1461. ± Comparison against base commit e3db788.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

tests.aws.services.cloudformation.resources.test_secretsmanager ‑ test_cfn_secret_policy

tests.aws.services.cloudformation.resources.test_secretsmanager ‑ test_cfn_secret_policy[default]
tests.aws.services.cloudformation.resources.test_secretsmanager ‑ test_cfn_secret_policy[true]