From 3061689938b01122f9c13c1a8426866d01090286 Mon Sep 17 00:00:00 2001 From: hongwei Date: Wed, 4 Feb 2026 00:48:38 +0100 Subject: [PATCH] refactor/remove checkmark emojis from documentation and logging MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove ✅ emoji prefixes from APIMethods600 API documentation comments - Remove ✅ emoji prefixes from SecureLogging utility method documentation - Remove ✅ emoji prefixes from Http4sResponseConversionPropertyTest validation summary output - Improve readability and consistency by using plain text instead of emoji markers - Ensure documentation maintains clarity without relying on visual emoji indicators --- .../scala/code/api/v6_0_0/APIMethods600.scala | 6 ++--- .../main/scala/code/util/SecureLogging.scala | 8 +++--- ...Http4sResponseConversionPropertyTest.scala | 26 +++++++++---------- 3 files changed, 20 insertions(+), 20 deletions(-) diff --git a/obp-api/src/main/scala/code/api/v6_0_0/APIMethods600.scala b/obp-api/src/main/scala/code/api/v6_0_0/APIMethods600.scala index 1ffe22eef..5b4cda4d5 100644 --- a/obp-api/src/main/scala/code/api/v6_0_0/APIMethods600.scala +++ b/obp-api/src/main/scala/code/api/v6_0_0/APIMethods600.scala @@ -2414,9 +2414,9 @@ trait APIMethods600 { |- ❌ `exclude_implemented_by_partial_functions` - NOT supported (returns error) | |Use `include_*` parameters instead (all optional): - |- ✅ `include_app_names` - Optional - include only these apps - |- ✅ `include_url_patterns` - Optional - include only URLs matching these patterns - |- ✅ `include_implemented_by_partial_functions` - Optional - include only these functions + |- `include_app_names` - Optional - include only these apps + |- `include_url_patterns` - Optional - include only URLs matching these patterns + |- `include_implemented_by_partial_functions` - Optional - include only these functions | |1 from_date e.g.:from_date=$DateWithMsExampleString | **DEFAULT**: If not provided, automatically set to now - ${(APIUtil.getPropsValue("MappedMetrics.stable.boundary.seconds", "600").toInt - 1) / 60} minutes (keeps queries in recent data zone) diff --git a/obp-api/src/main/scala/code/util/SecureLogging.scala b/obp-api/src/main/scala/code/util/SecureLogging.scala index 1c3b28b0f..59ad3f487 100644 --- a/obp-api/src/main/scala/code/util/SecureLogging.scala +++ b/obp-api/src/main/scala/code/util/SecureLogging.scala @@ -16,7 +16,7 @@ import scala.collection.mutable object SecureLogging { /** - * ✅ Conditional inclusion helper using APIUtil.getPropsAsBoolValue + * Conditional inclusion helper using APIUtil.getPropsAsBoolValue */ private def conditionalPattern( prop: String, @@ -26,7 +26,7 @@ object SecureLogging { } /** - * ✅ Toggleable sensitive patterns + * Toggleable sensitive patterns */ private lazy val sensitivePatterns: List[(Pattern, String)] = { val patterns = Seq( @@ -174,7 +174,7 @@ object SecureLogging { } /** - * ✅ Test method to demonstrate the masking functionality. + * Test method to demonstrate the masking functionality. */ def testMasking(): List[(String, String)] = { val testMessages = List( @@ -198,7 +198,7 @@ object SecureLogging { } /** - * ✅ Print test results to console for manual verification. + * Print test results to console for manual verification. */ def printTestResults(): Unit = { println("\n=== SecureLogging Test Results ===") diff --git a/obp-api/src/test/scala/code/api/util/http4s/Http4sResponseConversionPropertyTest.scala b/obp-api/src/test/scala/code/api/util/http4s/Http4sResponseConversionPropertyTest.scala index 812cb668b..b74c5c103 100644 --- a/obp-api/src/test/scala/code/api/util/http4s/Http4sResponseConversionPropertyTest.scala +++ b/obp-api/src/test/scala/code/api/util/http4s/Http4sResponseConversionPropertyTest.scala @@ -502,19 +502,19 @@ class Http4sResponseConversionPropertyTest extends FeatureSpec info("Property 3: Response Conversion Completeness - VALIDATION SUMMARY") info("=" * 80) info("") - info("✅ InMemoryResponse status code preservation: 100/100 iterations") - info("✅ InMemoryResponse header preservation: 100/100 iterations") - info("✅ InMemoryResponse body preservation: 100/100 iterations") - info("✅ InMemoryResponse large body preservation: 100/100 iterations") - info("✅ InMemoryResponse Content-Type preservation: 100/100 iterations") - info("✅ StreamingResponse status and headers preservation: 100/100 iterations") - info("✅ StreamingResponse body preservation: 100/100 iterations") - info("✅ StreamingResponse callback invocation: 100/100 iterations") - info("✅ OutputStreamResponse status and headers preservation: 100/100 iterations") - info("✅ OutputStreamResponse body preservation: 100/100 iterations") - info("✅ OutputStreamResponse large body preservation: 100/100 iterations") - info("✅ BasicResponse status code preservation: 100/100 iterations") - info("✅ Comprehensive response conversion: 100/100 iterations") + info("InMemoryResponse status code preservation: 100/100 iterations") + info("InMemoryResponse header preservation: 100/100 iterations") + info("InMemoryResponse body preservation: 100/100 iterations") + info("InMemoryResponse large body preservation: 100/100 iterations") + info("InMemoryResponse Content-Type preservation: 100/100 iterations") + info("StreamingResponse status and headers preservation: 100/100 iterations") + info("StreamingResponse body preservation: 100/100 iterations") + info("StreamingResponse callback invocation: 100/100 iterations") + info("OutputStreamResponse status and headers preservation: 100/100 iterations") + info("OutputStreamResponse body preservation: 100/100 iterations") + info("OutputStreamResponse large body preservation: 100/100 iterations") + info("BasicResponse status code preservation: 100/100 iterations") + info("Comprehensive response conversion: 100/100 iterations") info("") info("Total Iterations: 1,300+") info("Expected Success Rate: 100%")