From 7736b3c5a4384e0406ccadbe80871eaf0be04853 Mon Sep 17 00:00:00 2001
From: Noah Stapp <noah.stapp@mongodb.com>
Date: Wed, 30 Jul 2025 13:47:26 -0400
Subject: [PATCH] Remove skipped modifyCollection tests

---
 .evergreen/remove-unimplemented-tests.sh      |   3 +
 .../modifyCollection-errorResponse.json       | 118 ------------------
 .../modifyCollection-pre_and_post_images.json | 111 ----------------
 3 files changed, 3 insertions(+), 229 deletions(-)
 delete mode 100644 test/collection_management/modifyCollection-errorResponse.json
 delete mode 100644 test/collection_management/modifyCollection-pre_and_post_images.json

diff --git a/.evergreen/remove-unimplemented-tests.sh b/.evergreen/remove-unimplemented-tests.sh
index e75bb947c..92685ab2b 100755
--- a/.evergreen/remove-unimplemented-tests.sh
+++ b/.evergreen/remove-unimplemented-tests.sh
@@ -41,6 +41,9 @@ rm $PYMONGO/test/crud/unified/updateMany-rawdata.json
 rm $PYMONGO/test/crud/unified/updateOne-rawdata.json
 rm $PYMONGO/test/index_management/index-rawdata.json
 
+# PyMongo does not support modifyCollection
+rm $PYMONGO/test/collection_management/modifyCollection-*.json
+
 # PYTHON-5248 - Remove support for MongoDB 4.0
 rm $PYMONGO/test/**/pre-42-*.json
 
diff --git a/test/collection_management/modifyCollection-errorResponse.json b/test/collection_management/modifyCollection-errorResponse.json
deleted file mode 100644
index aea71eb08..000000000
--- a/test/collection_management/modifyCollection-errorResponse.json
+++ /dev/null
@@ -1,118 +0,0 @@
-{
-  "description": "modifyCollection-errorResponse",
-  "schemaVersion": "1.12",
-  "createEntities": [
-    {
-      "client": {
-        "id": "client0",
-        "observeEvents": [
-          "commandStartedEvent"
-        ]
-      }
-    },
-    {
-      "database": {
-        "id": "database0",
-        "client": "client0",
-        "databaseName": "collMod-tests"
-      }
-    },
-    {
-      "collection": {
-        "id": "collection0",
-        "database": "database0",
-        "collectionName": "test"
-      }
-    }
-  ],
-  "initialData": [
-    {
-      "collectionName": "test",
-      "databaseName": "collMod-tests",
-      "documents": [
-        {
-          "_id": 1,
-          "x": 1
-        },
-        {
-          "_id": 2,
-          "x": 1
-        }
-      ]
-    }
-  ],
-  "tests": [
-    {
-      "description": "modifyCollection prepareUnique violations are accessible",
-      "runOnRequirements": [
-        {
-          "minServerVersion": "5.2"
-        }
-      ],
-      "operations": [
-        {
-          "name": "createIndex",
-          "object": "collection0",
-          "arguments": {
-            "keys": {
-              "x": 1
-            }
-          }
-        },
-        {
-          "name": "modifyCollection",
-          "object": "database0",
-          "arguments": {
-            "collection": "test",
-            "index": {
-              "keyPattern": {
-                "x": 1
-              },
-              "prepareUnique": true
-            }
-          }
-        },
-        {
-          "name": "insertOne",
-          "object": "collection0",
-          "arguments": {
-            "document": {
-              "_id": 3,
-              "x": 1
-            }
-          },
-          "expectError": {
-            "errorCode": 11000
-          }
-        },
-        {
-          "name": "modifyCollection",
-          "object": "database0",
-          "arguments": {
-            "collection": "test",
-            "index": {
-              "keyPattern": {
-                "x": 1
-              },
-              "unique": true
-            }
-          },
-          "expectError": {
-            "isClientError": false,
-            "errorCode": 359,
-            "errorResponse": {
-              "violations": [
-                {
-                  "ids": [
-                    1,
-                    2
-                  ]
-                }
-              ]
-            }
-          }
-        }
-      ]
-    }
-  ]
-}
diff --git a/test/collection_management/modifyCollection-pre_and_post_images.json b/test/collection_management/modifyCollection-pre_and_post_images.json
deleted file mode 100644
index 8026faeb1..000000000
--- a/test/collection_management/modifyCollection-pre_and_post_images.json
+++ /dev/null
@@ -1,111 +0,0 @@
-{
-  "description": "modifyCollection-pre_and_post_images",
-  "schemaVersion": "1.4",
-  "runOnRequirements": [
-    {
-      "minServerVersion": "6.0",
-      "serverless": "forbid"
-    }
-  ],
-  "createEntities": [
-    {
-      "client": {
-        "id": "client0",
-        "observeEvents": [
-          "commandStartedEvent"
-        ]
-      }
-    },
-    {
-      "database": {
-        "id": "database0",
-        "client": "client0",
-        "databaseName": "papi-tests"
-      }
-    },
-    {
-      "collection": {
-        "id": "collection0",
-        "database": "database0",
-        "collectionName": "test"
-      }
-    }
-  ],
-  "tests": [
-    {
-      "description": "modifyCollection to changeStreamPreAndPostImages enabled",
-      "operations": [
-        {
-          "name": "dropCollection",
-          "object": "database0",
-          "arguments": {
-            "collection": "test"
-          }
-        },
-        {
-          "name": "createCollection",
-          "object": "database0",
-          "arguments": {
-            "collection": "test",
-            "changeStreamPreAndPostImages": {
-              "enabled": false
-            }
-          }
-        },
-        {
-          "name": "assertCollectionExists",
-          "object": "testRunner",
-          "arguments": {
-            "databaseName": "papi-tests",
-            "collectionName": "test"
-          }
-        },
-        {
-          "name": "modifyCollection",
-          "object": "database0",
-          "arguments": {
-            "collection": "test",
-            "changeStreamPreAndPostImages": {
-              "enabled": true
-            }
-          }
-        }
-      ],
-      "expectEvents": [
-        {
-          "client": "client0",
-          "events": [
-            {
-              "commandStartedEvent": {
-                "command": {
-                  "drop": "test"
-                },
-                "databaseName": "papi-tests"
-              }
-            },
-            {
-              "commandStartedEvent": {
-                "command": {
-                  "create": "test",
-                  "changeStreamPreAndPostImages": {
-                    "enabled": false
-                  }
-                }
-              }
-            },
-            {
-              "commandStartedEvent": {
-                "command": {
-                  "collMod": "test",
-                  "changeStreamPreAndPostImages": {
-                    "enabled": true
-                  }
-                }
-              }
-            }
-          ]
-        }
-      ]
-    }
-  ]
-}