Get test naming practices straight

Description

In the -elasticsearch plugin, we use this filter in the maven-failsafe-plugin configuration:

The filter <include>*/.java</include> means that we execute pretty much every test. Including unit tests, which are supposed to be suffixed with "Test" (as opposed to "IT" for integration tests).

From a test coverage point of view, there's no point in doing that, since those tests are supposedly unaffected by the change of indexing service. But we have to do that because many integration tests are actually using the *Test suffix, especially in the -orm and -engine modules, making unit tests indistinguishable from integration tests.

Thus a simple naming problem is currently slowing down our build. It may not have been a big deal until now, but with (add the -engine tests to the -elasticsearch module) it's getting worse, and it will also get worse each time we add another indexing service (Solr, ...).

We should probably consider renaming our integration tests in -engine and -orm to use the "IT" suffix, or move them to a separate folder (src/integration/main for instance), or both.

Environment

None

Assignee

Yoann Rodière

Reporter

Yoann Rodière

Labels

None

Suitable for new contributors

None

Feedback Requested

None

Components

Fix versions

Priority

Minor
Configure