sharanggupta opened a new pull request, #3020: URL: https://github.com/apache/karaf/pull/3020
`testTextResources` was added with KARAF-3111 in 2014 but never got a `@Test` annotation. The class is JUnit 4, so it has never run. This adds the annotation. It also points the storage location at the `TemporaryFolder` rule, like the other tests in the class, so it no longer leaves an instance directory under `target/`. Tested with `mvn -pl instance -am verify -Dtest=InstanceServiceImplTest`: 6 tests pass, up from 5. Removing the text resource from the settings makes the test fail, so it checks real behaviour. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
