gradle / ge-export

Apache License 2.0
3 stars 5 forks source link

Extract SQL queries to a properties file #14

Closed w25r closed 7 years ago

w25r commented 7 years ago

This is the first step towards allowing different target databases.

It is a precursor to fixing #3

@wisechengyi @KidDaedalus

w25r commented 7 years ago

@KidDaedalus please take another look when you have a chance.

wisechengyi commented 7 years ago

just started looking. the tests seem to be failing?

$ gr integrationTest

> Task :integrationTest

com.gradle.exportapi.BuildDaoIntegrationTest > shouldInsertBuild FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.CustomValueDaoIntegrationTest > shouldInsertCustomValue FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.SchemaIntegrationTest > shouldDropAndCreateSchema FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.TaskDaoIntegrationTest > shouldInsertTask FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

com.gradle.exportapi.TestDaoIntegrationTest > shouldInsertTest FAILED
    org.knowm.yank.exceptions.PropertiesFileNotFoundException
        Caused by: java.lang.NullPointerException

5 tests completed, 5 failed

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':integrationTest'.
> There were failing tests. See the report at: file:///Users/yic/workspace/ge-export/build/reports/tests/integrationTest/index.html

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
w25r commented 7 years ago

@wisechengyi you will need to create a test-db-info.properties pointing to your test DB. Note that the test will destroy any data in the DB, so don't do your normal one.