test: Remove unneeded test code

Author: SarahFrench

internal/command/state_identities_test.go

@@ -448,10 +448,6 @@ func TestStateIdentities_stateStore(t *testing.T) {
 		"default": stateBytes,
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	ui := cli.NewMockUi()
 	c := &StateIdentitiesCommand{
@@ -462,8 +458,7 @@ func TestStateIdentities_stateStore(t *testing.T) {
 					mockProviderAddress: providers.FactoryFixed(mockProvider),
 				},
 			},
-			ProviderSource: providerSource,
-			Ui:             ui,
+			Ui: ui,
 		},
 	}
 

internal/command/state_list_test.go

@@ -203,10 +203,6 @@ func TestStateList_stateStore(t *testing.T) {
 		"default": stateBuf.Bytes(),
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	ui := cli.NewMockUi()
 	c := &StateListCommand{
@@ -217,8 +213,7 @@ func TestStateList_stateStore(t *testing.T) {
 					mockProviderAddress: providers.FactoryFixed(mockProvider),
 				},
 			},
-			ProviderSource: providerSource,
-			Ui:             ui,
+			Ui: ui,
 		},
 	}
 

internal/command/state_mv_test.go

@@ -207,10 +207,6 @@ func TestStateMv_stateStore(t *testing.T) {
 		"default": stateBuf.Bytes(),
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	// Make the mock assert that the resource has been moved when the new state is persisted
 	oldAddr := "test_instance.foo"
@@ -260,7 +256,6 @@ func TestStateMv_stateStore(t *testing.T) {
 	}
 
 	ui := new(cli.MockUi)
-	view, _ := testView(t)
 	c := &StateMvCommand{
 		StateMeta{
 			Meta: Meta{
@@ -270,9 +265,7 @@ func TestStateMv_stateStore(t *testing.T) {
 						mockProviderAddress: providers.FactoryFixed(mockProvider),
 					},
 				},
-				ProviderSource: providerSource,
-				Ui:             ui,
-				View:           view,
+				Ui: ui,
 			},
 		},
 	}

internal/command/state_push_test.go

@@ -58,13 +58,8 @@ func TestStatePush_stateStore(t *testing.T) {
 	// Create a mock that doesn't have any internal states.
 	mockProvider := mockPluggableStateStorageProvider()
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	ui := new(cli.MockUi)
-	view, _ := testView(t)
 	c := &StatePushCommand{
 		Meta: Meta{
 			AllowExperimentalFeatures: true,
@@ -73,9 +68,7 @@ func TestStatePush_stateStore(t *testing.T) {
 					mockProviderAddress: providers.FactoryFixed(mockProvider),
 				},
 			},
-			ProviderSource: providerSource,
-			Ui:             ui,
-			View:           view,
+			Ui: ui,
 		},
 	}
 

internal/command/state_replace_provider_test.go

@@ -337,13 +337,8 @@ func TestStateReplaceProvider_stateStore(t *testing.T) {
 		"default": stateBuf.Bytes(),
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	ui := new(cli.MockUi)
-	view, _ := testView(t)
 	c := &StateReplaceProviderCommand{
 		StateMeta{
 			Meta: Meta{
@@ -353,9 +348,7 @@ func TestStateReplaceProvider_stateStore(t *testing.T) {
 						mockProviderAddress: providers.FactoryFixed(mockProvider),
 					},
 				},
-				ProviderSource: providerSource,
-				Ui:             ui,
-				View:           view,
+				Ui: ui,
 			},
 		},
 	}

internal/command/state_rm_test.go

@@ -135,10 +135,6 @@ func TestStateRm_stateStore(t *testing.T) {
 		"default": stateBuf.Bytes(),
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	// Make the mock assert that the removed resource is not present when the new state is persisted
 	keptResource := "test_instance.bar"
@@ -161,7 +157,6 @@ func TestStateRm_stateStore(t *testing.T) {
 	}
 
 	ui := new(cli.MockUi)
-	view, _ := testView(t)
 	c := &StateRmCommand{
 		StateMeta{
 			Meta: Meta{
@@ -171,9 +166,7 @@ func TestStateRm_stateStore(t *testing.T) {
 						mockProviderAddress: providers.FactoryFixed(mockProvider),
 					},
 				},
-				ProviderSource: providerSource,
-				Ui:             ui,
-				View:           view,
+				Ui: ui,
 			},
 		},
 	}

internal/command/state_show_test.go

@@ -376,10 +376,6 @@ func TestStateShow_stateStore(t *testing.T) {
 		"default": stateBuf.Bytes(),
 	}
 	mockProviderAddress := addrs.NewDefaultProvider("test")
-	providerSource, close := newMockProviderSource(t, map[string][]string{
-		"hashicorp/test": {"1.0.0"},
-	})
-	defer close()
 
 	ui := cli.NewMockUi()
 	streams, done := terminal.StreamsForTesting(t)
@@ -391,9 +387,8 @@ func TestStateShow_stateStore(t *testing.T) {
 					mockProviderAddress: providers.FactoryFixed(mockProvider),
 				},
 			},
-			ProviderSource: providerSource,
-			Ui:             ui,
-			Streams:        streams,
+			Ui:      ui,
+			Streams: streams,
 		},
 	}