Update to add test ensuring remote modules are cached

Author: nicholasjackson

parse_test.go

@@ -19,10 +19,11 @@ import (
 )
 
 func setupParser(t *testing.T, options ...*ParserOptions) *Parser {
-	os.Setenv("SHIPYARD_CONFIG", "/User/yamcha/.shipyard")
+	home := os.Getenv("HOME")
+	os.Setenv("HOME", t.TempDir())
 
 	t.Cleanup(func() {
-		os.Unsetenv("SHIPYARD_CONFIG")
+		os.Setenv("HOME", home)
 	})
 
 	o := DefaultOptions()
@@ -385,6 +386,25 @@ func TestParseModuleCreatesResources(t *testing.T) {
 
 }
 
+func TestParseModuleDoesNotCacheLocalFiles(t *testing.T) {
+	absoluteFolderPath, err := filepath.Abs("./test_fixtures/modules/modules.hcl")
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	p := setupParser(t)
+
+	c, err := p.ParseFile(absoluteFolderPath)
+	require.NoError(t, err)
+	require.NotNil(t, c)
+
+	// the remote module should be cached
+	require.DirExists(t, filepath.Join(p.options.ModuleCache, "github.com_jumppad-labs_hclconfig_test_fixtures_single"))
+
+	// the local module should not be cached
+	require.NoDirExists(t, filepath.Join(p.options.ModuleCache, "single"))
+}
+
 func TestParseModuleCreatesOutputs(t *testing.T) {
 	absoluteFolderPath, err := filepath.Abs("./test_fixtures/modules/modules.hcl")
 	if err != nil {

test_fixtures/modules/modules.hcl

@@ -33,11 +33,12 @@ module "consul_2" {
   }
 }
 
+// ensure that this module is from a remote source
 module "consul_3" {
   // all resources in this module will only be created after all the 
   // resources in 'consul_1' have been created.
   depends_on = ["module.consul_1"]
-  source     = "../single"
+  source     = "github.com/jumppad-labs/hclconfig/test_fixtures//single"
 }
 
 output "object" {