search

terraform-compliance / cli

a lightweight, security focused, BDD test framework against terraform.
https://terraform-compliance.com
MIT License
1.34k stars 151 forks source link

fix: replace mock module with unittest.mock #666

Closed burningalchemist closed 1 year ago

burningalchemist commented 1 year ago

Hi @eerkunt,

In 1.3.35 external mock module was removed from the dependency list, but the import wasn't updated in common/error_handling.py where it's requested. This predictably results in No module named 'mock' error.

This PR fixes the import statement.

Cheers!👍

eerkunt commented 1 year ago

Oh thanks for this PR! 🎉

eerkunt commented 1 year ago

Accidentally hit the merge button :D

eerkunt commented 1 year ago

Reverted this PR as it wasn't passing unit tests.

burningalchemist commented 1 year ago

@eerkunt yeah, happens. 😅 Shall I open a new PR for that? :)

eerkunt commented 1 year ago

Yes please. So sorry ;D