The tests ConcurrentStartTest and JSONAuthCodeTest are currently failing in
the nightly build. They are filed as GEODE-765 and GEODE-766.
Prior to last week, the build.gradle's filter for integrationTest was
**JUnitTest.class. One of my checkins changed that to **Test.class, so now
the build is finding these two previously (and silently) ignored tests.
If you execute either test case in Eclipse, it passes, but if you execute
it via gradle, both fail. They fail because they're finding the
gemfire.properties file that the build.gradle integrationTest task writes
out. It would be best if we could get rid of that gemfire.properties file,
but I'll just focus on getting these two tests passing right now.
It turns out that neither test needs mcast-port or locators (each test just
needs a loner), so the easiest fix is to change the test to ignore that
gemfire.properties file. There are several ways including:
@Rule
public RestoreSystemProperties restoreSystemProperties = new
RestoreSystemProperties();
@Before
public void setUp() {
System.setProperty(DistributedSystem.PROPERTIES_FILE_PROPERTY,
getClass().getSimpleName() + ".properties");
}
I'll go ahead and modify these two tests with the above rule and setUp.
If we really have to keep that gemfire.properties file for
IntegrationTests, then maybe I should simplify the above in a custom rule
IgnoreGemFirePropertiesRule to make it easier for people writing a new
IntegrationTest.
-Kirk
