Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Joel Brobecker <brobecker@adacore.com>
To: gdb-patches@sources.redhat.com
Subject: [RFA/RFC/testsuite] Adjust "help unset" test
Date: Fri, 07 Jul 2006 19:22:00 -0000	[thread overview]
Message-ID: <20060707192254.GE971@adacore.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 972 bytes --]

Hello,

:ADDPATCH testsuite:

While working on adding the "substitute-path" set of commands,
I noticed that the addition made one test fail because the output
of "help unset" now contains a new entry in the list of available
unset commands.

I am not sure we are trying to verify the exact output of this
command, and hence the output for each and every available unset
command. It seems to me that it's good enough to verify the start
and end of the output which should not change after new unset
commands are added. It's a bit similar to what we do when we test
the "help set"...

So I simply modified the expected output to accept any list of
unset commands inside the help text. Of course, if others disagree,
I am happy to just add the help of my new command.

2006-07-07  Joel Brobecker  <brobecker@adacore.com>

        * gdb.base/help.exp: Adjust "help unset" test to accept
        any list of unset commands.

Tested on x86-linux. OK to apply?

Thanks,
-- 
Joel

[-- Attachment #2: help.exp.diff --]
[-- Type: text/plain, Size: 1359 bytes --]

Index: help.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/help.exp,v
retrieving revision 1.20
diff -u -p -r1.20 help.exp
--- help.exp	7 May 2006 20:50:35 -0000	1.20
+++ help.exp	7 Jul 2006 19:14:25 -0000
@@ -572,7 +572,7 @@ gdb_test "help undisplay" "Cancel some e
 # test help unset environment
 gdb_test "help unset environment" "Cancel environment variable VAR for the program\.\[\r\n\]+This does not affect the program until the next \"run\" command\." "help unset environment"
 # test help unset
-gdb_test "help unset" "Complement to certain \"set\" commands\.\[\r\n\]+List of unset subcommands:\[\r\n\]+unset environment -- Cancel environment variable VAR for the program\[\r\n\]+Type \"help unset\" followed by unset subcommand name for full documentation\.\[\r\n\]+Command name abbreviations are allowed if unambiguous\." "help unset"
+gdb_test "help unset" "Complement to certain \"set\" commands\.\[\r\n\]+List of unset subcommands:\[\r\n\]+.*Type \"help unset\" followed by unset subcommand name for full documentation\.\[\r\n\]+Command name abbreviations are allowed if unambiguous\." "help unset"
 # test help up
 gdb_test "help up" "Select and print stack frame that called this one\.\[\r\n\]+An argument says how many frames up to go\." "help up"
 # test help up-silently

             reply	other threads:[~2006-07-07 19:22 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-07 19:22 Joel Brobecker [this message]
2006-07-12 17:39 ` Daniel Jacobowitz
2006-07-14 22:53   ` Joel Brobecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060707192254.GE971@adacore.com \
    --to=brobecker@adacore.com \
    --cc=gdb-patches@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox