Skip to content

test: enable CUBRID integration test suites#1815

Merged
velo merged 1 commit into
masterfrom
enable-cubrid-tests
Jun 21, 2026
Merged

test: enable CUBRID integration test suites#1815
velo merged 1 commit into
masterfrom
enable-cubrid-tests

Conversation

@velo

@velo velo commented Jun 21, 2026

Copy link
Copy Markdown
Member

Summary

Enables the CUBRID integration test suites, which were marked @Disabled (bare, no reason) during the JUnit 5 migration (#1776) and therefore ran 0 tests.

With the suites enabled, 608 CUBRID tests pass against cubrid/cubrid:11.3 (verified locally and in GHA). Two CUBRID-specific issues are handled the same way every other backend handles its gaps:

  • SelectBase.date_trunc2UnsupportedOperationException (the CUBRID dialect doesn't implement that date_trunc form, same as SQLite/Derby/H2/Turso). Added CUBRID to its existing @ExcludeIn.
  • KeywordQuotingBase.validateKeywordsCompleteness → excluded for CUBRID. The bundled CUBRID 9.3.9 JDBC driver reports a malformed keyword list (literally TATISTICS, plus DATA_TYPE___/SCOPE___), so completeness against the curated keyword file can't be asserted without committing a driver typo.

Result: Tests run: 608, Failures: 0, Errors: 0, Skipped: 131.

Notes

🤖 Generated with Claude Code

https://claude.ai/code/session_01WPLya3e3DqdYmfdaoH3L6o

…word check)

Signed-off-by: Marvin Froeder <velo.br@gmail.com>
@velo velo merged commit 9955201 into master Jun 21, 2026
5 checks passed
@velo velo deleted the enable-cubrid-tests branch June 21, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant