Suppress findings when conditions exist, except when --flag-all-risky-actions flag is included

[jacobappleton-orbis]

What does this PR do?

• Fix Infrastructure Modification check ignores conditions set on policy #278
  • Conditions on policy now suppress all issue types, unless the --flag-all-risky-actions switch is set
  • --flag-all-risky-actions switch is now available to all scanning commands: scan, scan-multi-account, and scan-policy-file
• Fix Unhandled exception when running multi-account scan #299

What gif best describes this PR or how it makes you feel?

Completion checklist

• Additions and changes have unit tests
• The pull request has been appropriately labeled using the provided PR labels
• GitHub actions automation is passing (make test, make lint, make security-test, make test-js)
• [ ] If the UI contents or JavaScript files have been modified, generate a new example report:

[kmcquade]

@jacobappleton-orbis - it looks like you ran the formatter across the code base. It's hard to read which parts of the PR actually changed the code vs. what was the formatter. Can you revert the files that are formatting related so I can just review the ones that introduce changes? Thanks :)

[salesforce-cla]

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Dan Jacobson <d***@o***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated.

[jacobappleton-orbis]

@kmcquade I have removed the commit that performed the formatting. It appears that salesforce-cla bot now thinks that I have not signed the CLA, when I definitely have. Otherwise this should be in a good state now.

[jacobappleton-orbis]

Ok, I think I've fixed the CLA issue as well.

[kmcquade]

@jacobappleton-orbis unit tests and NodeJS tests look good. But looks like it's failing the mypy check. If you can make that fix, then happy to merge. Thanks!

[gruebel]

I took the liberty and fixed the tiny type hint issue and since @kmcquade already gave his ok, so will I 😄