debug

.github/workflows/ci.yml

@@ -94,6 +94,9 @@ jobs:
       -
         name: Run go fmt
         run: go run github.com/mh-cbon/go-fmt-fail ./...
+      -
+        name: Run verbose test / debug
+        run: go test -cover -covermode=atomic -timeout=5m -race ./internal/terraform/module -run=TestParseProviderVersions_multipleVersions -v
       -
         name: Run tests
         run: go test -cover -covermode=atomic -timeout=5m -race ./...

internal/terraform/module/module_ops.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"log"
 
 	"github.com/hashicorp/go-multierror"
 	"github.com/hashicorp/go-version"
@@ -137,6 +138,7 @@ func ObtainSchema(ctx context.Context, modStore *state.ModuleStore, schemaStore
 		return err
 	}
 	if exist {
+		log.Printf("all schemas already exist: %#v", pReqs)
 		// avoid obtaining schemas if we already have it
 		return nil
 	}
@@ -159,6 +161,8 @@ func ObtainSchema(ctx context.Context, modStore *state.ModuleStore, schemaStore
 		return err
 	}
 
+	log.Printf("received schemas: %#v", ps)
+
 	for rawAddr, pJsonSchema := range ps.Schemas {
 		pAddr, err := tfaddr.ParseProviderSource(rawAddr)
 		if err != nil {
@@ -202,6 +206,7 @@ func ParseModuleConfiguration(ctx context.Context, fs ReadOnlyFS, modStore *stat
 	}
 
 	files, diags, err := parser.ParseModuleFiles(fs, modPath)
+	log.Printf("parsed module files: %#v, %s\nfiles (%d): %#v", err, diags, len(files), files)
 
 	sErr := modStore.UpdateParsedModuleFiles(modPath, files, err)
 	if sErr != nil {
@@ -313,6 +318,8 @@ func ParseProviderVersions(ctx context.Context, fs ReadOnlyFS, modStore *state.M
 
 	pvm, err := datadir.ParsePluginVersions(fs, modPath)
 
+	log.Printf("storing provider versions: %#v", pvm)
+
 	sErr := modStore.UpdateInstalledProviders(modPath, pvm, err)
 	if sErr != nil {
 		return sErr
@@ -359,6 +366,8 @@ func LoadModuleMetadata(ctx context.Context, modStore *state.ModuleStore, modPat
 	}
 	meta.ProviderReferences = providerRefs
 
+	log.Printf("storing module meta: %#v", meta)
+
 	sErr := modStore.UpdateMetadata(modPath, meta, mErr)
 	if sErr != nil {
 		return sErr

internal/terraform/module/module_ops_test.go

@@ -421,7 +421,7 @@ func TestParseProviderVersions_multipleVersions(t *testing.T) {
 }
 `),
 		},
-		filepath.Join(modPathFirst, "main.tf"): &fstest.MapFile{
+		modPathFirst + "/main.tf": &fstest.MapFile{
 			Data: []byte(`terraform {
 	required_providers {
 		aws = {
@@ -455,7 +455,7 @@ func TestParseProviderVersions_multipleVersions(t *testing.T) {
 }
 `),
 		},
-		filepath.Join(modPathSecond, "main.tf"): &fstest.MapFile{
+		modPathSecond + "/main.tf": &fstest.MapFile{
 			Data: []byte(`terraform {
 	required_providers {
 		aws = {

internal/terraform/parser/module.go

@@ -1,6 +1,7 @@
 package parser
 
 import (
+	"log"
 	"path/filepath"
 
 	"github.com/hashicorp/terraform-ls/internal/terraform/ast"
@@ -15,6 +16,8 @@ func ParseModuleFiles(fs FS, modPath string) (ast.ModFiles, ast.ModDiags, error)
 		return nil, nil, err
 	}
 
+	log.Printf("dir entries (%d): %#v", len(infos), infos)
+
 	for _, info := range infos {
 		if info.IsDir() {
 			// We only care about files