The ValidateAllTerraformModules function doesn't work on all repo structures. In particular, it calls runValidateOnAllTerraformModules under the hood, which tries to find the "git root" through a 100% hard-coded path of "go two folders up": filepath.Abs(filepath.Join(cwd, "../../")). This does not work for any Terraform code that isn't exactly two folders from the Git root! In fact, in a repo I tried this on, the code was just one folder down, and two folders up was a folder with ~100GB of code, so when this function then tried to copy all this to temp, it would hang. This code should use git.GetRepoRoot instead.
(Minor) The ValidateAllTerraformModules method is in the test_structure package. Since this is a function for validating Terraform, it should be in the terraform package.
ValidateAllTerraformModulesfunction doesn't work on all repo structures. In particular, it callsrunValidateOnAllTerraformModulesunder the hood, which tries to find the "git root" through a 100% hard-coded path of "go two folders up":filepath.Abs(filepath.Join(cwd, "../../")). This does not work for any Terraform code that isn't exactly two folders from the Git root! In fact, in a repo I tried this on, the code was just one folder down, and two folders up was a folder with ~100GB of code, so when this function then tried to copy all this to temp, it would hang. This code should usegit.GetRepoRootinstead.ValidateAllTerraformModulesmethod is in thetest_structurepackage. Since this is a function for validating Terraform, it should be in theterraformpackage.