Fixed a bug resulting in a ValueError exception when getting

attribute "MessageProperties.enqtime" if the value is not available or
None.

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -43,6 +43,8 @@ Thick Mode Changes
     :attr:`DeqOptions.msgid`, :attr:`DeqOptions.transformation`,
     :attr:`EnqOptions.transformation`, :attr:`MessageProperties.correlation`,
     or :attr:`MessageProperties.exceptionq` are set to ``None``.
+#)  Fixed a bug resulting in a ``ValueError`` exception when getting attribute
+    :attr:`MessageProperties.enqtime` if the value is not available or None.
 
 Common Changes
 ++++++++++++++

src/oracledb/impl/thick/queue.pyx

@@ -400,6 +400,7 @@ cdef class ThickMsgPropsImpl(BaseMsgPropsImpl):
     cdef:
         dpiMsgProps* _handle
         ThickConnImpl _conn_impl
+        bint _has_been_dequeued
 
     def __dealloc__(self):
         if self._handle != NULL:
@@ -414,6 +415,7 @@ cdef class ThickMsgPropsImpl(BaseMsgPropsImpl):
             dpiJsonNode *node
             dpiJson *json
 
+        self._has_been_dequeued = True
         self._conn_impl = queue_impl._conn_impl
         if queue_impl.is_json:
             if dpiMsgProps_getPayloadJson(self._handle, &json) < 0:
@@ -480,12 +482,13 @@ cdef class ThickMsgPropsImpl(BaseMsgPropsImpl):
         Internal method for getting the enqueue time.
         """
         cdef dpiTimestamp timestamp
-        if dpiMsgProps_getEnqTime(self._handle, &timestamp) < 0:
-            _raise_from_odpi()
-        return cydatetime.datetime_new(timestamp.year, timestamp.month,
-                                       timestamp.day, timestamp.hour,
-                                       timestamp.minute, timestamp.second,
-                                       timestamp.fsecond // 1000, None)
+        if self._has_been_dequeued:
+            if dpiMsgProps_getEnqTime(self._handle, &timestamp) < 0:
+                _raise_from_odpi()
+            return cydatetime.datetime_new(timestamp.year, timestamp.month,
+                                           timestamp.day, timestamp.hour,
+                                           timestamp.minute, timestamp.second,
+                                           timestamp.fsecond // 1000, None)
 
     def get_exception_queue(self):
         """

tests/test_7800_aq_raw.py

@@ -140,6 +140,7 @@ def test_7806(self):
         self.__verify_attr(props, "priority", 1)
         self.assertEqual(props.state, oracledb.MSG_READY)
         self.assertEqual(props.deliverymode, 0)
+        self.assertIsNone(props.enqtime)
 
     def test_7807(self):
         "7807 - test enqueue visibility option - ENQ_ON_COMMIT"