summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMiguel Sofer <miguel.sofer@gmail.com>2007-12-05 19:25:09 (GMT)
committerMiguel Sofer <miguel.sofer@gmail.com>2007-12-05 19:25:09 (GMT)
commit23b63fa1bae88cf7965c0113afc1738d23b51dd4 (patch)
tree2436bbf83803f754caefa4f04dd4904f2b51d251
parent17fd365668a4dbca60cb9d949353bc47e9d6e356 (diff)
downloadtcl-23b63fa1bae88cf7965c0113afc1738d23b51dd4.zip
tcl-23b63fa1bae88cf7965c0113afc1738d23b51dd4.tar.gz
tcl-23b63fa1bae88cf7965c0113afc1738d23b51dd4.tar.bz2
* tests/stack.test: made the tests for stack overflow not care
about which mechanism caused the error (interp's recursion limit or C-stack depth detector).
-rw-r--r--ChangeLog6
-rw-r--r--tests/stack.test27
2 files changed, 28 insertions, 5 deletions
diff --git a/ChangeLog b/ChangeLog
index a8faf78..baf9a18 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2007-12-05 Miguel Sofer <msofer@users.sf.net>
+
+ * tests/stack.test: made the tests for stack overflow not care
+ about which mechanism caused the error (interp's recursion limit
+ or C-stack depth detector).
+
2007-12-05 Jeff Hobbs <jeffh@ActiveState.com>
* win/configure, win/tcl.m4 (LIBS_GUI): mingw needs -lole32
diff --git a/tests/stack.test b/tests/stack.test
index e09e9c1..aca6571 100644
--- a/tests/stack.test
+++ b/tests/stack.test
@@ -9,7 +9,7 @@
# See the file "license.terms" for information on usage and redistribution
# of this file, and for a DISCLAIMER OF ALL WARRANTIES.
#
-# RCS: @(#) $Id: stack.test,v 1.20 2006/03/22 18:32:39 rmax Exp $
+# RCS: @(#) $Id: stack.test,v 1.21 2007/12/05 19:25:10 msofer Exp $
if {[lsearch [namespace children] ::tcltest] == -1} {
package require tcltest 2
@@ -35,23 +35,40 @@ if {[testConstraint unix]} {
}
}
-test stack-1.1 {maxNestingDepth reached on infinite recursion} {minStack2400} {
+#
+# Custom match to detect a stack overflow independently of the mechanism that
+# triggered the error.
+#
+
+customMatch stackOverflow StackOverflow
+proc StackOverflow {- res} {
+ set msgList [list \
+ "too many nested evaluations (infinite loop?)"\
+ "out of stack space (infinite loop?)"]
+ expr {$res in $msgList}
+}
+
+test stack-1.1 {maxNestingDepth reached on infinite recursion} -constraints {
+ minStack2400
+} -body {
# do this in a sub process in case it segfaults
exec [interpreter] << {
proc recurse {} { recurse }
catch { recurse } rv
puts $rv
}
-} {too many nested evaluations (infinite loop?)}
+} -match stackOverflow
-test stack-2.1 {maxNestingDepth reached on infinite recursion} {minStack2400} {
+test stack-2.1 {maxNestingDepth reached on infinite recursion} -constraints {
+ minStack2400
+} -body {
# do this in a sub process in case it segfaults
exec [interpreter] << {
interp alias {} unknown {} notaknownproc
catch { unknown } msg
puts $msg
}
-} {too many nested evaluations (infinite loop?)}
+} -match stackOverflow
# Make sure that there is enough stack to run regexp even if we're
# close to the recursion limit. [Bug 947070] [Patch 746378]