Merge pull request #3499 from dokku/3491-buildpacks-writing

Ensure the .buildpacks file exists before writing to it
This commit is contained in:
Jose Diaz-Gonzalez
2019-04-17 02:52:49 -04:00
committed by GitHub
2 changed files with 23 additions and 1 deletions

View File

@@ -26,7 +26,7 @@ func main() {
}
buildpacksPath := path.Join(tmpWorkDir, ".buildpacks")
file, err := os.OpenFile(buildpacksPath, os.O_RDWR|os.O_TRUNC, 0600)
file, err := os.OpenFile(buildpacksPath, os.O_RDWR|os.O_TRUNC|os.O_CREATE, 0600)
if err != nil {
common.LogFail(fmt.Sprintf("Error writing .buildpacks file: %s", err.Error()))
return

View File

@@ -209,3 +209,25 @@ teardown() {
echo "status: $status"
assert_output_not_exists
}
@test "(buildpacks) buildpacks deploy" {
destroy_app
echo "output: $output"
echo "status: $status"
assert_success
create_app
echo "output: $output"
echo "status: $status"
assert_success
run /bin/bash -c "dokku buildpacks:set $TEST_APP https://github.com/heroku/heroku-buildpack-nodejs"
echo "output: $output"
echo "status: $status"
assert_success
run deploy_app
echo "output: $output"
echo "status: $status"
assert_success
}