addressed James' feedback

Author: fcannizzohz

testing/README.md

@@ -104,6 +104,8 @@ execution of a listener:
     }
 ```
 
+The test class this method is extracted from is [OrderServiceWithListenerTest](src/test/java/com/hazelcast/samples/testing/junit4/OrderServiceWithListenerTest.java).
+
 ### Testing streaming applications
 
 Test streaming applications is also supported - this is done extending `JetTestSupport` (itself an extension

testing/src/test/java/com/hazelcast/samples/testing/samples/junit4/MyPipelineTest.java

@@ -16,25 +16,28 @@
 import static org.junit.Assert.assertEquals;
 
 @RunWith(JUnit4.class)
-public class MyPipelineTest
-        extends JetTestSupport {
+public class MyPipelineTest extends JetTestSupport {
 
     @Test
     public void testSimplePipeline() {
         Config config = new Config();
         config.setJetConfig(new JetConfig().setEnabled(true));
 
         HazelcastInstance instance = createHazelcastInstance(config);
-        JetService jet = instance.getJet();
+        try {
+            JetService jet = instance.getJet();
 
-        Pipeline p = Pipeline.create();
-        p.readFrom(TestSources.items(1, 2, 3)).writeTo(Sinks.list("out"));
+            Pipeline p = Pipeline.create();
+            p.readFrom(TestSources.items(1, 2, 3)).writeTo(Sinks.list("out"));
 
-        jet.newJob(p).join();
+            jet.newJob(p).join();
 
-        IList<Integer> result = instance.getList("out");
-        assertEquals(3, result.size());
-
-        instance.shutdown();
+            IList<Integer> result = instance.getList("out");
+            assertEquals(3, result.size());
+        } finally {
+            if (instance != null) {
+                instance.shutdown();
+            }
+        }
     }
 }

testing/src/test/java/com/hazelcast/samples/testing/samples/junit5/MyClusterFailureTest.java

@@ -24,6 +24,7 @@ public class MyClusterFailureTest {
     private HazelcastInstance member1;
     private HazelcastInstance member2;
     private MembershipListener mockListener;
+    private TestHazelcastFactory factory;
 
     private static Config getConfig(String v) {
         MemberAttributeConfig mAttr = new MemberAttributeConfig();
@@ -35,7 +36,7 @@ private static Config getConfig(String v) {
 
     @BeforeEach
     void setupCluster() {
-        TestHazelcastFactory factory = new TestHazelcastFactory(2);
+        factory = new TestHazelcastFactory(2);
         member1 = factory.newHazelcastInstance(getConfig("1"));
         member2 = factory.newHazelcastInstance(getConfig("2"));
 
@@ -49,12 +50,8 @@ void setupCluster() {
 
     @AfterEach
     void tearDownCluster() {
-        client.shutdown();
-        if (member1 != null) {
-            member1.shutdown();
-        }
-        if (member2 != null) {
-            member2.shutdown();
+        if (factory != null) {
+            factory.shutdownAll();
         }
     }