From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 97153 invoked by alias); 21 Jul 2019 18:49:23 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 96911 invoked by uid 89); 21 Jul 2019 18:49:21 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-18.3 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,KAM_SHORT,RCVD_IN_DNSWL_NONE,RCVD_IN_SEMBLACK,SPF_HELO_PASS autolearn=ham version=3.3.1 spammy=boxes, render X-HELO: gateway34.websitewelcome.com Received: from gateway34.websitewelcome.com (HELO gateway34.websitewelcome.com) (192.185.149.62) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 21 Jul 2019 18:49:18 +0000 Received: from cm10.websitewelcome.com (cm10.websitewelcome.com [100.42.49.4]) by gateway34.websitewelcome.com (Postfix) with ESMTP id 1E46A289DA for ; Sun, 21 Jul 2019 13:49:17 -0500 (CDT) Received: from box5379.bluehost.com ([162.241.216.53]) by cmsmtp with SMTP id pGtZhiWTl2PzOpGtZhqadl; Sun, 21 Jul 2019 13:49:17 -0500 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=tromey.com; s=default; h=References:In-Reply-To:Message-Id:Date:Subject:Cc:To:From: Sender:Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=baRMiAVEbRk7Hyta5mLtVvi2Lt2f5gRhXqsHDinnwt8=; b=FCJ6c4G5Fl6dpsMi3PJK7mt/KJ WvCKfLaPd18hdtX2ZgMe1qPcpXy/O0VK225DyIAkCU/7vIea36qEcqC1nSqpIOvLKdpQ7J1+PpYo6 estf6VWZ/rOO6mzt/ZaERNctW; Received: from 97-122-178-82.hlrn.qwest.net ([97.122.178.82]:58458 helo=bapiya.Home) by box5379.bluehost.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256) (Exim 4.92) (envelope-from ) id 1hpGtY-002hWd-U2; Sun, 21 Jul 2019 13:49:17 -0500 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH 7/8] Add test case for empty TUI windows Date: Sun, 21 Jul 2019 18:49:00 -0000 Message-Id: <20190721184910.26679-8-tom@tromey.com> In-Reply-To: <20190721184910.26679-1-tom@tromey.com> References: <20190721184910.26679-1-tom@tromey.com> X-SW-Source: 2019-07/txt/msg00497.txt.bz2 My original intent here was to add a test case to test that empty TUI windows re-render their contents after a resize. However, this seems pretty broken at the moment, so a lot of the test is actually disabled. gdb/testsuite/ChangeLog 2019-07-21 Tom Tromey * lib/tuiterm.exp (Term::clean_restart): Make "executable" optional. * gdb.tui/empty.exp: New file. --- gdb/testsuite/ChangeLog | 6 ++ gdb/testsuite/gdb.tui/empty.exp | 108 ++++++++++++++++++++++++++++++++ gdb/testsuite/lib/tuiterm.exp | 11 +++- 3 files changed, 122 insertions(+), 3 deletions(-) create mode 100644 gdb/testsuite/gdb.tui/empty.exp diff --git a/gdb/testsuite/gdb.tui/empty.exp b/gdb/testsuite/gdb.tui/empty.exp new file mode 100644 index 00000000000..e24ac58f718 --- /dev/null +++ b/gdb/testsuite/gdb.tui/empty.exp @@ -0,0 +1,108 @@ +# Copyright 2019 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 3 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, see . + +# Test TUI resizing with empty windows. + +load_lib "tuiterm.exp" + +standard_testfile + +Term::clean_restart 24 80 + +if {![Term::enter_tui]} { + unsupported "TUI not supported" +} + +# Each entry describes a layout. It has these items: +# 1. Layout name +# 2. Test name +# 3. List of boxes in 80x24 mode +# 4. List of boxes in 90x40 mode +# 5. List of test name and text for the empty window +set layouts { + {src src {{3 0 77 15}} {{3 0 87 23}} + {{"no source" "No Source Available"}}} + {regs src-regs {{0 0 80 8} {3 7 77 8}} {{0 0 90 13} {3 13 87 13}} + { + {"no source" "No Source Available"} + {"no regs" "Register Values Unavailable"} + }} + {asm asm {{3 0 77 15}} {{3 0 87 24}} + {"no asm" "No Assembly Available"}} + {regs asm-regs {{0 0 80 8} {3 7 77 9}} {{0 0 90 13} {3 13 87 14}} + { + {"no asm" "No Assembly Available"} + {"no regs" "Register Values Unavailable"} + }} + {split split {{3 0 77 8} {3 7 77 9}} {{3 0 87 14} {3 14 87 14}} + { + {"no source" "No Source Available"} + {"no asm" "No Assembly Available"} + }} + {regs split-regs {{0 0 80 8} {3 7 77 9}} {{0 0 90 13} {3 13 87 14}} + { + {"no asm" "No Assembly Available"} + {"no regs" "Register Values Unavailable"} + }} +} + +# Helper function to verify a list of boxes. +proc check_boxes {boxes} { + set boxno 1 + foreach box $boxes { + if {$boxno > 1} { + # The upper-left corner of the second box may not render + # properly, due to overlap. + setup_xfail *-*-* + } + eval Term::check_box [list "box $boxno"] $box + incr boxno + } +} + +# Helper function to verify text. +proc check_text {text_list} { + set text [Term::get_all_lines] + foreach item $text_list { + lassign $item testname check + if {[string first $check $text]} { + pass $testname + } else { + fail $testname + } + } + +} + +foreach layout $layouts { + lassign $layout name testname small_boxes large_boxes text_list + + with_test_prefix $testname { + Term::command "layout $name" + with_test_prefix 80x24 { + check_boxes $small_boxes + check_text $text_list + } + + # FIXME: resizing is broken enough that we don't test it for + # now. + # Term::resize 40 90 + # with_test_prefix 90x40 { + # check_boxes $large_boxes + # check_text $text_list + # } + # Term::resize 24 80 + } +} diff --git a/gdb/testsuite/lib/tuiterm.exp b/gdb/testsuite/lib/tuiterm.exp index 3fc4fcb614a..e56e4a48952 100644 --- a/gdb/testsuite/lib/tuiterm.exp +++ b/gdb/testsuite/lib/tuiterm.exp @@ -392,13 +392,18 @@ namespace eval Term { # Like ::clean_restart, but ensures that gdb starts in an # environment where the TUI can work. ROWS and COLS are the size - # of the terminal. EXECUTABLE is passed to clean_restart. - proc clean_restart {rows cols executable} { + # of the terminal. EXECUTABLE, if given, is passed to + # clean_restart. + proc clean_restart {rows cols {executable {}}} { global env stty_init save_vars {env(TERM) stty_init} { setenv TERM ansi _setup $rows $cols - ::clean_restart $executable + if {$executable == ""} { + ::clean_restart + } else { + ::clean_restart $executable + } } } -- 2.17.2