clarify existing job name/type

hashicorp · Sep 20, 2023 · 07b4e9b · 07b4e9b
1 parent a9c0847
commit 07b4e9b
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
diff --git a/internal/indexer/document_change.go b/internal/indexer/document_change.go
@@ -151,9 +151,9 @@ func (idx *Indexer) decodeModule(ctx context.Context, modHandle document.DirHand
 		_, err = idx.jobStore.EnqueueJob(ctx, job.Job{
 			Dir: modHandle,
 			Func: func(ctx context.Context) error {
-				return module.SchemaValidation(ctx, idx.modStore, idx.schemaStore, modHandle.Path())
+				return module.SchemaModuleValidation(ctx, idx.modStore, idx.schemaStore, modHandle.Path())
 			},
-			Type:        op.OpTypeSchemaValidation.String(),
+			Type:        op.OpTypeSchemaModuleValidation.String(),
 			DependsOn:   job.IDs{metaId},
 			IgnoreState: ignoreState,
 		})

diff --git a/internal/terraform/module/module_ops.go b/internal/terraform/module/module_ops.go
@@ -668,7 +668,7 @@ func DecodeVarsReferences(ctx context.Context, modStore *state.ModuleStore, sche
 	return rErr
 }
 
-func SchemaValidation(ctx context.Context, modStore *state.ModuleStore, schemaReader state.SchemaReader, modPath string) error {
+func SchemaModuleValidation(ctx context.Context, modStore *state.ModuleStore, schemaReader state.SchemaReader, modPath string) error {
 	mod, err := modStore.ModuleByPath(modPath)
 	if err != nil {
 		return err

diff --git a/internal/terraform/module/module_ops_test.go b/internal/terraform/module/module_ops_test.go
@@ -998,7 +998,7 @@ func TestSchemaValidation_FullModule(t *testing.T) {
 		URI:    "file:///test/variables.tf",
 	})
 	ctx = lsctx.WithLanguageId(ctx, ilsp.Terraform.String())
-	err = SchemaValidation(ctx, ss.Modules, ss.ProviderSchemas, modPath)
+	err = SchemaModuleValidation(ctx, ss.Modules, ss.ProviderSchemas, modPath)
 	if err != nil {
 		t.Fatal(err)
 	}
@@ -1043,7 +1043,7 @@ func TestSchemaValidation_SingleFile(t *testing.T) {
 		URI:    "file:///test/variables.tf",
 	})
 	ctx = lsctx.WithLanguageId(ctx, ilsp.Terraform.String())
-	err = SchemaValidation(ctx, ss.Modules, ss.ProviderSchemas, modPath)
+	err = SchemaModuleValidation(ctx, ss.Modules, ss.ProviderSchemas, modPath)
 	if err != nil {
 		t.Fatal(err)
 	}

diff --git a/internal/terraform/module/operation/op_type_string.go b/internal/terraform/module/operation/op_type_string.go
diff --git a/internal/terraform/module/operation/operation.go b/internal/terraform/module/operation/operation.go
@@ -30,7 +30,7 @@ const (
 	OpTypeGetModuleDataFromRegistry
 	OpTypeParseProviderVersions
 	OpTypePreloadEmbeddedSchema
-	OpTypeSchemaValidation
+	OpTypeSchemaModuleValidation
 	OpTypeSchemaVarsValidation
 	OpTypeReferenceValidation
 	OpTypeTerraformValidate