> On Dec. 10, 2015, 11:16 p.m., Jens Deppe wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessWrapper.java, > > line 53 > > <https://reviews.apache.org/r/41044/diff/1/?file=1155207#file1155207line53> > > > > Can the timeouts be consolidated at all? > > > > Can the defaults be shortened? If something is actually wrong, a > > timeout of 10 minutes is a long time to wait to find out.
The current default is 5 minutes. This has been a gradually growing timeout because slow, overloaded machines timeout. This unfortunately is the problem with IntegrationTests and forking of JVMs on unknown machines. > On Dec. 10, 2015, 11:16 p.m., Jens Deppe wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessWrapper.java, > > line 52 > > <https://reviews.apache.org/r/41044/diff/1/?file=1155207#file1155207line52> > > > > Is this used? This one can actually be deleted now. It's no longer used and I apparently missed that. - Kirk ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41044/#review109886 ----------------------------------------------------------- On Dec. 7, 2015, 5:25 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41044/ > ----------------------------------------------------------- > > (Updated Dec. 7, 2015, 5:25 p.m.) > > > Review request for geode, Darrel Schneider and Jens Deppe. > > > Bugs: GEODE-291 > https://issues.apache.org/jira/browse/GEODE-291 > > > Repository: geode > > > Description > ------- > > Add build dependencies for testing frameworks Awaitility and Catch-Exception. > Upgrade AssertJ build dependency. > > Refactor DistributedSystem PROPERTY_FILE constants, system property and APIs > to facilitate automated testing. Create UnitTest and IntegrationTest for > DistributedSystem. > > Tighten up ProcessWrapper, ProcessStreamReader and ProcessOutputReader to > eliminate intermittent failures and unwanted stderr output in Jenkins. > > Extract integration tests from launcher UnitTests to new IntegrationTests. > > Convert tests in LocatorLauncherIntegrationJUnitTest and > ServerLauncherIntegrationJUnitTest to follow BDD style. > > I will delete Geode291TestSuite before merging to develop. > > > Diffs > ----- > > build.gradle b5465b8 > > gemfire-core/src/main/java/com/gemstone/gemfire/distributed/AbstractLauncher.java > bfd3de7 > > gemfire-core/src/main/java/com/gemstone/gemfire/distributed/DistributedSystem.java > e2fccd9 > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/AbstractLauncherIntegrationJUnitTest.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/AbstractLauncherJUnitTest.java > 399c78f > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/CommonLauncherTestSuite.java > 94ba320 > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/DistributedSystemIntegrationJUnitTest.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/DistributedSystemJUnitTest.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherIntegrationJUnitTest.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/LocatorLauncherJUnitTest.java > 003a098 > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/ServerLauncherIntegrationJUnitTest.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/distributed/ServerLauncherJUnitTest.java > 6884e58 > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/Geode291TestSuite.java > PRE-CREATION > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessOutputReader.java > e99dceb > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessStreamReader.java > f99f5f9 > > gemfire-core/src/test/java/com/gemstone/gemfire/test/process/ProcessWrapper.java > 2847b34 > gradle/dependency-versions.properties 3e6b6a5 > > Diff: https://reviews.apache.org/r/41044/diff/ > > > Testing > ------- > > new tests and precheckin > > > Thanks, > > Kirk Lund > >
