Fix version logging statement

TerrafilePath is now a complete path to Terrafile
Add TODO section to README

Author: nritholtz

README.md

@@ -40,7 +40,7 @@ INFO[0000] [*] Checking out master of [email protected]:terraform-aws-modules/terra
 
 Terrafile config file in custom directory
 ```sh
-$ terrafile -f config/
+$ terrafile -f config/Terrafile
 INFO[0000] [*] Checking out v1.46.0 of [email protected]:terraform-aws-modules/terraform-aws-vpc  
 INFO[0000] [*] Checking out master of [email protected]:terraform-aws-modules/terraform-aws-vpc  
 ```
@@ -50,4 +50,10 @@ Terraform modules exported to custom directory
 $ terrafile -p custom_directory
 INFO[0000] [*] Checking out master of [email protected]:terraform-aws-modules/terraform-aws-vpc  
 INFO[0001] [*] Checking out v1.46.0 of [email protected]:terraform-aws-modules/terraform-aws-vpc  
-```
+```
+
+## TODO
+* Break out the main logic into seperate commands (e.g. version, help, run)
+* Update tests to include unit tests for broken out commands
+* Add coverage tool and badge
+* May be worth renaming Terrafile config file to something that won't be misinterpreted as the binary

main.go

@@ -20,7 +20,7 @@ type module struct {
 var opts struct {
 	ModulePath string `short:"p" long:"module_path" default:"./vendor/modules" description:"File path to install generated terraform modules"`
 
-	TerrafilePath string `short:"f" long:"terrafile_file" default:"." description:"File path in which the Terrafile file is located"`
+	TerrafilePath string `short:"f" long:"terrafile_file" default:"./Terrafile" description:"File path to the Terrafile file"`
 }
 
 // To be set by goreleaser on build
@@ -46,7 +46,7 @@ func gitClone(repository string, version string, moduleName string) {
 }
 
 func main() {
-	log.Printf("Terraform: version %v, commit %v, built at %v", version, commit, date)
+	fmt.Printf("Terrafile: version %v, commit %v, built at %v \n", version, commit, date)
 	_, err := flags.Parse(&opts)
 
 	// Invalid choice
@@ -55,7 +55,7 @@ func main() {
 	}
 
 	// Read File
-	yamlFile, err := ioutil.ReadFile(fmt.Sprint(opts.TerrafilePath, "/Terrafile"))
+	yamlFile, err := ioutil.ReadFile(opts.TerrafilePath)
 	if err != nil {
 		log.Fatalln(err)
 	}

main_test.go

@@ -1,6 +1,7 @@
 package main
 
 import (
+	"fmt"
 	"io/ioutil"
 	"os"
 	"path"
@@ -25,7 +26,7 @@ func TestTerraformWithTerrafilePath(t *testing.T) {
 	folder, back := setup(t)
 	defer back()
 
-	testcli.Run(terrafileBinaryPath, "-f", folder)
+	testcli.Run(terrafileBinaryPath, "-f", fmt.Sprint(folder, "/terrafile"))
 
 	if !testcli.Success() {
 		t.Fatalf("Expected to succeed, but failed: %q with message: %q", testcli.Error(), testcli.Stderr())