public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [pushed] Remove path name from test case
@ 2023-11-14 18:49 Tom Tromey
  2023-11-15  9:17 ` Andrew Burgess
  0 siblings, 1 reply; 2+ messages in thread
From: Tom Tromey @ 2023-11-14 18:49 UTC (permalink / raw)
  To: gdb-patches; +Cc: Tom Tromey

'runtest' complains about a path in a test name, from the new test
case py-missing-debug.exp.

This patch fixes the problem by providing an explicit test name to
gdb_test.  I chose something very basic because the block in question
is already wrapped in with_test_prefix.
---
 gdb/testsuite/gdb.python/py-missing-debug.exp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.python/py-missing-debug.exp b/gdb/testsuite/gdb.python/py-missing-debug.exp
index 3e0d70b9d22..0d73d2bf8d8 100644
--- a/gdb/testsuite/gdb.python/py-missing-debug.exp
+++ b/gdb/testsuite/gdb.python/py-missing-debug.exp
@@ -321,7 +321,8 @@ with_test_prefix "enable 'abc-def'" {
     set re [string_to_regexp $binfile]
 
     gdb_test "enable missing-debug-handler \"$re\" abc-def" \
-	"^1 missing debug handler enabled"
+	"^1 missing debug handler enabled" \
+	"enable missing-debug-handler"
 
     gdb_test "info missing-debug-handlers" \
 	[multi_line \
-- 
2.41.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [pushed] Remove path name from test case
  2023-11-14 18:49 [pushed] Remove path name from test case Tom Tromey
@ 2023-11-15  9:17 ` Andrew Burgess
  0 siblings, 0 replies; 2+ messages in thread
From: Andrew Burgess @ 2023-11-15  9:17 UTC (permalink / raw)
  To: Tom Tromey, gdb-patches; +Cc: Tom Tromey

Tom Tromey <tromey@adacore.com> writes:

> 'runtest' complains about a path in a test name, from the new test
> case py-missing-debug.exp.
>
> This patch fixes the problem by providing an explicit test name to
> gdb_test.  I chose something very basic because the block in question
> is already wrapped in with_test_prefix.

Thanks for fixing this.  Not sure how I managed to miss this warning.

Thanks,
Andrew




> ---
>  gdb/testsuite/gdb.python/py-missing-debug.exp | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gdb/testsuite/gdb.python/py-missing-debug.exp b/gdb/testsuite/gdb.python/py-missing-debug.exp
> index 3e0d70b9d22..0d73d2bf8d8 100644
> --- a/gdb/testsuite/gdb.python/py-missing-debug.exp
> +++ b/gdb/testsuite/gdb.python/py-missing-debug.exp
> @@ -321,7 +321,8 @@ with_test_prefix "enable 'abc-def'" {
>      set re [string_to_regexp $binfile]
>  
>      gdb_test "enable missing-debug-handler \"$re\" abc-def" \
> -	"^1 missing debug handler enabled"
> +	"^1 missing debug handler enabled" \
> +	"enable missing-debug-handler"
>  
>      gdb_test "info missing-debug-handlers" \
>  	[multi_line \
> -- 
> 2.41.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-11-15  9:17 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-14 18:49 [pushed] Remove path name from test case Tom Tromey
2023-11-15  9:17 ` Andrew Burgess

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).