From 3f16fee39a73893bfe2b19001703d8e6992471e2 Mon Sep 17 00:00:00 2001
From: antirez <antirez@gmail.com>
Date: Tue, 25 Feb 2014 08:23:48 +0100
Subject: [PATCH] Sentinel test: restart_instance should refresh pid attrib.

Also kill_instance was modified to warn when a test will try to kill the
same instance multiple times for error.
---
 tests/sentinel.tcl | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/sentinel.tcl b/tests/sentinel.tcl
index 669bc830..e278aea7 100644
--- a/tests/sentinel.tcl
+++ b/tests/sentinel.tcl
@@ -280,6 +280,9 @@ proc get_instance_id_by_port {type port} {
 # The instance can be restarted with restart-instance.
 proc kill_instance {type id} {
     set pid [get_instance_attrib $type $id pid]
+    if {$pid == -1} {
+        error "You tried to kill $type $id twice."
+    }
     exec kill -9 $pid
     set_instance_attrib $type $id pid -1
     set_instance_attrib $type $id link you_tried_to_talk_with_killed_instance
@@ -302,6 +305,7 @@ proc restart_instance {type id} {
         set prgname redis-sentinel
     }
     set pid [exec ../../src/${prgname} $cfgfile &]
+    set_instance_attrib $type $id pid $pid
     lappend ::pids $pid
 
     # Check that the instance is running
-- 
GitLab