contrib/ivorysql_ora/sql/utl_encode.sql (1)

11-73: ⚡ Quick win

Add explicit tests for empty RAW and mixed embedded whitespace decode paths.

Line 11 onward validates most behavior, but the feature contract for empty input and embedded whitespace stripping is not asserted directly. Please add those cases and update expected output accordingly.

✅ Proposed test additions

 -- NULL input returns NULL (STRICT modifier)
 SELECT utl_encode.base64_encode(NULL) IS NULL;
+-- Empty input returns empty RAW
+SELECT rawtohex(utl_encode.base64_encode(hextoraw(''))) IS NULL;

@@
 -- NULL input returns NULL (STRICT modifier)
 SELECT utl_encode.base64_decode(NULL) IS NULL;
+-- Empty input returns empty RAW
+SELECT rawtohex(utl_encode.base64_decode(hextoraw(''))) IS NULL;

@@
 -- Decode known base64 string: 'SGVsbG8=\n' -> 'Hello' bytes
 -- hextoraw('534756736247383D0A') = 'SGVsbG8=\n', hextoraw('48656C6C6F') = 'Hello'
 SELECT rawtohex(utl_encode.base64_decode(hextoraw('534756736247383D0A'))) = '48656C6C6F';
+-- Embedded whitespace tolerance (LF/CR/TAB/SPACE inside payload)
+-- 'SGV s\r\nbG8=\t' -> 'Hello'
+SELECT rawtohex(utl_encode.base64_decode(hextoraw('53475620730D0A6247383D09'))) = '48656C6C6F';

As per coding guidelines `**/sql/*.sql`: Test SQL files. Ensure comprehensive coverage of features.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@contrib/ivorysql_ora/sql/utl_encode.sql` around lines 11 - 73, Add explicit
tests asserting empty-RAW behavior and the whitespace-stripping decode path: add
a case that invokes utl_encode.base64_encode and utl_encode.base64_decode with
an empty RAW input and assert the result is an empty RAW (use rawtohex checks
consistent with existing tests), and add tests that feed base64 inputs
containing embedded whitespace/CR/LF/tabs into utl_encode.base64_decode and
assert they decode to the same binary as the equivalent whitespace-free base64
string; reference the existing tests around utl_encode.base64_encode and
utl_encode.base64_decode to place these new assertions adjacent to the NULL and
round-trip checks.