test: assert delete_table raises on version_token mismatch

moto/s3tables/models.py

@@ -406,7 +406,7 @@ def list_tables(
         return tables, next_continuation_token
 
     def delete_table(
-        self, table_bucket_arn: str, namespace: str, name: str, version_token: str
+        self, table_bucket_arn: str, namespace: str, name: str, version_token: Optional[str] = None
     ) -> None:
         bucket = self.table_buckets.get(table_bucket_arn)
         if (
@@ -418,6 +418,8 @@ def delete_table(
 
         ns = bucket.namespaces[namespace]
         table = ns.tables[name]
+        if version_token and not version_token == table.version_token:
+            raise VersionTokenMismatch()
         from moto.s3.models import s3_backends
 
         s3_backends[self.account_id][self.partition].delete_table_storage_bucket(

moto/s3tables/responses.py

@@ -247,8 +247,7 @@ def delete_table(self) -> TYPE_RESPONSE:
         _, table_bucket_arn, namespace, name = self.raw_path.lstrip("/").split("/")
         table_bucket_arn = unquote(table_bucket_arn)
         params = self._get_params()
-        version_token = params.get("versionToken", "")
-        # TODO: if versinToken not supplied, raise exception
+        version_token = params.get("versionToken")
         self.s3tables_backend.delete_table(
             table_bucket_arn=table_bucket_arn,
             namespace=namespace,

tests/test_s3tables/test_s3tables.py

@@ -236,6 +236,21 @@ def test_delete_table_deletes_underlying_table_storage():
         s3.head_bucket(Bucket=bucket_name)
 
 
+@mock_aws
+def test_delete_table_fails_on_version_token_mismatch():
+    client = boto3.client("s3tables", region_name="us-east-2")
+    arn = client.create_table_bucket(name="foo")["arn"]
+    client.create_namespace(tableBucketARN=arn, namespace=["bar"])
+    resp = client.create_table(
+        tableBucketARN=arn, namespace="bar", name="baz", format="ICEBERG"
+    )
+    token = resp["versionToken"]
+
+
+    with pytest.raises(client.exceptions.ConflictException):
+        client.delete_table(tableBucketARN=arn, namespace="bar", name="baz", versionToken=f"{token}-foo")
+
+
 @mock_aws
 def test_get_table_metadata_location():
     client = boto3.client("s3tables", region_name="us-east-2")