Add integration tests. Suggest adding an arbitrary init script in a
bucket. Reference the get_from_bucket integration test for fixture
examples. Have the init script echo something to the console output and
validate the expected output from kitchen.
Suggest exercising the missing mandatory argument case inside
integration tests. This was a sharp edge in the field and has been
refactored when porting over here.
9740c7d (Jeff McCune, 5 minutes ago)
Remaining work:
Ensure the function body uses stdlib:: prefixes appropriately.
Add a feature flag to incorporate the function. Reference https://github.com/terraform-google-modules/terraform-google-startup-scripts/pull/12 for how to assemble the content from snippets and user facing feature flags.
Add integration tests. Suggest adding an arbitrary init script in a bucket. Reference the get_from_bucket integration test for fixture examples. Have the init script echo something to the console output and validate the expected output from kitchen.
Suggest exercising the missing mandatory argument case inside integration tests. This was a sharp edge in the field and has been refactored when porting over here.