csonto/bash-completion
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

(testsuite) Removed local short timeout values

Browse Source 
The local timeout settings `-timeout 1' in some loops seemed to cause
inpredictable results, e.g. with `./runCompletion dkpg.exp' since this
test has a long list to match.  Sometimes the `spawn_id' was gone before
the `teardown' method was called.
Removing the short timeout seems to give more predictable results.
This commit is contained in:
Freddy Vulto 2009-07-16 22:47:44 +02:00
parent c6e916bece
commit 2d905413fb
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/lib/library.exp
View File

@ -125,10 +125,7 @@ proc assert_complete_any {cmd {test ""} {prompt /@}} {
expect -ex "$cmd" expect -ex "$cmd"
# Escape special regexp characters # Escape special regexp characters
regsub -all {([\[\]\(\)\.\\\+])} $cmd {\\\1} cmd regsub -all {([\[\]\(\)\.\\\+])} $cmd {\\\1} cmd
# NOTE: A timeout of 1 is used to match quicker when no completion are
# returned. E.g. within `cancel.exp' when no print jobs are available.
expect { expect {
-timeout 1
-re "^\r\n.*$prompt$cmd$" { pass "$test" } -re "^\r\n.*$prompt$cmd$" { pass "$test" }
-re /@ { unresolved "$test at prompt" } -re /@ { unresolved "$test at prompt" }
-re eof { unresolved "eof" } -re eof { unresolved "eof" }
@ -253,7 +250,6 @@ proc match_items {items test {size 20}} {
}; # for }; # for
if {[llength $items] == 1} { if {[llength $items] == 1} {
expect { expect {
-timeout 1
-re "$expected" { set result true } -re "$expected" { set result true }
"\r\n" { set result false; break } "\r\n" { set result false; break }
default { set result false; break } default { set result false; break }
@ -261,7 +257,6 @@ proc match_items {items test {size 20}} {
}; # expect }; # expect
} else { } else {
expect { expect {
-timeout 1
-re "$expected" { set result true } -re "$expected" { set result true }
default { set result false; break } default { set result false; break }
timeout { set result false; break } timeout { set result false; break }