This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH v2 5/8] Fix test names starting with uppercase using multi-line gdb_test/mi_gdb_test


On 11/30/2016 02:11 PM, Pedro Alves wrote:
On 11/25/2016 08:54 PM, Luis Machado wrote:
This fixes offender testcases that have test names starting with uppercase
when using gdb_test/mi_gdb_test in a multi-line construct.

gdb/testsuite/ChangeLog
2016-11-25  Luis Machado  <lgustavo@codesourcery.com>

	Fix test names starting with uppercase throughout the files.

	* gdb/testsuite/gdb.ada/array_return.exp

Drop gdb/testsuite/.  Otherwise LGTM.


Thanks. This is fixed locally throughout.

diff --git a/gdb/testsuite/gdb.ada/array_return.exp b/gdb/testsuite/gdb.ada/array_return.exp
index f4dae48..37f2845 100644
--- a/gdb/testsuite/gdb.ada/array_return.exp
+++ b/gdb/testsuite/gdb.ada/array_return.exp
@@ -52,7 +52,7 @@ gdb_test "break create_small_float_vector" \

 gdb_test "cont" \
          "Breakpoint \[0-9\]+, pck.create_small \\(\\).*" \
-         "Continuing to Create_Small"
+         "continuing to Create_Small"

This looks like a good example showing why enforcing uppercase
in the testsuite's internal routines would be going too far.
I.e., it's quite reasonable to want to print the name of some random
symbol from the test's source code which might want to be uppercase.


Yeah, given the current test names we have, it doesn't make much sense. It would only make sense if had patterns for test names that did not include variables, technology names and the like. Not sure if that's sensible or not at this point.

But, with the series in place, i think we are more coherent with regards to test names.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]