Modeshape does not load cnd files from conf directory when running tests

[amischler]

The Modeshape configuration files may reference nodes types definition files to load:

{
  "name": "cloud-repository-test",
  "node-types": ["conf/test.cnd"]
}

This configuration works when running the application but does not work when running tests.

When running tests, it seems that Modeshape tries to load the cnd file from the root directory of the module, so that the cnd file has to be duplicated there to allow the tests to be executed (see jcr-test module for example).

Any idea why the behavior is different in both cases and how to fix this?

[cescoffier]

The lookup should refer to the getBaseDirectory of the runtime available from the ApplicationConfiguration service.

[Kricket]

I located the problem in the method org.modeshape.jcr.NodeTypesImporter.CndImportOperation.getInputStreamForFile(String). Modeshape attempts to load the CND file from the classloader/classpath/(etc). When using wisdom:run, the application is executed in the target/wisdom directory; however, by default surefire executes tests in the project's base directory.

I was able to work around this by adding the following to the buildsection of the pom (changing the workingDirectory caused other errors...):

            <plugin>
                <artifactId>maven-surefire-plugin</artifactId>
                <configuration>
                    <additionalClasspathElements>
                        <additionalClasspathElement>${basedir}/target/wisdom</additionalClasspathElement>
                    </additionalClasspathElements>
                </configuration>
            </plugin>

[amischler]

Nice workaround (at least, better than duplicating the file at the root of the project!).

I let the issue opened, it could be great to find a fix without workaround for this. Don't you think the base directory should be the same in DEV mode and while running the tests using surefire @cescoffier ?

[cescoffier]

Yes definitely. Please open an issue on wisdom directly.