----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41044/#review110046 -----------------------------------------------------------
Ship it! Ship It! - Jens Deppe On Dec. 10, 2015, 11:54 p.m., Kirk Lund wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41044/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2015, 11:54 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 > >
