Report an error when the schema load fails

[#135665631]

src/main/kotlin/io/pivotal/trilogy/testrunner/DatabaseTestProjectRunner.kt

@@ -12,17 +12,27 @@ class DatabaseTestProjectRunner(val testCaseRunner: TestCaseRunner, val scriptEx
     }
 
     private fun TrilogyTestProject.runTests(): TestProjectResult {
-        applySchema()
+        tryToApplySchema()
         runSourceScripts()
         return runTestCases()
     }
 
+    private fun TrilogyTestProject.tryToApplySchema() {
+        try {
+            applySchema()
+        } catch(e: RuntimeException) {
+            val errorObjects = listOf(e.localizedMessage.prependIndent("    "))
+            val message = createErrorMessage("testProjectRunner.errors.schema.invalid", errorObjects)
+            throw SchemaLoadFailedException(message, e)
+        }
+    }
+
     private fun TrilogyTestProject.runSourceScripts() {
-        sourceScripts.forEach { script ->
+        sourceScripts.forEach { (name, content) ->
             try {
-                scriptExecuter.execute(script.content)
+                scriptExecuter.execute(content)
             } catch(e: BadSqlGrammarException) {
-                val errorObjects = listOf(script.name, e.localizedMessage.prependIndent("    "))
+                val errorObjects = listOf(name, e.localizedMessage.prependIndent("    "))
                 val message = createErrorMessage("testProjectRunner.errors.scripts.invalid", errorObjects)
                 throw SourceScriptLoadException(message, e)
             }

src/main/kotlin/io/pivotal/trilogy/testrunner/SchemaLoadFailedException.kt

@@ -0,0 +1,3 @@
+package io.pivotal.trilogy.testrunner
+
+class SchemaLoadFailedException(message: String, exception: RuntimeException) : UnrecoverableException(message, exception)

src/main/resources/messages.properties

@@ -11,6 +11,7 @@ testCaseRunner.errors.missingFixture = Unable to find fixture ''{0}''
 testCaseRunner.errors.fixtureRun = Unable to load the ''{0}'' {1} fixture\n{2}
 assertionExecuter.error = Assertion failure: {0}\n{1}
 testProjectRunner.errors.scripts.invalid = Unable to load script ''{0}'':\n{1}
+testProjectRunner.errors.schema.invalid = Unable to load schema:\n{0}
 connectionFailure = Unable to connect to the database:\n{0}
 applicationUsage = Usage: trilogy [<filePath>|--project=<path to trilogy test project>] --db_url=<jdbc url> --db_user=<db user name> --db_password=<db user password>\nThe db_url, db_user and db_password can be replaced by setting environment variables with the same name in upper case
 fatalFailure = [STOP] Execution aborted - the database may be in an inconsistent state

src/test/kotlin/io/pivotal/trilogy/testrunner/DatabaseTestProjectRunnerTests.kt

@@ -12,6 +12,7 @@ import io.pivotal.trilogy.testcase.ProcedureTrilogyTestCase
 import io.pivotal.trilogy.testproject.TestProjectBuilder
 import io.pivotal.trilogy.testproject.TrilogyTestProject
 import io.pivotal.trilogy.testproject.UrlTestProjectResourceLocator
+import org.amshove.kluent.AnyException
 import org.jetbrains.spek.api.Spek
 import org.springframework.jdbc.BadSqlGrammarException
 import java.io.File
@@ -76,7 +77,7 @@ class DatabaseTestProjectRunnerTests : Spek({
         }
     }
 
-    describe("test cases with schema definition") {
+    describe("test project with schema definition") {
         val project = projectNamed("schema")
 
         it("excludes fixtures from test case file list") {
@@ -90,6 +91,27 @@ class DatabaseTestProjectRunnerTests : Spek({
             DatabaseTestProjectRunner(TestCaseRunnerMock(), scriptExecuterMock).run(project)
             scriptExecuterMock.executeArgList.first() shouldStartWith "CREATE TABLE CLIENTS"
         }
+
+        it("stops on failed schema load") {
+            val scriptExecuterMock = ScriptExecuterMock()
+            scriptExecuterMock.shouldFailExecution = true
+            { DatabaseTestProjectRunner(TestCaseRunnerMock(), scriptExecuterMock).run(project) } shouldThrow SchemaLoadFailedException::class
+        }
+
+        it("describes the schema load error") {
+            val scriptExecutorMock = ScriptExecuterMock()
+            scriptExecutorMock.shouldFailExecution = true
+            scriptExecutorMock.failureException = RuntimeException("The transformator\nis quickly seismic.")
+            val exception = try {
+                DatabaseTestProjectRunner(TestCaseRunnerMock(), scriptExecutorMock).run(project)
+                null
+            } catch (e: SchemaLoadFailedException) {
+                e
+            }
+
+            val error = "Unable to load schema:\n    The transformator\n    is quickly seismic."
+            expect(error) { exception!!.localizedMessage }
+        }
     }
 
     describe("test case with broken source scripts") {

src/test/resources/projects/broken_schema/src/001.count_clients.sql

@@ -0,0 +1,4 @@
+THE SPACE HARVESTS COLLISION COURSE LIKE A HUMAN GREEN PEOPLE.
+  GREEN PEOPLE
+    THE TRANSFORMATOR DIES RUMOUR LIKE AN EVASIVE MERMAID.
+  CAPTAIN'S QUARTERS;

src/test/resources/projects/broken_schema/tests/fixtures/schema.sql

@@ -0,0 +1,2 @@
+Everyone loves the asperity of pickles pudding coverd with ground corn syrup.
+Ho-ho-ho! horror of love. Fly proudly like a final green people.  Era secundus caesium est.

src/test/resources/projects/broken_schema/tests/testcase.stt

@@ -0,0 +1,11 @@
+# TEST CASE COUNT_CLIENTS
+Example
+## TEST
+We should be able to count the clients
+### DATA
+|CLIENT_COUNT$|=ERROR=|
+|------------:|-------|
+| 0           |       |
+| 0           |       |
+| 0           |       |
+| 0           |       |