openjpa icon indicating copy to clipboard operation
openjpa copied to clipboard

HerdDB profile for integration tests

Open eolivelli opened this issue 4 years ago • 5 comments

This is a draft, I am going to add a profile that adds HerdDB 0.20.0 dependency and start HerdDB server in embeded in-memory only mode.

https://github.com/diennea/herddb/wiki/Installing-and-running-HerdDB

https://github.com/diennea/herddb/wiki/Using-JDBC-Driver

eolivelli avatar Apr 10 '21 14:04 eolivelli

@struberg @rmannibucau FYI I am just starting... I hope we can have something useful to test the release

eolivelli avatar Apr 10 '21 15:04 eolivelli

@rmannibucau @struberg I am able to run the tests using HerdDB is pure "embedded" mode. the problem is that there are many tests that are failing.

When I developed the fixes for OpenJPA I did not run all of the tests, but only a basic subset.

I have updated the patch, with -Dtest-herddb you run the tests using HerdDB mvn clean install -Dtest-herddb

eolivelli avatar Apr 11 '21 09:04 eolivelli

I added a few references to the docs about the ways to run HerdDB

probably many failures are due to:

  • missing using of "delimiters"
  • unsupported syntaxes in HerdDB (all in all it is a new dialect....)

This is a minimal test case project we have in HerdDB repository to ensure that OpenJPA is working with the basic features https://github.com/diennea/herddb/tree/master/herddb-thirdparty/openjpa-test

eolivelli avatar Apr 11 '21 09:04 eolivelli

We can use assume to skip unsupported tests I guess for now and flag them explicitly as "todo" maybe? (For next release to not block it)

rmannibucau avatar Apr 11 '21 11:04 rmannibucau

@rmannibucau @struberg I am adding assumptions in order to skip test that fail to HerdDB.

some tests fail due to assertions that do not keep into consideration that the dictionary is applying delimiters, some tests are due to some other syntax error (probably something not supported by Calcite) and some other errors fail due to some DB error that needs more investigation.

eolivelli avatar Apr 17 '21 10:04 eolivelli