From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1551) id CEFBB384F017; Wed, 25 May 2022 12:47:13 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org CEFBB384F017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Pedro Alves To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Add -nopass option to gdb_test/gdb_test_multiple X-Act-Checkin: binutils-gdb X-Git-Author: Pedro Alves X-Git-Refname: refs/heads/master X-Git-Oldrev: 51498ab9abc6122817428198e831d36923e293a4 X-Git-Newrev: aee9dcf8a836fe08e3d7f1b58a8b6dd2e16d9b68 Message-Id: <20220525124713.CEFBB384F017@sourceware.org> Date: Wed, 25 May 2022 12:47:13 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 25 May 2022 12:47:13 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3Daee9dcf8a836= fe08e3d7f1b58a8b6dd2e16d9b68 commit aee9dcf8a836fe08e3d7f1b58a8b6dd2e16d9b68 Author: Pedro Alves Date: Wed May 18 13:22:02 2022 +0100 Add -nopass option to gdb_test/gdb_test_multiple =20 The previous patch to add -prompt/-lbl to gdb_test introduced a regression: Before, you could specify an explicit empty message to indicate you didn't want to PASS, like so: =20 gdb_test COMMAND PATTERN "" =20 After said patch, gdb_test no longer distinguishes no-message-specified vs empty-message, so tests that previously would be silent on PASS, now started emitting PASS messages based on COMMAND. This in turn introduced a number of PATH/DUPLICATE violations in the testsuite. =20 I think that not issuing a PASS should be restricted to only a few cases -- namely in shared routines exported by gdb.exp, which happen to use gdb_test internally. In tests that iterate an unknown number of tests exercising some racy scenario. In the latter case, if we emit PASSes for each iteration, we run into the situation where different testsuite runs emit a different number of PASSes. =20 Thus, this patch preserves the current behavior, and, instead, adds a new "-nopass" option to gdb_test and gdb_test_no_output. Compared to the old way of supressing PASS with an empty message, this has the advantage that you can specify a FAIL message that is distinct from the command string, and, it's also more explicit. =20 Change-Id: I5375f23f073493e0672190a0ec2e847938a580b2 Diff: --- gdb/testsuite/lib/gdb.exp | 60 +++++++++++++++++++++++++------------------= ---- 1 file changed, 32 insertions(+), 28 deletions(-) diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp index a6780d8d634..960c0eb2313 100644 --- a/gdb/testsuite/lib/gdb.exp +++ b/gdb/testsuite/lib/gdb.exp @@ -834,6 +834,13 @@ proc gdb_internal_error_resync {} { return 0 } =20 +# Fill in the default prompt if PROMPT_REGEXP is empty. +proc fill_in_default_prompt {prompt_regexp} { + if { "$prompt_regexp" =3D=3D "" } { + return "$::gdb_prompt $" + } + return $prompt_regexp +} =20 # gdb_test_multiple COMMAND MESSAGE [ -prompt PROMPT_REGEXP] [ -lbl ] # EXPECT_ARGUMENTS @@ -960,9 +967,7 @@ proc gdb_test_multiple { command message args } { error "Too few arguments to gdb_test_multiple" } =20 - if { "$prompt_regexp" =3D=3D "" } { - set prompt_regexp "$gdb_prompt $" - } + set prompt_regexp [fill_in_default_prompt $prompt_regexp] =20 if { $message =3D=3D "" } { set message $command @@ -1335,6 +1340,7 @@ proc gdb_test_multiline { name args } { # -prompt PROMPT_REGEXP specifies a regexp matching the expected prompt # after the command output. If empty, defaults to "$gdb_prompt $". # -lbl specifies that line-by-line matching will be used. +# -nopass specifies that a PASS should not be issued. # # Returns: # 1 if the test failed, @@ -1348,6 +1354,7 @@ proc gdb_test { args } { parse_args { {prompt ""} {lbl} + {nopass} } =20 lassign $args command pattern message question response @@ -1361,21 +1368,17 @@ proc gdb_test { args } { set message $command } =20 - if { $prompt =3D=3D "" } { - set prompt "$gdb_prompt $" - } + set prompt [fill_in_default_prompt $prompt] =20 set saw_question 0 =20 set user_code {} lappend user_code { -re "\[\r\n\]*(?:$pattern)\[\r\n\]+$prompt" { - if ![string match "" $message] then { - if { $question !=3D "" } { - gdb_assert $saw_question "$message" - } else { - pass "$message" - } + if { $question !=3D "" & !$saw_question} { + fail $message + } elseif {!$nopass} { + pass $message } } } @@ -1439,30 +1442,31 @@ if { [tcl_version_at_least 8 5] =3D=3D 0 } { } } =20 -# gdb_test_no_output COMMAND MESSAGE +# gdb_test_no_output [-prompt PROMPT_REGEXP] [-nopass] COMMAND [MESSAGE] # Send a command to GDB and verify that this command generated no output. # -# See gdb_test_multiple for a description of the COMMAND and MESSAGE -# parameters. If MESSAGE is ommitted, then COMMAND will be used as -# the message. (If MESSAGE is the empty string, then sometimes we do not -# call pass or fail at all; I don't understand this at all.) +# See gdb_test for a description of the -prompt, -nopass, COMMAND, and +# MESSAGE parameters. =20 proc gdb_test_no_output { args } { global gdb_prompt - set command [lindex $args 0] - if [llength $args]>1 then { - set message [lindex $args 1] - } else { - set message $command + + parse_args { + {prompt_re ""} + {nopass} } =20 + lassign $args command message + + set prompt_re [fill_in_default_prompt $prompt_re] + set command_regex [string_to_regexp $command] - gdb_test_multiple $command $message { - -re "^$command_regex\r\n$gdb_prompt $" { - if ![string match "" $message] then { - pass "$message" - } - } + gdb_test_multiple $command $message -prompt $prompt_re { + -re "^$command_regex\r\n$prompt_re" { + if {!$nopass} { + pass $gdb_test_name + } + } } }