From 8ca0b3770b0f309ba2e5a33fb22aee6768c5693a Mon Sep 17 00:00:00 2001
From: Jonathan Kuleff <jonathankuleff@gmail.com>
Date: Thu, 2 Sep 2021 23:42:33 +1000
Subject: [PATCH 1/2] fix(lib): fix source path generated for local modules

---
 packages/cdktf/lib/terraform-module.ts        |   2 +-
 .../terraform-hcl-module.test.ts.snap         | 291 ++++++++++++------
 .../modules/__snapshots__/test.ts.snap        |   4 +-
 3 files changed, 206 insertions(+), 91 deletions(-)

diff --git a/packages/cdktf/lib/terraform-module.ts b/packages/cdktf/lib/terraform-module.ts
index 28128d4631..552217593e 100644
--- a/packages/cdktf/lib/terraform-module.ts
+++ b/packages/cdktf/lib/terraform-module.ts
@@ -33,7 +33,7 @@ export abstract class TerraformModule
     super(scope, id);
 
     if (options.source.startsWith("./") || options.source.startsWith("../")) {
-      this.source = path.join("..", options.source);
+      this.source = path.join("../../..", options.source);
     } else {
       this.source = options.source;
     }
diff --git a/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap b/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
index 10bca53bcd..a9edebcceb 100644
--- a/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
+++ b/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
@@ -2,12 +2,25 @@
 
 exports[`add provider 1`] = `
 "{
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
+    }
+  },
   \\"module\\": {
     \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
       \\"providers\\": {
         \\"test\\": \\"test.provider1\\"
       },
-      \\"source\\": \\"../foo\\"
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
     }
   },
   \\"provider\\": {
@@ -17,32 +30,17 @@ exports[`add provider 1`] = `
         \\"alias\\": \\"provider1\\"
       }
     ]
-  },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
-      }
-    }
   }
 }"
 `;
 
 exports[`complex providers 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"param1\\": \\"name\\",
-      \\"param2\\": 1,
-      \\"param3\\": [
-        \\"id1\\",
-        \\"id2\\"
-      ],
-      \\"providers\\": {
-        \\"test.dst\\": \\"test.provider2\\",
-        \\"test.src\\": \\"test.provider1\\"
-      },
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
@@ -57,10 +55,24 @@ exports[`complex providers 1`] = `
       }
     ]
   },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"param1\\": \\"name\\",
+      \\"param2\\": 1,
+      \\"param3\\": [
+        \\"id1\\",
+        \\"id2\\"
+      ],
+      \\"source\\": \\"../../../foo\\",
+      \\"providers\\": {
+        \\"test.src\\": \\"test.provider1\\",
+        \\"test.dst\\": \\"test.provider2\\"
+      },
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
       }
     }
   }
@@ -69,9 +81,11 @@ exports[`complex providers 1`] = `
 
 exports[`depend on module 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
@@ -79,20 +93,30 @@ exports[`depend on module 1`] = `
       {}
     ]
   },
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
+    }
+  },
   \\"resource\\": {
     \\"test_resource\\": {
       \\"resource\\": {
+        \\"name\\": \\"foo\\",
         \\"depends_on\\": [
           \\"module.test\\"
         ],
-        \\"name\\": \\"foo\\"
-      }
-    }
-  },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+        \\"//\\": {
+          \\"metadata\\": {
+            \\"path\\": \\"test/resource\\",
+            \\"uniqueId\\": \\"resource\\"
+          }
+        }
       }
     }
   }
@@ -101,15 +125,34 @@ exports[`depend on module 1`] = `
 
 exports[`depend on other module 1`] = `
 "{
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
+    }
+  },
   \\"module\\": {
     \\"test_1\\": {
-      \\"source\\": \\"../foo\\"
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test_1\\",
+          \\"uniqueId\\": \\"test_1\\"
+        }
+      }
     },
     \\"test_2\\": {
+      \\"source\\": \\"../../../foo\\",
       \\"depends_on\\": [
         \\"module.test_1\\"
       ],
-      \\"source\\": \\"../foo\\"
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test_2\\",
+          \\"uniqueId\\": \\"test_2\\"
+        }
+      }
     }
   }
 }"
