Exist: Disabling the SQL Module breaks the build

[adamretter]

Because a test for the SQL Module is in the core src/org/exist/xquery/modules/sql/ExecuteFunctionTest.java if you disable the SQL module in extensions/build.properties or extensions/local.build.properties then eXist fails to build.

[opax]

Where would be the proper place for a unit test for a module? There is no test folder in extensions/modules. The only other module test I could find is for the compression module and lives in test/src/xquery/modules/compression.

Would it be better to place ExecuteFunctionTest.java in test/src/xquery/modules/sql?

[adamretter]

@opax I think we might have to wait until we can reorganise the build into proper modules, e.g. https://github.com/eXist-db/exist/issues/736

[gmella]

@adamretter I just had a look on the sql module and think that this issue may be closed.

[adamretter]

@gmella thanks for the update :-)