Allow an ENV var for APPPACK_TOML to specify the path to the toml file. See Issue #10

Author: issackelly

builder/build/apppacktoml.go

@@ -103,19 +103,27 @@ func (a *AppPackToml) GetTestEnv() map[string]string {
 func ParseAppPackToml(ctx context.Context) (*AppPackToml, error) {
 	var config AppPackToml
 	// if the file doesn't exist, just return an empty config
-	if _, err := os.Stat("apppack.toml"); os.IsNotExist(err) {
-		log.Ctx(ctx).Debug().Msg("apppack.toml not found")
+	filename := "apppack.toml"
+	if envFile := os.Getenv("APPPACK_TOML"); envFile != "" {
+		filename = envFile
+	}
+	if _, err := os.Stat(filename); os.IsNotExist(err) {
+		log.Ctx(ctx).Debug().Msg(fmt.Sprintf("%s not found", filename))
 		return &config, nil
 	}
-	if _, err := toml.DecodeFile("apppack.toml", &config); err != nil {
+	if _, err := toml.DecodeFile(filename, &config); err != nil {
 		return nil, err
 	}
 	return &config, nil
 }
 
 func (a AppPackToml) Write(ctx context.Context) error {
-	log.Ctx(ctx).Debug().Msg("writing apppack.toml")
-	f, err := os.Create("apppack.toml")
+	filename := "apppack.toml"
+	if envFile := os.Getenv("APPPACK_TOML"); envFile != "" {
+		filename = envFile
+	}
+	log.Ctx(ctx).Debug().Msg(fmt.Sprintf("writing %s", filename))
+	f, err := os.Create(filename)
 	if err != nil {
 		return err
 	}

builder/build/prebuild.go

@@ -148,15 +148,19 @@ func (b *Build) ConvertAppJson() error {
 		return err
 	}
 	// check if apppack.toml file exists
-	apppackTomlExists, err := b.state.FileExists("apppack.toml")
+	filename := "apppack.toml"
+	if envFile := os.Getenv("APPPACK_TOML"); envFile != "" {
+		filename = envFile
+	}
+	apppackTomlExists, err := b.state.FileExists(filename)
 	if err != nil {
 		return err
 	}
 	if appJsonExists && !apppackTomlExists {
 		// convert app.json to apppack.toml
-		b.Log().Info().Msg("Converting app.json to apppack.toml")
+		b.Log().Info().Msg(fmt.Sprintf("Converting %s to %s", "app.json", filename))
 		t := b.AppJSON.ToApppackToml()
-		return b.state.WriteTomlToFile("apppack.toml", t)
+		return b.state.WriteTomlToFile(filename, t)
 	}
 	return nil
 }

builder/filesystem/filesystem.go

@@ -58,7 +58,12 @@ func (f *FileState) Log() *zerolog.Logger {
 
 func (f *FileState) CreateIfNotExists() error {
 	// touch files codebuild expects to exist
-	for _, filename := range []string{"apppack.toml", "build.log", "metadata.toml", "test.log"} {
+	apppackToml := "apppack.toml"
+	if envFile := os.Getenv("APPPACK_TOML"); envFile != "" {
+		apppackToml = envFile
+	}
+
+	for _, filename := range []string{apppackToml, "build.log", "metadata.toml", "test.log"} {
 		exists, err := f.FileExists(filename)
 		if err != nil {
 			return err

builder/filesystem/filesystem_test.go

@@ -16,8 +16,14 @@ var testContext = zerolog.New(os.Stdout).With().Timestamp().Logger().WithContext
 
 func TestCreateIfNotExists(t *testing.T) {
 	fs := afero.Afero{Fs: afero.NewMemMapFs()}
-	if _, err := fs.Stat("apppack.toml"); !os.IsNotExist(err) {
-		t.Error("apppack.toml should not exist")
+
+	filename := "apppack.toml"
+	if envFile := os.Getenv("APPPACK_TOML"); envFile != "" {
+		filename = envFile
+	}
+
+	if _, err := fs.Stat(filename); !os.IsNotExist(err) {
+		t.Error(fmt.Sprintf("%s should not exist", filename))
 	}
 	s := &FileState{
 		fs:  fs,
@@ -27,8 +33,8 @@ func TestCreateIfNotExists(t *testing.T) {
 	if err != nil {
 		t.Error(err)
 	}
-	if _, err := fs.Stat("apppack.toml"); os.IsNotExist(err) {
-		t.Error("apppack.toml should exist")
+	if _, err := fs.Stat(filename); os.IsNotExist(err) {
+		t.Error(fmt.Sprintf("%s should exist", filename))
 	}
 }