don't explicitly specify defaults in argument parsing

internal/command/arguments/state_mv.go

@@ -51,9 +51,7 @@ type StateMv struct {
 // representing the best effort interpretation of the arguments.
 func ParseStateMv(args []string) (*StateMv, tfdiags.Diagnostics) {
 	var diags tfdiags.Diagnostics
-	mv := &StateMv{
-		StateLock: true,
-	}
+	mv := &StateMv{}
 
 	cmdFlags := defaultFlagSet("state mv")
 	cmdFlags.BoolVar(&mv.DryRun, "dry-run", false, "dry run")

internal/command/arguments/state_mv_test.go

@@ -18,31 +18,22 @@ func TestParseStateMv_valid(t *testing.T) {
 		"addresses only": {
 			[]string{"test_instance.foo", "test_instance.bar"},
 			&StateMv{
-				DryRun:              false,
-				BackupPath:          "-",
-				BackupOutPath:       "-",
-				StateLock:           true,
-				StateLockTimeout:    0,
-				StatePath:           "",
-				StateOutPath:        "",
-				IgnoreRemoteVersion: false,
-				SourceAddr:          "test_instance.foo",
-				DestAddr:            "test_instance.bar",
+				BackupPath:    "-",
+				BackupOutPath: "-",
+				StateLock:     true,
+				SourceAddr:    "test_instance.foo",
+				DestAddr:      "test_instance.bar",
 			},
 		},
 		"dry run": {
 			[]string{"-dry-run", "test_instance.foo", "test_instance.bar"},
 			&StateMv{
-				DryRun:              true,
-				BackupPath:          "-",
-				BackupOutPath:       "-",
-				StateLock:           true,
-				StateLockTimeout:    0,
-				StatePath:           "",
-				StateOutPath:        "",
-				IgnoreRemoteVersion: false,
-				SourceAddr:          "test_instance.foo",
-				DestAddr:            "test_instance.bar",
+				DryRun:        true,
+				BackupPath:    "-",
+				BackupOutPath: "-",
+				StateLock:     true,
+				SourceAddr:    "test_instance.foo",
+				DestAddr:      "test_instance.bar",
 			},
 		},
 		"all options": {