Make sure that SELECT argument is an integer or return an error.

[redis.git] / tests / integration / aof.tcl

diff --git a/tests/integration/aof.tcl b/tests/integration/aof.tcl
index a554f9ef13573d16fb4dd6e480510d7f30ac53c0..ebf9cb5649888843192bebd4eff8eeb4b365251b 100644 (file)
--- a/tests/integration/aof.tcl
+++ b/tests/integration/aof.tcl
@@ -31,14 +31,20 @@ tags {"aof"} {
     }
 
     start_server_aof [list dir $server_path] {
     }
 
     start_server_aof [list dir $server_path] {

-        test "Unfinished MULTI: Server should not have been started" {
-            if {$::valgrind} {after 2000}
-            assert_equal 0 [is_alive $srv]
-        }
-

         test "Unfinished MULTI: Server should have logged an error" {
         test "Unfinished MULTI: Server should have logged an error" {

-            set result [exec cat [dict get $srv stdout] | tail -n1]
-            assert_match "*Unexpected end of file reading the append only file*" $result
+            set pattern "*Unexpected end of file reading the append only file*"
+            set retry 10
+            while {$retry} {
+                set result [exec tail -n1 < [dict get $srv stdout]]
+                if {[string match $pattern $result]} {
+                    break
+                }
+                incr retry -1
+                after 1000
+            }
+            if {$retry == 0} {
+                error "assertion:expected error not found on config file"
+            }

         }
     }
 
         }
     }
 


@@ -49,14 +55,20 @@ tags {"aof"} {
     }
 
     start_server_aof [list dir $server_path] {
     }
 
     start_server_aof [list dir $server_path] {

-        test "Short read: Server should not have been started" {
-            if {$::valgrind} {after 2000}
-            assert_equal 0 [is_alive $srv]
-        }
-

         test "Short read: Server should have logged an error" {
         test "Short read: Server should have logged an error" {

-            set result [exec cat [dict get $srv stdout] | tail -n1]
-            assert_match "*Bad file format reading the append only file*" $result
+            set pattern "*Bad file format reading the append only file*"
+            set retry 10
+            while {$retry} {
+                set result [exec tail -n1 < [dict get $srv stdout]]
+                if {[string match $pattern $result]} {
+                    break
+                }
+                incr retry -1
+                after 1000
+            }
+            if {$retry == 0} {
+                error "assertion:expected error not found on config file"
+            }

         }
     }
 
         }
     }
 


@@ -69,7 +81,7 @@ tags {"aof"} {
     }
 
     test "Short read: Utility should be able to fix the AOF" {
     }
 
     test "Short read: Utility should be able to fix the AOF" {

-        set result [exec echo y | src/redis-check-aof --fix $aof_path]
+        set result [exec src/redis-check-aof --fix $aof_path << "y\n"]

         assert_match "*Successfully truncated AOF*" $result
     }
 
         assert_match "*Successfully truncated AOF*" $result
     }