Fixed a bug when calling cursor.executemany() with a PL/SQL statement

and a single row of data (#30).

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -17,6 +17,9 @@ Thin Mode Changes
     supported; an error is now raised only when the first attempt to use
     NCHAR, NVARCHAR2 or NCLOB data is made
     (`issue 16 <https://github.com/oracle/python-oracledb/issues/16>`__).
+#)  Fixed a bug when calling `cursor.executemany()` with a PL/SQL statement and
+    a single row of data
+    (`issue 30 <https://github.com/oracle/python-oracledb/issues/30>`__).
 #)  When using the connection parameter `https_proxy` while using protocol
     `tcp`, a more meaningful exception is now raised:
     `DPY-2029: https_proxy requires use of the tcps protocol`.

src/oracledb/impl/thin/cursor.pyx

@@ -148,7 +148,7 @@ cdef class ThinCursorImpl(BaseCursorImpl):
                 for i in range(num_execs - 1):
                     message.offset = i + 1
                     self._conn_impl._protocol._process_single_message(message)
-            else:
+            elif num_execs > 1:
                 message.offset = 1
                 message.num_execs = num_execs - 1
                 self._conn_impl._protocol._process_single_message(message)

tests/test_4000_cursor_executemany.py

@@ -286,5 +286,13 @@ def test_4019_re_executemany_pl_sql_with_in_and_out_binds(self):
                 end;""", data)
             self.assertEqual(out_bind.values, [5, 8, 17, 24, 6])
 
+    def test_4020_executemany_with_plsql_single_row(self):
+        "4020 - test PL/SQL statement with single row bind"
+        value = 4020
+        var = self.cursor.var(int)
+        data = [[var, value]]
+        self.cursor.executemany("begin :1 := :2; end;", data)
+        self.assertEqual(var.values, [value])
+
 if __name__ == "__main__":
     test_env.run_test_cases()