Fix whitespace handling in ScriptSplitter

hspragg-godaddy · web-flow · commit 90c16e5a3f47 · 2025-10-17T15:05:59.000-07:00
Changed the logic from `isNotEmpty and doesNotEndInSpace` to `isEmpty or doesNotEndInSpace` to align with previous implementation behavior.
diff --git a/modules/database-commons/src/main/java/org/testcontainers/ext/ScriptSplitter.java b/modules/database-commons/src/main/java/org/testcontainers/ext/ScriptSplitter.java
@@ -36,7 +36,7 @@ void split() {
                     //skip
                     break;
                 case WHITESPACE:
-                    if (sb.length() != 0 && sb.charAt(sb.length() - 1) != ' ') {
+                    if (sb.length() == 0 || sb.charAt(sb.length() - 1) != ' ') {
                         sb.append(' ');
                     }
                     break;

Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ void split() {`
`36`	`36`	`//skip`
`37`	`37`	`break;`
`38`	`38`	`case WHITESPACE:`
`39`		`- if (sb.length() != 0 && sb.charAt(sb.length() - 1) != ' ') {`
	`39`	`+ if (sb.length() == 0 \|\| sb.charAt(sb.length() - 1) != ' ') {`
`40`	`40`	`sb.append(' ');`
`41`	`41`	`}`
`42`	`42`	`break;`