PolicyStatementProps should accept Effect.

[nikamipara]

Note: for support questions, please first reference our documentation, then use Stackoverflow. This repository's issues are intended for feature requests and bug reports.

• I'm submitting a ...

  • [X ] 🪲 bug report

• What is the current behavior?
  If the current behavior is a 🪲bug🪲: Please provide the steps to reproduce
  As per Current Behaviour to set allow policy on on say dummy action we have to write following.

var policy = new PolicyStatement(PolicyStatementProps.builder()
.withActions(List.of("dummy_action"))
.build()
);
policy.setEffect(Effect.ALLOW);

• What is the expected behavior (or behavior of feature suggested)?
  Expected is following.

var policy = new PolicyStatement(PolicyStatementProps.builder()
.withActions(List.of("dummy_action"))
.withEffect(Effect.ALLOW)
.build()
);

• What is the motivation / use case for changing the behavior or adding this feature?

• Please tell us about your environment:

  • CDK CLI Version: 00.36.00
  • Module Version: xx.xx.xx
  • OS: [all | Windows 10 | OSX Mojave | Ubuntu | etc... ]
  • Language: [all | TypeScript | Java | Python ]

• Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)

[rix0rrr]

Fixed in #3165