[PATCH,v2 testsuite] Fix some duplicate test names

Luis Machado luis.machado@linaro.org
Tue May 26 18:42:19 GMT 2020


On 5/26/20 2:50 PM, Pedro Alves wrote:
> Hi, spotted a couple more issues:
> 
> On 5/26/20 6:39 PM, Luis Machado wrote:
>>       gdb_test_multiple "$test" "$test" {
>>   	-re "Breakpoint (\[0-9\]*) at .*$where.*$gdb_prompt $" {
>>   	    set bp $expect_out(1,string)
>> -	    pass "$test"
>> +	    pass "$test inserted as number $bp"
> 
> Same issue with FAIL vs PASS messages.  This needs to be:
> 
>   -     gdb_test_multiple "$test" "$test" {
>   +     gdb_test_multiple "$test" "$test inserted as number $bp" {
>    	-re "Breakpoint (\[0-9\]*) at .*$where.*$gdb_prompt $" {
>    	    set bp $expect_out(1,string)
>   -	    pass "$test"
>   +	    pass $gdb_test_name
> 
> 

This one has a small problem. $bp gets calculated after parsing the 
output. We may have to pass a param to proc so we can differentiate 
between calls.

For example, we attempt to add four breakpoints at "main" later in the test.

We may have to add some prefixes to callers of this proc.


More information about the Gdb-patches mailing list