@@ -117,9 +160,22 @@ exports[`depend on other module 1`] = `
 
 exports[`minimal configuration 1`] = `
 "{
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
+    }
+  },
   \\"module\\": {
     \\"test\\": {
-      \\"source\\": \\"../foo\\"
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
     }
   }
 }"
@@ -127,34 +183,37 @@ exports[`minimal configuration 1`] = `
 
 exports[`multiple providers 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"providers\\": {
-        \\"differentType\\": \\"differentType\\",
-        \\"test\\": \\"test\\"
-      },
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
-    \\"differentType\\": [
+    \\"test\\": [
       {
         \\"access_key\\": \\"key\\"
       }
     ],
-    \\"test\\": [
+    \\"differentType\\": [
       {
         \\"access_key\\": \\"key\\"
       }
     ]
   },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"differentType\\": {
-        \\"version\\": \\"~> 2.0\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
+      \\"providers\\": {
+        \\"test\\": \\"test\\",
+        \\"differentType\\": \\"differentType\\"
       },
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
       }
     }
   }
@@ -163,6 +222,13 @@ exports[`multiple providers 1`] = `
 
 exports[`pass variables 1`] = `
 "{
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
+    }
+  },
   \\"module\\": {
     \\"test\\": {
       \\"param1\\": \\"name\\",
@@ -171,7 +237,13 @@ exports[`pass variables 1`] = `
         \\"id1\\",
         \\"id2\\"
       ],
-      \\"source\\": \\"../foo\\"
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
     }
   }
 }"
@@ -179,9 +251,11 @@ exports[`pass variables 1`] = `
 
 exports[`reference module 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
@@ -189,17 +263,27 @@ exports[`reference module 1`] = `
       {}
     ]
   },
-  \\"resource\\": {
-    \\"test_resource\\": {
-      \\"resource\\": {
-        \\"name\\": \\"\${module.test.name}\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
       }
     }
   },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+  \\"resource\\": {
+    \\"test_resource\\": {
+      \\"resource\\": {
+        \\"name\\": \\"\${module.test.name}\\",
+        \\"//\\": {
+          \\"metadata\\": {
+            \\"path\\": \\"test/resource\\",
+            \\"uniqueId\\": \\"resource\\"
+          }
+        }
       }
     }
   }
@@ -208,9 +292,11 @@ exports[`reference module 1`] = `
 
 exports[`reference module list 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
@@ -218,18 +304,28 @@ exports[`reference module list 1`] = `
       {}
     ]
   },
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
+    }
+  },
   \\"resource\\": {
     \\"test_resource\\": {
       \\"resource\\": {
         \\"name\\": \\"test\\",
-        \\"names\\": \\"\${module.test.names}\\"
-      }
-    }
-  },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+        \\"names\\": \\"\${module.test.names}\\",
+        \\"//\\": {
+          \\"metadata\\": {
+            \\"path\\": \\"test/resource\\",
+            \\"uniqueId\\": \\"resource\\"
+          }
+        }
       }
     }
   }
@@ -238,10 +334,23 @@ exports[`reference module list 1`] = `
 
 exports[`set variable 1`] = `
 "{
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
+    }
+  },
   \\"module\\": {
     \\"test\\": {
       \\"param1\\": \\"value1\\",
-      \\"source\\": \\"../foo\\"
+      \\"source\\": \\"../../../foo\\",
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
+      }
     }
   }
 }"
@@ -249,12 +358,11 @@ exports[`set variable 1`] = `
 
 exports[`simple provider 1`] = `
 "{
-  \\"module\\": {
-    \\"test\\": {
-      \\"providers\\": {
-        \\"test\\": \\"test.provider1\\"
-      },
-      \\"source\\": \\"../foo\\"
+  \\"//\\": {
+    \\"metadata\\": {
+      \\"version\\": \\"stubbed\\",
+      \\"stackName\\": \\"test\\",
+      \\"backend\\": \\"local\\"
     }
   },
   \\"provider\\": {
@@ -265,10 +373,17 @@ exports[`simple provider 1`] = `
       }
     ]
   },
