fix: Removes delete partition_fields statements. (#1499)

* [bug] Removes delete partition_fields statements. * changes test expectations. * tries to add a 'migration' test to verify no breaking changes between versions. * removes migration test as we should write this separately as part of our testing refactoring.
mongodb · Oct 4, 2023 · ce1ad7a · ce1ad7a
1 parent 726261d
commit ce1ad7a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/mongodbatlas/resource_mongodbatlas_online_archive.go b/mongodbatlas/resource_mongodbatlas_online_archive.go
@@ -513,7 +513,6 @@ func fromOnlineArchiveToMap(in *matlas.OnlineArchive) map[string]interface{} {
 
 func fromOnlineArchiveToMapInCreate(in *matlas.OnlineArchive) map[string]interface{} {
 	localSchema := fromOnlineArchiveToMap(in)
-	delete(localSchema, "partition_fields")
 	return localSchema
 }
 

diff --git a/mongodbatlas/resource_mongodbatlas_online_archive_test.go b/mongodbatlas/resource_mongodbatlas_online_archive_test.go
@@ -98,6 +98,12 @@ func TestAccBackupRSOnlineArchive(t *testing.T) {
 					resource.TestCheckNoResourceAttr(onlineArchiveResourceName, "schedule.#"),
 				),
 			},
+			{
+				Config: testAccBackupRSOnlineArchiveConfigWithoutSchedule(orgID, projectName, name),
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr(onlineArchiveResourceName, "partition_fields.0.field_name", "last_review"),
+				),
+			},
 		},
 	})
 }