This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[PATCH] gdb.mi/mi2-console.exp: fix racy test (PR testsuite/12649)
- From: Marek Polacek <mpolacek at redhat dot com>
- To: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Wed, 22 Jun 2011 13:26:17 +0200
- Subject: [PATCH] gdb.mi/mi2-console.exp: fix racy test (PR testsuite/12649)
Use mi_gdb_test in place of gdb_test_multiple. Note, that I had to
add "()?" around the first \r\n. The reason is that mi_gdb_test
appends \r\n itself, but the "*running,thread-id=all" part is not mandatory.
Tested with both read{,1}. OK to apply?
2011-06-22 Marek Polacek <mpolacek@redhat.com>
* gdb.mi/mi2-console.exp: Replace gdb_test_multiple with mi_gdb_test.
--- gdb/gdb/testsuite/gdb.mi/mi2-console.exp.mp 2011-06-22 12:58:23.697574353 +0200
+++ gdb/gdb/testsuite/gdb.mi/mi2-console.exp 2011-06-22 13:16:48.577665717 +0200
@@ -48,11 +48,8 @@ if { [gdb_compile "${srcdir}/${subdir}/
mi_run_to_main
# Next over the hello() call which will produce lots of output
-gdb_test_multiple "220-exec-next" "Started step over hello" {
- -re "220\\^running\r\n(\\*running,thread-id=\"all\"\r\n)?$mi_gdb_prompt" {
- pass "Started step over hello"
- }
-}
+mi_gdb_test "220-exec-next" "220\\^running(\r\n)?(\\*running,thread-id=\"all\")?" \
+ "Started step over hello"
if { ![target_info exists gdb,noinferiorio] } {
gdb_expect {