Fix bogus gdb_test invocations
authorPedro Alves <pedro@palves.net>
Wed, 30 Mar 2022 13:31:56 +0000 (14:31 +0100)
committerPedro Alves <pedro@palves.net>
Tue, 17 May 2022 09:31:36 +0000 (10:31 +0100)
A following patch will make gdb_test error out if bogus arguments are
passed, which exposed bugs in a few testcases:

 - gdb.python/py-parameter.exp, passing a spurious "1" as extra
   parameter, resulting in:

   ERROR: Unexpected arguments: {set test-file-param bar.txt} {The name of the file has been changed to bar.txt} {set new file parameter} 1

 - gdb.python/py-xmethods.exp, a missing test message, resulting in
   the next gdb_test being interpreted as message, question and
   response!  With the enforcing patch, this was caught with:

   ERROR: Unexpected arguments: {p g.mul<char>('a')} {From Python G<>::mul.*} gdb_test {p g_ptr->mul<char>('a')} {From Python G<>::mul.*} {after: g_ptr->mul<char>('a')}

 - gdb.base/pointers.exp, missing a quote.

Change-Id: I66f2db4412025a64121db7347dfb0b48240d46d4

gdb/testsuite/gdb.base/pointers.exp
gdb/testsuite/gdb.python/py-parameter.exp
gdb/testsuite/gdb.python/py-xmethods.exp

index 15f7879a27fba0b79a340fe1141d9a2d8bd2a8da..e64066133184165d647fa07535d33a11009bd1a3 100644 (file)
@@ -209,7 +209,7 @@ gdb_test "print **ptr_to_ptr_to_float" " = 100" \
 
 gdb_test "break marker1" ".*" ""
 gdb_test "cont" "Break.* marker1 \\(\\) at .*:$decimal.*" \
-    continue to marker1"
+    "continue to marker1"
 gdb_test "up" "more_code.*" "up from marker1"
 
 gdb_test "print *pUC" " = 21 \'.025\'.*" "print value of *pUC"
index 98d4b2d4684cee56602bffa67a9fc7c934926fb8..199d3bc16ec3d704cb826c97f4d0058b04633662 100644 (file)
@@ -199,7 +199,7 @@ proc_with_prefix test_file_parameter { } {
        "The name of the file is foo.txt.*" "show initial file value"
     gdb_test "set test-file-param bar.txt" \
        "The name of the file has been changed to bar.txt" \
-       "set new file parameter" 1
+       "set new file parameter"
     gdb_test "show test-file-param" \
        "The name of the file is bar.txt.*" "show new file value"
     gdb_test "python print (test_file_param.value)" \
index f879e9c15a22662edc9806f3621cd1eaa7d4bcd7..1805778214335d44ba7d18ab6b65a8f1659ffe5d 100644 (file)
@@ -130,6 +130,7 @@ gdb_test "p g.size_mul<  5  >()" "From Python G<>::size_mul.*" \
 gdb_test "p g.mul<double>(2.0)" "From Python G<>::mul.*" \
   "after: g.mul<double>(2.0)"
 gdb_test "p g.mul<char>('a')" "From Python G<>::mul.*" \
+  "after: g.mul<char>('a')"
 gdb_test "p g_ptr->mul<char>('a')" "From Python G<>::mul.*" \
   "after: g_ptr->mul<char>('a')"