From 14a71cf545bf652202a3f056daefd654ad311eb5 Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Thu, 19 May 2016 13:22:46 +0300 Subject: [PATCH] Added and fixed tests related to "finally" handling --- Zend/tests/try/bug70228.phpt | 4 ++-- Zend/tests/try/bug72213.phpt | 27 +++++++++++++++++++++++++++ Zend/tests/try/bug72213_2.phpt | 25 +++++++++++++++++++++++++ 3 files changed, 54 insertions(+), 2 deletions(-) create mode 100644 Zend/tests/try/bug72213.phpt create mode 100644 Zend/tests/try/bug72213_2.phpt diff --git a/Zend/tests/try/bug70228.phpt b/Zend/tests/try/bug70228.phpt index 23f5864740..e85724765e 100644 --- a/Zend/tests/try/bug70228.phpt +++ b/Zend/tests/try/bug70228.phpt @@ -7,10 +7,10 @@ See https://bugs.php.net/bug.php?id=70228 function foo() { try { return str_repeat("a", 2); } - finally { return true; } + finally { return str_repeat("b", 2); } } var_dump(foo()); ?> --EXPECT-- -string(3) "bar" +string(2) "bb" diff --git a/Zend/tests/try/bug72213.phpt b/Zend/tests/try/bug72213.phpt new file mode 100644 index 0000000000..aee4c95b20 --- /dev/null +++ b/Zend/tests/try/bug72213.phpt @@ -0,0 +1,27 @@ +--TEST-- +Bug #72213 (Finally leaks on nested exceptions) +--XFAIL-- +See https://bugs.php.net/bug.php?id=72213 +--FILE-- +getMessage()); + var_dump($e->getPrevious()->getMessage()); +} +?> +--EXPECT-- +string(1) "b" +string(1) "a" diff --git a/Zend/tests/try/bug72213_2.phpt b/Zend/tests/try/bug72213_2.phpt new file mode 100644 index 0000000000..790abe125d --- /dev/null +++ b/Zend/tests/try/bug72213_2.phpt @@ -0,0 +1,25 @@ +--TEST-- +Bug #72213 (Finally leaks on nested exceptions) +--FILE-- +getMessage()}\n"; +} +--EXPECT-- +caught 1 -- 2.50.1