-  \\"terraform\\": {
-    \\"required_providers\\": {
-      \\"test\\": {
-        \\"version\\": \\"~> 2.0\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\",
+      \\"providers\\": {
+        \\"test\\": \\"test.provider1\\"
+      },
+      \\"//\\": {
+        \\"metadata\\": {
+          \\"path\\": \\"test/test\\",
+          \\"uniqueId\\": \\"test\\"
+        }
       }
     }
   }
diff --git a/test/typescript/modules/__snapshots__/test.ts.snap b/test/typescript/modules/__snapshots__/test.ts.snap
index 485e3d05c2..0310ea03ca 100644
--- a/test/typescript/modules/__snapshots__/test.ts.snap
+++ b/test/typescript/modules/__snapshots__/test.ts.snap
@@ -11,7 +11,7 @@ exports[`full integration test build modules posix 1`] = `
   },
   \\"module\\": {
     \\"localmodule\\": {
-      \\"source\\": \\"../local-module\\",
+      \\"source\\": \\"../../../local-module\\",
       \\"//\\": {
         \\"metadata\\": {
           \\"path\\": \\"hello-modules/local-module\\",
@@ -55,7 +55,7 @@ exports[`full integration test build modules windows 1`] = `
   },
   \\"module\\": {
     \\"localmodule\\": {
-      \\"source\\": \\"..\\\\\\\\local-module\\",
+      \\"source\\": \\"..\\\\\\\\..\\\\\\\\..\\\\\\\\local-module\\",
       \\"//\\": {
         \\"metadata\\": {
           \\"path\\": \\"hello-modules/local-module\\",

From dfca7c5c363c123a10b27c68c344dd8c273a4c28 Mon Sep 17 00:00:00 2001
From: Jon Steinich <jsteinich@gmail.com>
Date: Thu, 16 Sep 2021 14:40:57 +0000
Subject: [PATCH 2/2] update snapshot

---
 .../terraform-hcl-module.test.ts.snap         | 291 ++++++------------
 1 file changed, 88 insertions(+), 203 deletions(-)

diff --git a/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap b/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
index a9edebcceb..623d07f016 100644
--- a/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
+++ b/packages/cdktf/test/__snapshots__/terraform-hcl-module.test.ts.snap
@@ -2,25 +2,12 @@
 
 exports[`add provider 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
-    }
-  },
   \\"module\\": {
     \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
       \\"providers\\": {
         \\"test\\": \\"test.provider1\\"
       },
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -30,17 +17,32 @@ exports[`add provider 1`] = `
         \\"alias\\": \\"provider1\\"
       }
     ]
+  },
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
+      }
+    }
   }
 }"
 `;
 
 exports[`complex providers 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"param1\\": \\"name\\",
+      \\"param2\\": 1,
+      \\"param3\\": [
+        \\"id1\\",
+        \\"id2\\"
+      ],
+      \\"providers\\": {
+        \\"test.dst\\": \\"test.provider2\\",
+        \\"test.src\\": \\"test.provider1\\"
+      },
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -55,24 +57,10 @@ exports[`complex providers 1`] = `
       }
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"param1\\": \\"name\\",
-      \\"param2\\": 1,
-      \\"param3\\": [
-        \\"id1\\",
-        \\"id2\\"
-      ],
-      \\"source\\": \\"../../../foo\\",
-      \\"providers\\": {
-        \\"test.src\\": \\"test.provider1\\",
-        \\"test.dst\\": \\"test.provider2\\"
-      },
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }
@@ -81,11 +69,9 @@ exports[`complex providers 1`] = `
 
 exports[`depend on module 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -93,30 +79,20 @@ exports[`depend on module 1`] = `
       {}
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
-    }
-  },
   \\"resource\\": {
     \\"test_resource\\": {
       \\"resource\\": {
-        \\"name\\": \\"foo\\",
         \\"depends_on\\": [
           \\"module.test\\"
         ],
-        \\"//\\": {
-          \\"metadata\\": {
-            \\"path\\": \\"test/resource\\",
-            \\"uniqueId\\": \\"resource\\"
-          }
-        }
+        \\"name\\": \\"foo\\"
+      }
+    }
+  },
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }
@@ -125,34 +101,15 @@ exports[`depend on module 1`] = `
 
 exports[`depend on other module 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
-    }
-  },
   \\"module\\": {
     \\"test_1\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test_1\\",
-          \\"uniqueId\\": \\"test_1\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     },
     \\"test_2\\": {
-      \\"source\\": \\"../../../foo\\",
       \\"depends_on\\": [
         \\"module.test_1\\"
       ],
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test_2\\",
-          \\"uniqueId\\": \\"test_2\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     }
   }
 }"
