Fixed bug when attempting to append an element to a DbObject which is

not actually a collection.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -66,6 +66,8 @@ Common Changes
     (`issue 525 <https://github.com/oracle/python-oracledb/issues/525>`__).
 #)  Fixed bug when attempting to convert an integer that cannot be represented
     as a native C ``int`` value to an Arrow data frame.
+#)  Fixed bug when attempting to append an element to a
+    :ref:`DbObject <dbobjecttype>` which is not actually a collection.
 #)  API documentation is now generated from the source code.
 #)  Internal change: typing_extensions is now a dependency.
 

src/oracledb/dbobject.py

@@ -91,6 +91,7 @@ def append(self, element: Any) -> None:
         creates an element immediately following the highest index available in
         the collection.
         """
+        self._ensure_is_collection()
         self._impl.append(element)
 
     def asdict(self) -> dict:

tests/test_2300_object_var.py

@@ -939,6 +939,39 @@ def test_2347(self):
         expected_data = [[4, 8], None, [1, 3, 5], None, [2, 6, 10, 7, 9]]
         self.assertEqual(plain_obj, expected_data)
 
+    def test_2348(self):
+        "2348 - test using collection methods on an object that is not one"
+        obj_type = self.conn.gettype("UDT_OBJECT")
+        obj = obj_type.newobject()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.append(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.asdict()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.aslist()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.delete(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.exists(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.extend([5])
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.first()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.getelement(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.last()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.next(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.prev(5)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.setelement(5, None)
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.size()
+        with self.assertRaisesFullCode("DPY-2036"):
+            obj.trim(0)
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()