[Tables] Fix a bug in submit_transaction() with empty operation list (Azure#31482)

Author: Yalin Li

sdk/tables/azure-data-tables/CHANGELOG.md

@@ -8,6 +8,7 @@
 ### Breaking Changes
 
 ### Bugs Fixed
+* Fixed a bug when submitting transactions with an empty operation list. ([#31471](https://github.com/Azure/azure-sdk-for-python/issues/31471))
 
 ### Other Changes
 * Bumped minimum dependency on `azure-core` to `>=1.27.1`. ([#28918](https://github.com/Azure/azure-sdk-for-python/issues/28918))

sdk/tables/azure-data-tables/azure/data/tables/_table_client.py

@@ -658,4 +658,10 @@ def submit_transaction(self, operations: Iterable[TransactionOperationType], **k
                 "The value of 'operations' must be an iterator "
                 "of Tuples. Please check documentation for correct Tuple format."
             ) from exc
-        return self._batch_send(self.table_name, *batched_requests.requests, **kwargs)  # type: ignore
+
+        try:
+            return self._batch_send(self.table_name, *batched_requests.requests, **kwargs)  # type: ignore
+        except HttpResponseError as ex:
+            if ex.status_code == 400 and not batched_requests.requests:
+                return []
+            raise

sdk/tables/azure-data-tables/azure/data/tables/aio/_table_client_async.py

@@ -682,4 +682,9 @@ async def submit_transaction(
                     "of Tuples. Please check documentation for correct Tuple format."
                 ) from exc
 
-        return await self._batch_send(self.table_name, *batched_requests.requests, **kwargs)
+        try:
+            return await self._batch_send(self.table_name, *batched_requests.requests, **kwargs)
+        except HttpResponseError as ex:
+            if ex.status_code == 400 and not batched_requests.requests:
+                return []
+            raise

sdk/tables/azure-data-tables/tests/test_table_batch.py

@@ -965,6 +965,19 @@ def test_batch_with_specialchar_partitionkey(self, tables_storage_account_name,
         finally:
             self._tear_down()
 
+    # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
+    @pytest.mark.live_test_only
+    @tables_decorator
+    @recorded_by_proxy
+    def test_empty_batch(self, tables_storage_account_name, tables_primary_storage_account_key):
+        url = self.account_url(tables_storage_account_name, "table")
+        table_name = self.get_resource_name("mytable")
+        with TableClient(url, table_name, credential=tables_primary_storage_account_key) as client:
+            client.create_table()
+            result = client.submit_transaction([])
+            assert result == []
+            client.delete_table()
+
     # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
     @pytest.mark.live_test_only
     @tables_decorator

sdk/tables/azure-data-tables/tests/test_table_batch_async.py

@@ -889,6 +889,19 @@ async def generate_entities(count):
         finally:
             await self._tear_down()
 
+    # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
+    @pytest.mark.live_test_only
+    @tables_decorator_async
+    @recorded_by_proxy_async
+    async def test_empty_batch(self, tables_storage_account_name, tables_primary_storage_account_key):
+        url = self.account_url(tables_storage_account_name, "table")
+        table_name = self.get_resource_name("mytable")
+        async with TableClient(url, table_name, credential=tables_primary_storage_account_key) as client:
+            await client.create_table()
+            result = await client.submit_transaction([])
+            assert result == []
+            await client.delete_table()
+
     # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
     @pytest.mark.live_test_only
     @tables_decorator_async

sdk/tables/azure-data-tables/tests/test_table_batch_cosmos.py

@@ -690,6 +690,19 @@ def test_batch_with_specialchar_partitionkey(self, tables_cosmos_account_name, t
         finally:
             self._tear_down()
 
+    # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
+    @pytest.mark.live_test_only
+    @cosmos_decorator
+    @recorded_by_proxy
+    def test_empty_batch(self, tables_cosmos_account_name, tables_primary_cosmos_account_key):
+        url = self.account_url(tables_cosmos_account_name, "cosmos")
+        table_name = self.get_resource_name("mytable")
+        with TableClient(url, table_name, credential=tables_primary_cosmos_account_key) as client:
+            # client.create_table()
+            result = client.submit_transaction([])
+            assert result == []
+            client.delete_table()
+
     # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
     @pytest.mark.live_test_only
     @cosmos_decorator

sdk/tables/azure-data-tables/tests/test_table_batch_cosmos_async.py

@@ -797,6 +797,19 @@ async def generate_entities(count):
         finally:
             await self._tear_down()
 
+    # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
+    @pytest.mark.live_test_only
+    @cosmos_decorator_async
+    @recorded_by_proxy_async
+    async def test_empty_batch(self, tables_cosmos_account_name, tables_primary_cosmos_account_key):
+        url = self.account_url(tables_cosmos_account_name, "cosmos")
+        table_name = self.get_resource_name("mytable")
+        async with TableClient(url, table_name, credential=tables_primary_cosmos_account_key) as client:
+            await client.create_table()
+            result = await client.submit_transaction([])
+            assert result == []
+            await client.delete_table()
+
     # Playback doesn't work as test proxy issue: https://github.com/Azure/azure-sdk-tools/issues/2900
     @pytest.mark.live_test_only
     @cosmos_decorator_async