search

surrealdb / surrealdb.java

SurrealDB SDK for Java
https://surrealdb.com
Apache License 2.0
67 stars 21 forks source link

Move integration tests to separate file set #26

Closed eharrow closed 1 year ago

eharrow commented 1 year ago

This PR attempts to split unit tests from integration tests. The current unit tests require a running SurrealDB instance either locally or more recently as test containers in docker which makes them integration tests by definition. Not having used gradle before I simply adopted the standard grade pattern for integration tests with a new fileset rather than renaming them as *IT.java or something like that and use pattern matching to exclude them from unit tests.

Running gradle check will execute them along with checkstyle.

phughk commented 1 year ago

Hey @eharrow could you please resolve merge conflicts? That makes it easier to review. Thanks

eharrow commented 1 year ago

I think I might start the PR from scratch and the merge from main is too busy

eharrow commented 1 year ago

Nightmare resolving the merge. Closing and reopening as a new one.