test(spp_dci_client): assert outgoing-log integration at the log-service seam (#443)

spp_dci_client/README.rst

@@ -140,6 +140,22 @@ Dependencies
 Changelog
 =========
 
+19.0.2.0.2
+~~~~~~~~~~
+
+- test(dci_client): make the outgoing-log integration tests hold on a
+  database that has ``spp_api_v2`` installed, the only place they run at
+  all (they skip without ``spp.api.outgoing.log``, so they had never
+  executed before the full-stack CI). The client writes the audit row
+  through a separate, committed cursor so it survives the request's
+  rollback; a test transaction at REPEATABLE READ cannot see that row
+  and instead found an earlier class's committed row, so every status
+  assertion compared against a stale success entry. The tests now assert
+  at the log-service seam, one test reads a real row back through a
+  fresh cursor and removes it, and the 401-retry test builds a valid
+  OAuth2 source and asserts the two log calls in the order the code
+  guarantees. No behaviour change (#443)
+
 19.0.2.0.0
 ~~~~~~~~~~
 

spp_dci_client/__manifest__.py

@@ -2,7 +2,7 @@
 {
     "name": "OpenSPP DCI Client",
     "summary": "Base DCI client infrastructure with OAuth2 and data source management",
-    "version": "19.0.2.0.1",
+    "version": "19.0.2.0.2",
     "category": "OpenSPP/Integration",
     "author": "OpenSPP.org",
     "website": "https://github.com/OpenSPP/OpenSPP2",

spp_dci_client/readme/HISTORY.md

@@ -1,3 +1,7 @@
+### 19.0.2.0.2
+
+- test(dci_client): make the outgoing-log integration tests hold on a database that has `spp_api_v2` installed, the only place they run at all (they skip without `spp.api.outgoing.log`, so they had never executed before the full-stack CI). The client writes the audit row through a separate, committed cursor so it survives the request's rollback; a test transaction at REPEATABLE READ cannot see that row and instead found an earlier class's committed row, so every status assertion compared against a stale success entry. The tests now assert at the log-service seam, one test reads a real row back through a fresh cursor and removes it, and the 401-retry test builds a valid OAuth2 source and asserts the two log calls in the order the code guarantees. No behaviour change (#443)
+
 ### 19.0.2.0.0
 
 - Initial migration to OpenSPP2

spp_dci_client/static/description/index.html

@@ -514,6 +514,23 @@ <h2><a class="toc-backref" href="#toc-entry-1">Changelog</a></h2>
 </div>
 </div>
 <div class="section" id="section-1">
+<h1>19.0.2.0.2</h1>
+<ul class="simple">
+<li>test(dci_client): make the outgoing-log integration tests hold on a
+database that has <tt class="docutils literal">spp_api_v2</tt> installed, the only place they run at
+all (they skip without <tt class="docutils literal">spp.api.outgoing.log</tt>, so they had never
+executed before the full-stack CI). The client writes the audit row
+through a separate, committed cursor so it survives the request’s
+rollback; a test transaction at REPEATABLE READ cannot see that row
+and instead found an earlier class’s committed row, so every status
+assertion compared against a stale success entry. The tests now assert
+at the log-service seam, one test reads a real row back through a
+fresh cursor and removes it, and the 401-retry test builds a valid
+OAuth2 source and asserts the two log calls in the order the code
+guarantees. No behaviour change (#443)</li>
+</ul>
+</div>
+<div class="section" id="section-2">
 <h1>19.0.2.0.0</h1>
 <ul class="simple">
 <li>Initial migration to OpenSPP2</li>