fix(terraform): Security group attached to an Elastic DocumentDB cluster is not recognized by check CKV2_AWS_5

[bhean]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description

Include Elastic DocumentDB as a valid associated resource for an aws_security_group for graph check CKV2_AWS_5

Fixes #6686

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my feature, policy, or fix is effective and works
• New and existing tests pass locally with my changes

[bo156]

thanks for the contribution!
please see my comment

[bhean]

thanks for the contribution! please see my comment

Hi!! If I'm not wrong, the failing test is defined in line 833 of this file.
This is a check for an specific thing (a security group attached to nothing) so we only need a passing test for each passing condition, but only one failing test (for a security group attached to nothing), is that correct?
So, IMHO the pull-request is complete... but this is my first contribution here, so if I'm wrong about that, please don't hesitate to pointing me in the right direction...

[itai1357]

Hi @bhean.
you seems to be right. will check it with engineers with knowledge in the area and will return an answer =]