@@ -160,22 +117,9 @@ exports[`depend on other module 1`] = `
 
 exports[`minimal configuration 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
-    }
-  },
   \\"module\\": {
     \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     }
   }
 }"
@@ -183,37 +127,34 @@ exports[`minimal configuration 1`] = `
 
 exports[`multiple providers 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"providers\\": {
+        \\"differentType\\": \\"differentType\\",
+        \\"test\\": \\"test\\"
+      },
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
-    \\"test\\": [
+    \\"differentType\\": [
       {
         \\"access_key\\": \\"key\\"
       }
     ],
-    \\"differentType\\": [
+    \\"test\\": [
       {
         \\"access_key\\": \\"key\\"
       }
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"providers\\": {
-        \\"test\\": \\"test\\",
-        \\"differentType\\": \\"differentType\\"
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"differentType\\": {
+        \\"version\\": \\"~> 2.0\\"
       },
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }
@@ -222,13 +163,6 @@ exports[`multiple providers 1`] = `
 
 exports[`pass variables 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
-    }
-  },
   \\"module\\": {
     \\"test\\": {
       \\"param1\\": \\"name\\",
@@ -237,13 +171,7 @@ exports[`pass variables 1`] = `
         \\"id1\\",
         \\"id2\\"
       ],
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     }
   }
 }"
@@ -251,11 +179,9 @@ exports[`pass variables 1`] = `
 
 exports[`reference module 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -263,27 +189,17 @@ exports[`reference module 1`] = `
       {}
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
-    }
-  },
   \\"resource\\": {
     \\"test_resource\\": {
       \\"resource\\": {
-        \\"name\\": \\"\${module.test.name}\\",
-        \\"//\\": {
-          \\"metadata\\": {
-            \\"path\\": \\"test/resource\\",
-            \\"uniqueId\\": \\"resource\\"
-          }
-        }
+        \\"name\\": \\"\${module.test.name}\\"
+      }
+    }
+  },
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }
@@ -292,11 +208,9 @@ exports[`reference module 1`] = `
 
 exports[`reference module list 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -304,28 +218,18 @@ exports[`reference module list 1`] = `
       {}
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
-    }
-  },
   \\"resource\\": {
     \\"test_resource\\": {
       \\"resource\\": {
         \\"name\\": \\"test\\",
-        \\"names\\": \\"\${module.test.names}\\",
-        \\"//\\": {
-          \\"metadata\\": {
-            \\"path\\": \\"test/resource\\",
-            \\"uniqueId\\": \\"resource\\"
-          }
-        }
+        \\"names\\": \\"\${module.test.names}\\"
+      }
+    }
+  },
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }
@@ -334,23 +238,10 @@ exports[`reference module list 1`] = `
 
 exports[`set variable 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
-    }
-  },
   \\"module\\": {
     \\"test\\": {
       \\"param1\\": \\"value1\\",
-      \\"source\\": \\"../../../foo\\",
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
-      }
+      \\"source\\": \\"../../../foo\\"
     }
   }
 }"
@@ -358,11 +249,12 @@ exports[`set variable 1`] = `
 
 exports[`simple provider 1`] = `
 "{
-  \\"//\\": {
-    \\"metadata\\": {
-      \\"version\\": \\"stubbed\\",
-      \\"stackName\\": \\"test\\",
-      \\"backend\\": \\"local\\"
+  \\"module\\": {
+    \\"test\\": {
+      \\"providers\\": {
+        \\"test\\": \\"test.provider1\\"
+      },
+      \\"source\\": \\"../../../foo\\"
     }
   },
   \\"provider\\": {
@@ -373,17 +265,10 @@ exports[`simple provider 1`] = `
       }
     ]
   },
-  \\"module\\": {
-    \\"test\\": {
-      \\"source\\": \\"../../../foo\\",
-      \\"providers\\": {
-        \\"test\\": \\"test.provider1\\"
-      },
-      \\"//\\": {
-        \\"metadata\\": {
-          \\"path\\": \\"test/test\\",
-          \\"uniqueId\\": \\"test\\"
-        }
+  \\"terraform\\": {
+    \\"required_providers\\": {
+      \\"test\\": {
+        \\"version\\": \\"~> 2.0\\"
       }
     }
   }