[PATCH] Command trace testsuite

[Andrew STUBBS <andrew dot stubbs at st dot com>]

Hi,

Here is a set of tests for the 'set trace-commands' command that I added 
a while back.

I also spotted a small omission in the original patch - it did not 
increment the nest depth when calling user-defined commands. I have 
included a patch for this issue.

Sorry it has taken a little while to get around to this.

Andrew Stubbs

:ADDPATCH testsuite:

2006-08-07  Andrew Stubbs  <andrew.stubbs@st.com>

gdb/
	* cli/cli-script.c (execute_user_command): Update command_next_depth
	on user-command call.

gdb/testsuite
	* gdb.base/trace-commands.exp: New file.

Index: src/gdb/testsuite/gdb.base/trace-commands.exp
===================================================================
--- /dev/null	1970-01-01 00:00:00.000000000 +0000
+++ src/gdb/testsuite/gdb.base/trace-commands.exp	2006-08-07 17:52:15.000000000 +0100
@@ -0,0 +1,83 @@
+# Copyright 2006 Free Software Foundation, Inc.
+
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+
+# Test that the source command's verbose mode works, the 'set trace-commands'
+# command works, and that the nest depth is correct in various circumstances.
+
+if $tracelevel then {
+    strace $tracelevel
+}
+
+gdb_exit
+gdb_start
+
+# Create a file to source
+set fd [open "tracecommandsscript" w]
+puts $fd "\
+echo in tracecommandsscript\\n
+define func
+ echo in func \$arg0\\n
+end
+if 1
+ if 2
+  if 3
+   if 4
+    echo deep\\n
+    func 999
+   end
+  end
+ end
+end
+"
+close $fd
+
+# Make sure that the show trace-commands exists and the default is 'off'.
+gdb_test "show trace-commands" "State of GDB CLI command tracing is off\\." \
+	 "show trace-commands says off"
+
+# Source the script with verbose mode.
+gdb_test "source -v tracecommandsscript" {source -v tracecommandsscript[\r\n]+\+echo in tracecommandsscript\\n[\r\n]+in tracecommandsscript[\r\n]+\+define func[\r\n]+\+if 1[\r\n]+\+\+if 2[\r\n]+\+\+\+if 3[\r\n]+\+\+\+\+if 4[\r\n]+\+\+\+\+\+echo deep\\n[\r\n]+deep[\r\n]+\+\+\+\+\+func 999[\r\n]+\+\+\+\+\+\+echo in func 999\\n[\r\n]+in func 999} "source -v"
+
+# Turn on command tracing.
+gdb_test "set trace-commands" "" "set trace-commands"
+
+# Make sure show trace-commands now gives 'on'.
+gdb_test "show trace-commands" \
+    {\+show trace-commands[\r\n]+State of GDB CLI command tracing is on\.} \
+	 "show trace-commands says on"
+
+# Simple test
+gdb_test "echo hi\\n" {\+echo hi\\n[\r\n]+hi} "simple trace-commands test"
+
+# Nested test
+gdb_test "if 1\nset \$i = 0\nwhile \$i < 5\nfunc \$i\nset \$i += 1\nend\nend" \
+	 {if 1[\r\n]+\+if 1[\r\n]+set \$i = 0[\r\n]+while \$i < 5[\r\n]+func \$i[\r\n]+set \$i \+= 1[\r\n]+end[\r\n]+end[\r\n]+ >set \$i = 0[\r\n]+ >while \$i < 5[\r\n]+  >func \$i[\r\n]+  >set \$i \+= 1[\r\n]+  >end[\r\n]+ >end[\r\n]+\+\+set \$i = 0[\r\n]+\+\+while \$i < 5[\r\n]+\+\+\+func \$i[\r\n]+\+\+\+\+echo in func \$i\\n[\r\n]+in func \$i[\r\n]+\+\+\+set \$i \+= 1[\r\n]+\+\+\+func \$i[\r\n]+\+\+\+\+echo in func \$i\\n[\r\n]+in func \$i[\r\n]+\+\+\+set \$i \+= 1[\r\n]+\+\+\+func \$i[\r\n]+\+\+\+\+echo in func \$i\\n[\r\n]+in func \$i[\r\n]+\+\+\+set \$i \+= 1[\r\n]+\+\+\+func \$i[\r\n]+\+\+\+\+echo in func \$i\\n[\r\n]+in func \$i[\r\n]+\+\+\+set \$i \+= 1[\r\n]+\+\+\+func \$i[\r\n]+\+\+\+\+echo in func \$i\\n[\r\n]+in func \$i[\r\n]+\+\+\+set \$i \+= 1} \
+	 "nested trace-commands test"
+
+
+# Function with source works
+gdb_test "define topfunc\nsource tracecommandsscript\nend" \
+	 {define topfunc[\r\n]+\+define topfunc[\r\n]+Type commands for definition of "topfunc"\.[\r\n]+End with a line saying just "end"\.[\r\n]+source tracecommandsscript[\r\n]+end[\r\n]+>source tracecommandsscript[\r\n]+>end} \
+	 ""
+gdb_test "topfunc" \
+	 {topfunc[\r\n]+\+topfunc[\r\n]+\+\+source tracecommandsscript[\r\n]+\+\+echo in tracecommandsscript\\n[\r\n]+in tracecommandsscript[\r\n]+\+\+define func[\r\n]+\+\+if 1[\r\n]+\+\+\+if 2[\r\n]+\+\+\+\+if 3[\r\n]+\+\+\+\+\+if 4[\r\n]+\+\+\+\+\+\+echo deep\\n[\r\n]+deep[\r\n]+\+\+\+\+\+\+func 999[\r\n]+\+\+\+\+\+\+\+echo in func 999\\n[\r\n]+in func 999} \
+	 "nested trace-commands test with source"
+
+# Test nest depth resets properly on error
+gdb_test "if 1\nif 2\nload\necho should not get here\\n\nend\nend" \
+	 {if 1[\r\n]+\+if 1[\r\n]+if 2[\r\n]+load[\r\n]+echo should not get here\\n[\r\n]+end[\r\n]+end[\r\n]+ >if 2[\r\n]+  >load[\r\n]+  >echo should not get here\\n[\r\n]+  >end[\r\n]+ >end[\r\n]+\+\+if 2[\r\n]+\+\+\+load[\r\n]+No executable file specified\.[\r\n]+Use the "file" or "exec-file" command\.} \
+	 "depth resets on error part 1"
+gdb_test "echo hi\\n" {\+echo hi\\n[\r\n]+hi} "depth resets on error part 2"
Index: src/gdb/cli/cli-script.c
===================================================================
--- src.orig/gdb/cli/cli-script.c	2006-08-07 16:07:55.000000000 +0100
+++ src/gdb/cli/cli-script.c	2006-08-07 17:10:03.000000000 +0100
@@ -286,6 +286,7 @@ execute_user_command (struct cmd_list_el
      not confused with Insight.  */
   in_user_command = 1;
 
+  command_nest_depth++;
   while (cmdlines)
     {
       ret = execute_control_command (cmdlines);
@@ -296,6 +297,7 @@ execute_user_command (struct cmd_list_el
 	}
       cmdlines = cmdlines->next;
     }
+  command_nest_depth--;
   do_cleanups (old_chain);
 }