refactor: improve SQL function implementations and add tests for ifem…

[robfrank]

This pull request introduces enhancements and bug fixes to the SQL functions if, ifempty, and ifnull in the ArcadeDB codebase. It includes improvements to code readability, handling of edge cases, and the addition of comprehensive unit tests for the ifempty and ifnull functions.

Improvements to SQL Function Logic:

• Refactored the execute method in SQLFunctionIf to replace if-else conditions with a switch statement for better readability and maintainability. Added handling for null and default cases. (engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIf.java, engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIf.javaL60-R74)
• Enhanced the execute method in SQLFunctionIfEmpty to handle empty arrays in addition to strings and collections. (engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfEmpty.java, engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfEmpty.javaL78-R79)

Code Cleanup:

• Updated imports in SQLFunctionIf and SQLFunctionIfEmpty to use specific classes instead of wildcard imports for clarity. (engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIf.java, [1]; engine/src/main/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfEmpty.java, [2]

Testing Enhancements:

• Added a new test class SQLFunctionIfEmptyTest to validate the behavior of the ifempty function, including edge cases such as empty arrays and null values. (engine/src/test/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfEmptyTest.java, engine/src/test/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfEmptyTest.javaR1-R141)
• Added a new test class SQLFunctionIfNullTest to validate the behavior of the ifnull function, including nested usage and fallback logic. (engine/src/test/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfNullTest.java, engine/src/test/java/com/arcadedb/query/sql/function/misc/SQLFunctionIfNullTest.javaR1-R117)

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.25%	✅ 66.67%

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (0b80599)	69901	44687	63.93%
Head commit (d4f3642)	69899 (-2)	44863 (+176)	64.18% (+0.25%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#2186)	9	6	66.67%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences