iterative / terraform-provider-iterative

☁️ Terraform plugin for machine learning workloads: spot instance recovery & auto-termination | AWS, GCP, Azure, Kubernetes
https://registry.terraform.io/providers/iterative/iterative/latest/docs
Apache License 2.0
290 stars 27 forks source link

Fix broken shebang on smoke test script #675

Closed 0x2b3bfa0 closed 2 years ago

0x2b3bfa0 commented 2 years ago

Continuation of https://github.com/iterative/terraform-provider-iterative/pull/673#discussion_r974478041

Shells try to interpret executable files even if they don't have a valid shebang, but the new string's shebang is broken due to the initial indentation whitespace. While this is not breaking tests, it would be nice to fix the string so the -e option becomes effective again.

Before #673

https://github.com/iterative/terraform-provider-iterative/blob/672acef656f83f31ca36764a92e10b146fb02469/task/task_test.go#L83-L90

After #673

https://github.com/iterative/terraform-provider-iterative/blob/7461e69db5a0f2e6a82931113369ae7e8143e1ae/task/task_smoke_test.go#L94-L102