had to remove without state converter

Emyrk · Emyrk · commit 743e3b814ea9 · 2025-11-20T12:27:08.000-06:00
The graph is required for daily cost information
diff --git a/provisioner/terraform/convertstate.go b/provisioner/terraform/convertstate.go
@@ -24,27 +24,6 @@ import (
 	"github.com/coder/coder/v2/provisionersdk/proto"
 )
 
-type ReducedState struct {
-	Resources []*proto.Resource
-}
-
-func ConvertStateWithoutGraph(ctx context.Context, modules []*tfjson.StateModule, logger slog.Logger) (*ReducedState, error) {
-	converter := newConverter(ctx, logger, gographviz.NewGraph())
-
-	// Index all resources from the state.
-	converter.indexTerraformResources(modules)
-
-	// Associate agent instance IDs.
-	converter.processAgentInstances()
-
-	// Build the final resource list.
-	converter.buildResources()
-
-	return &ReducedState{
-		Resources: converter.resources,
-	}, nil
-}
-
 // ConvertState consumes Terraform state and a GraphViz representation
 // produced by `terraform graph` to produce resources consumable by Coder.
 func ConvertState(ctx context.Context, modules []*tfjson.StateModule, rawGraph string, logger slog.Logger) (*State, error) {
diff --git a/provisioner/terraform/resources_test.go b/provisioner/terraform/resources_test.go
@@ -128,40 +128,6 @@ func TestConvertStateGolden(t *testing.T) {
 				require.NoError(t, err)
 
 				require.JSONEq(t, string(expBytes), string(gotBytes), "converted state")
-
-				if expectedState, ok := expectedOutput.(*terraform.State); ok {
-					t.Run("WithoutGraph", func(t *testing.T) {
-						gotState, err := terraform.ConvertStateWithoutGraph(ctx, modules, logger)
-						require.NoError(t, err)
-						sortResources(gotState.Resources)
-
-						for _, res := range expectedState.Resources {
-							// Agents will not exist without the graph.
-							original := res.Agents
-							t.Cleanup(func() {
-								res.Agents = original
-							})
-							res.Agents = nil
-
-							// Metadata will not exist without the graph.
-							originalMetadata := res.Metadata
-							t.Cleanup(func() {
-								res.Metadata = originalMetadata
-							})
-							res.Metadata = nil
-						}
-
-						require.Equal(t, len(expectedState.Resources), len(gotState.Resources), "number of resources")
-						for i := range gotState.Resources {
-							got, err := json.Marshal(gotState.Resources[i])
-							require.NoError(t, err, "marshaling got resource to JSON")
-							expected, err := json.Marshal(expectedState.Resources[i])
-							require.NoError(t, err, "marshaling expected resource to JSON")
-
-							require.JSONEq(t, string(expected), string(got), "json cmp")
-						}
-					})
-				}
 			})
 		}
 	}
