Make some of the Java classes proper utilities

ballerina/Dependencies.toml

@@ -69,7 +69,7 @@ modules = [
 [[package]]
 org = "ballerina"
 name = "http"
-version = "2.10.5"
+version = "2.10.6"
 scope = "testOnly"
 dependencies = [
 	{org = "ballerina", name = "auth"},

changelog.md

@@ -30,6 +30,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 - [Support retrieval of enum types](https://github.com/ballerina-platform/ballerina-standard-library/issues/4588)
+- [Make some of the Java classes proper utility classes](https://github.com/ballerina-platform/ballerina-standard-library/issues/4921)
 
 ### Changed
 - [Fix failure for unsupported time type](https://github.com/ballerina-platform/ballerina-standard-library/issues/4531)

native/src/main/java/io/ballerina/stdlib/sql/utils/ProcedureCallResultUtils.java

@@ -52,7 +52,7 @@
  * This class provides functionality for the `ProcedureCallResult` to iterate through the sql result sets.
  */
 public class ProcedureCallResultUtils {
-    public ProcedureCallResultUtils() {
+    private ProcedureCallResultUtils() {
     }
 
     public static Object getNextQueryResult(BObject procedureCallResult) {

native/src/main/java/io/ballerina/stdlib/sql/utils/RecordIteratorUtils.java

@@ -37,7 +37,7 @@
  * @since 1.2.0
  */
 public class RecordIteratorUtils {
-    public RecordIteratorUtils() {
+    private RecordIteratorUtils() {
     }
 
     public static Object nextResult(BObject recordIterator) {

native/src/test/java/io/ballerina/stdlib/sql/TestUtils.java

@@ -73,7 +73,9 @@
 /**
  * TestUtils class for testing ballerina native classes/methods.
  */
-public class TestUtils {
+public final class TestUtils {
+
+    private TestUtils() {}
 
     /**
      * ExtendedColumnDefinition Class for test utils.

test-utils/src/main/java/io/ballerina/stdlib/sql/testutils/nativeimpl/Exec.java

@@ -36,7 +36,9 @@
  *
  * @since 0.6.0
  */
-public class Exec {
+public final class Exec {
+
+    private Exec() {}
 
     public static Object exec(BString command, BMap<BString, BString> env, Object dir, BString[] args) {
         List<String> commandList = new ArrayList<>();

test-utils/src/main/java/io/ballerina/stdlib/sql/testutils/nativeimpl/ExitCode.java

@@ -27,7 +27,9 @@
  *
  * @since 0.6.0
  */
-public class ExitCode {
+public final class ExitCode {
+
+    private ExitCode() {}
 
     public static Object exitCode(BObject objVal) {
         Process process = OSUtils.processFromObject(objVal);

test-utils/src/main/java/io/ballerina/stdlib/sql/testutils/nativeimpl/WaitForExit.java

@@ -27,7 +27,9 @@
  *
  * @since 0.6.0
  */
-public class WaitForExit {
+public final class WaitForExit {
+
+    private WaitForExit() {}
 
     public static Object waitForExit(BObject objVal) {
         Process process = OSUtils.processFromObject(objVal);

test-utils/src/main/java/io/ballerina/stdlib/sql/testutils/utils/OSUtils.java

@@ -30,7 +30,9 @@
 /**
  * @since 0.6.0
  */
-public class OSUtils {
+public final class OSUtils {
+
+    private OSUtils() {}
 
     private static final BString UNKNOWN_MESSAGE = StringUtils.fromString("Unknown Error");