----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39445/ -----------------------------------------------------------
(Updated Oct. 28, 2015, 4:29 a.m.) Review request for geode, Dick Cavender, Tushar Khairnar, and Dan Smith. Summary (updated) ----------------- GEODE-12/GEODE-304 Fix Pulse tests Repository: geode Description (updated) ------- This includes several clean-ups for Pulse tests including: - Build gradle (to invoke the right set of tests) - Pulse JMX test beans update based on Geode JMX changes in recent releases - Test references (corrected or ignored) This is set of 3 commits on feature/GEODE-12 branch. Diffs ----- pulse/build.gradle b2bd5fc pulse/src/main/java/com/vmware/gemfire/tools/pulse/internal/data/JMXDataUpdater.java 27aed67 pulse/src/main/resources/pulse.properties 17299c2 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/Member.java 812e055 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/PulseAutomatedTests.java 21c14d5 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/PulseTestData.java d1a92a2 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/PulseTests.java b6dd7c6 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/RegionOnMember.java 7402a8b pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/ServerObject.java 97acac7 pulse/src/test/java/com/vmware/gemfire/tools/pulse/tests/ServerObjectMBean.java 9487c0e pulse/src/test/resources/test.properties fabeea1 Diff: https://reviews.apache.org/r/39445/diff/ Testing (updated) ------- Tested with the latest Firefox browser 41.0.2. All UI tests pass. File Attachments (updated) ---------------- Patch_02 - JUnit & gradle updates https://reviews.apache.org/media/uploaded/files/2015/10/28/4a2bf570-14f8-458d-9cc6-529d0f9b8f77__patch_02.diff Patch_03 - Test clean-up & fixes https://reviews.apache.org/media/uploaded/files/2015/10/28/b5497b9a-8742-46e8-870e-a16e8622cebc__patch_03.diff Thanks, Nitin Lamba