chore: remove internal short-code markers from test comments (#88)

PR: #88

Author: OmarAlJarrah

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/auth/DigestChallengeHandlerTest.kt

@@ -788,7 +788,7 @@ class DigestChallengeHandlerTest {
         assertEquals(threadCount * iterationsPerThread, collected.max())
     }
 
-    // ---- AtomicLong overflow safety (H4) ----------------------------------------
+    // ---- AtomicLong overflow safety ----------------------------------------
 
     @Test
     fun `nc counter past Int MAX_VALUE stays positive and well-formed`() {

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/pipeline/steps/AsyncInstrumentationStepTest.kt

@@ -364,7 +364,7 @@ class AsyncInstrumentationStepTest {
 
     @Test
     fun `http_response event carries trace_id from MDC set by span scope`() {
-        // B5: verify that the response event captured by FakeSlf4jLogger carries trace.id.
+        // Verify that the response event captured by FakeSlf4jLogger carries trace.id.
         // We set up MDC before the pipeline call so the MdcSnapshot capture inside the
         // use{} block picks up the trace.id installed by the span's makeCurrentWithLoggingContext.
         installBasicMdcAdapter()
@@ -403,7 +403,7 @@ class AsyncInstrumentationStepTest {
 
     @Test
     fun `processAsync handles a synchronous throw from next as a failed future`() {
-        // B13: a next step whose processAsync throws synchronously must be normalised to a
+        // A next step whose processAsync throws synchronously must be normalised to a
         // failed future; failure event emitted; span ended with the throwable.
         val fakeSlf4j = FakeSlf4jLogger("test.async.sync.throw")
         val clientLogger = ClientLogger.forTesting(fakeSlf4j)

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/pipeline/steps/BearerTokenAuthStepTest.kt

@@ -28,10 +28,10 @@ class BearerTokenAuthStepTest {
     private val nowInstant: Instant = Instant.parse("2026-01-01T12:00:00Z")
     private val clock = FixedClock(nowInstant)
 
-    // ---- B14: fresh-token validation does NOT apply refreshMargin ----
+    // ---- fresh-token validation does NOT apply refreshMargin ----
 
     /**
-     * B14: A provider that returns a token whose expiresAt is within the refreshMargin of
+     * A provider that returns a token whose expiresAt is within the refreshMargin of
      * `now` is considered misbehaving — it minted a token that would immediately be treated
      * as expired on the next request. The [BearerTokenAuthStep] must throw
      * [IllegalStateException] even though the token is technically not yet expired (it is

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/pipeline/steps/RedirectStepTest.kt

@@ -941,7 +941,7 @@ class RedirectStepTest {
 
     @Test
     fun `stripping userinfo preserves percent-encoding in path and query`() {
-        // L3: clearing the userinfo must not re-encode the Location. A path segment carrying
+        // Clearing the userinfo must not re-encode the Location. A path segment carrying
         // %2F and a query value carrying %26 must survive verbatim — rebuilding via URI's
         // decoded multi-arg constructor would turn %2F into '/' and %26 into '&', silently
         // changing the path/query structure of the reissued request.

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/pipeline/steps/RetryStepTest.kt

@@ -1031,7 +1031,7 @@ class RetryStepTest {
 
     @Test
     fun `throwing shouldRetryCondition closes the retryable response before propagating`() {
-        // M6: the should-retry predicate runs while the retryable response is still open. If it
+        // The should-retry predicate runs while the retryable response is still open. If it
         // throws, the response must be closed before the exception propagates — otherwise the
         // 5xx response's socket/buffer leaks. Assert both the close() is observed and the
         // exception still surfaces.
@@ -1063,7 +1063,7 @@ class RetryStepTest {
 
     @Test
     fun `throwing computeResponseDelay override closes the retryable response before propagating`() {
-        // M6: a subclass override of computeResponseDelay runs after the predicate approves the
+        // A subclass override of computeResponseDelay runs after the predicate approves the
         // retry, while the response is still open. If it throws, the response must be closed
         // before the exception propagates.
         val closes = AtomicInteger(0)

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/response/LoggableResponseBodyTest.kt

@@ -52,7 +52,7 @@ class LoggableResponseBodyTest {
         }
     }
 
-    // ----- H2: source() that throws before entering .use {} -----
+    // ----- source() that throws before entering .use {} -----
 
     @Test
     fun `when delegate source() itself throws close() still closes the delegate`() {
@@ -166,7 +166,7 @@ class LoggableResponseBodyTest {
         )
     }
 
-    // ----- A12: additional failure-semantics tests -----
+    // ----- additional failure-semantics tests -----
 
     @Test
     fun `source() after drainError is set re-throws the cached exception`() {
@@ -271,7 +271,7 @@ class LoggableResponseBodyTest {
         )
     }
 
-    // ----- H1: bounded capture + live-tail behavior -----
+    // ----- bounded capture + live-tail behavior -----
 
     @Test
     fun `over-cap body returns the full bytes to the consumer while snapshot is bounded`() {

sdk-core/src/test/kotlin/org/dexpace/sdk/core/http/sse/ServerSentEventExtensionsTest.kt

@@ -25,7 +25,7 @@ class ServerSentEventExtensionsTest {
 
     private fun source(text: String): BufferedSource = Io.provider.source(text.toByteArray(Charsets.UTF_8))
 
-    // ----- A13 cases -----
+    // ----- readServerSentEvents() sequence behavior -----
 
     @Test
     fun `valid stream of two events yields the expected ServerSentEvent sequence`() {

sdk-core/src/test/kotlin/org/dexpace/sdk/core/instrumentation/LoggingEventTest.kt

@@ -616,7 +616,7 @@ class LoggingEventTest {
         assertContains(rendered, "anon-msg")
     }
 
-    // -- MDC allow-list (B9) ------------------------------------------------------------------
+    // -- MDC allow-list ------------------------------------------------------------------
 
     @Test
     fun `default allow-list folds only trace_id and span_id from MDC`() {

sdk-core/src/test/kotlin/org/dexpace/sdk/core/io/TeeSinkTest.kt

@@ -256,7 +256,7 @@ class TeeSinkTest {
         assertContentEquals("2345".toByteArray(), tap.snapshot())
     }
 
-    // ----- M4: bounded tap (tapLimit) -----
+    // ----- bounded tap (tapLimit) -----
 
     @Test
     fun `tapLimit caps the tap but forwards the full payload to primary - typed write`() {

sdk-core/src/test/kotlin/org/dexpace/sdk/core/util/FuturesTest.kt

@@ -61,9 +61,8 @@ class FuturesTest {
 
     @Test
     fun `unwrap walks an arbitrarily-deep wrapper chain to the leaf`() {
-        // 20 levels: matches the deep-chain test pattern in RetryUtilsTest after M4 removed
-        // the MAX_DEPTH cap. The HashSet-based cycle guard replaces the depth cap, so a
-        // long-but-acyclic chain unwraps fully to the IOException leaf.
+        // 20 levels: unwrap has no fixed depth cap — a HashSet-based cycle guard replaces any
+        // MAX_DEPTH limit, so a long-but-acyclic chain unwraps fully to the IOException leaf.
         val leaf = IOException("leaf")
         var current: Throwable = leaf
         repeat(20) { current = CompletionException("wrap-$it", current) }