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 #31

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.

eharrow commented 1 year ago

@phughk any further views on this PR? Would be nice to code up some actual unit tests as I can see a possible NPE.