Cluster test: more reliable 01-faildet unit.

Do things in a sequence that prevents failover during failure detection.

tests/cluster/tests/01-faildet.tcl

@@ -6,19 +6,7 @@ test "Create a 5 nodes cluster" {
     create_cluster 5 5
 }
 
-test "Killing one master node" {
+test "Cluster should start ok" {
-    kill_instance redis 0
-}
-
-test "Cluster should be down now" {
-    assert_cluster_state fail
-}
-
-test "Restarting master node" {
-    restart_instance redis 0
-}
-
-test "Cluster should be up again" {
     assert_cluster_state ok
 }
 
@@ -30,3 +18,21 @@ test "Killing two slave nodes" {
 test "Cluster should be still up" {
     assert_cluster_state ok
 }
+
+test "Killing one master node" {
+    kill_instance redis 0
+}
+
+# Note: the only slave of instance 0 is already down so no
+# failover is possible, that would change the state back to ok.
+test "Cluster should be down now" {
+    assert_cluster_state fail
+}
+
+test "Restarting master node" {
+    restart_instance redis 0
+}
+
+test "Cluster should be up again" {
+    assert_cluster_state ok
+}