Fix: wrap fail.items() in list() to prevent Py2->Py3 dict mutation error

In Python 2, dict.items() returned a list copy; in Python 3 it returns
a view, so calling fail.pop(key) inside the loop raises RuntimeError.
Fixed in batch_create, batch_update, and batch_replace.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Change-Id: I38627dfab8cec6e3243fa0960f2b309e89759fce
Reviewed-on: https://review.couchbase.org/c/TAF/+/245679
Tested-by: Ashwin <ashwin.govindarajulu@couchbase.com>
Reviewed-by: Ashwin <ashwin.govindarajulu@couchbase.com>

Author: 2 people

lib/Jython_tasks/task.py

@@ -1099,7 +1099,7 @@ def batch_create(self, key_val, client=None, persist_to=0,
                     # self.test_log.debug("Reading values {0} after failure"
                     # .format(fail.keys()))
                     read_map, read_fail = self.batch_read(list(fail.keys()))
-                    for key, value in fail.items():
+                    for key, value in list(fail.items()):
                         if key in read_map and read_map[key]["cas"] != 0:
                             success[key] = value
                             success[key].pop("error", None)
@@ -1147,7 +1147,7 @@ def batch_update(self, key_val, client=None, persist_to=0,
                     self.test_log.debug("Reading values {0} after failure"
                                         .format(fail.keys()))
                     read_map, _ = self.batch_read(list(fail.keys()))
-                    for key, value in fail.items():
+                    for key, value in list(fail.items()):
                         if key in read_map and read_map[key]["cas"] != 0 \
                                 and value == read_map[key]["value"]:
                             success[key] = value
@@ -1197,7 +1197,7 @@ def batch_replace(self, key_val, client=None, persist_to=0,
                     self.test_log.debug("Reading values {0} after failure"
                                         .format(fail.keys()))
                     read_map, _ = self.batch_read(list(fail.keys()))
-                    for key, value in fail.items():
+                    for key, value in list(fail.items()):
                         if key in read_map and read_map[key]["cas"] != 0:
                             success[key] = value
                             success[key].pop("error")