demery-pivotal commented on pull request #5324:
URL: https://github.com/apache/geode/pull/5324#issuecomment-659087627
Definitely a problem. Lacking ideas, I explored a little further.
I checked the distributed tests in several modules, and each had tests that
DID run in your PR:
- geode-connectors
- geode-cq
- geode-dunit
- geode-gfsh
- geode-wan
- geode-web
- geode-web-management
I did not do a full search to see if any distributed tests from those
modules are missing. But each distributedTest class that I checked from each of
those modules was run in your PR.
I searched the names of the test classes that ran in your PR and found:
- 55 class names with DistributedTest
- 308 class names with DUnitTest
- 73 additional class names with DUnit
So it appears that there are lots of distributedTests running.
I was suspicious of this in geode-core's build.gradle file (though I
couldn't see how it would affect test class selection):
distributedTest {
// Some tests have inner tests that should be ignored
exclude "**/*\$*.class"
}
But geode-dunit has the same exclusion in its distributedTest dependencies,
and its tests ran in your PR.
Several other modules (geode-logging, geode-membership, and
geode-serialization) declare the same exclusion in the distributedTest
dependencies, though they don't seem to actually have a distributedTest
directory.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
