search

GoogleCloudPlatform / terraform-python-testing-helper

Simple Python test helper for Terraform.
https://pypi.org/project/tftest/
Apache License 2.0
214 stars 31 forks source link

fix: On windows if a file is read only the cleanup reports errors. #38

Closed grahamhar closed 2 years ago

grahamhar commented 2 years ago

Adding the onerror to change the file permission then attempt the delete again fixes the issue reported in #37.

ludoo commented 2 years ago

Thanks a lot for this! Can you sign the CLA, then it's good to go. Nitpick: does it make sense to exit cleanly from remove_readonly in case of errors there?

ludoo commented 2 years ago

Merging this, do you need a release on Pypi?

grahamhar commented 2 years ago

Merging this, do you need a release on Pypi?

At your convenience that would be great.

ludoo commented 2 years ago

Here you go :) https://pypi.org/project/tftest/1.6.1/