From patchwork Tue Jul 10 21:39:26 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Philippe Waroquiers X-Patchwork-Id: 28305 Received: (qmail 58102 invoked by alias); 10 Jul 2018 21:39:54 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 57984 invoked by uid 89); 10 Jul 2018 21:39:53 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-27.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_LOW, SPF_PASS autolearn=ham version=3.3.2 spammy=mutual, exclusion X-HELO: mailsec114.isp.belgacom.be Received: from mailsec114.isp.belgacom.be (HELO mailsec114.isp.belgacom.be) (195.238.20.110) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 10 Jul 2018 21:39:50 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=skynet.be; i=@skynet.be; q=dns/txt; s=securemail; t=1531258790; x=1562794790; h=from:to:cc:subject:date:message-id:in-reply-to: references; bh=0/aAhqQlQ3fvkKsR3m4+xgjIM2LttjQ70+UeHmd3pv8=; b=JAngVdmKX0Xe/EXB83gaYH9B03cb+rEhK8EIsXXRBckwzUD59+9sKhOg O+hYsjh6w4/45nh+MzFSntTEI9gkCw==; Received: from 160.177-177-91.adsl-dyn.isp.belgacom.be (HELO md.home) ([91.177.177.160]) by relay.skynet.be with ESMTP/TLS/DHE-RSA-AES128-GCM-SHA256; 10 Jul 2018 23:39:32 +0200 From: Philippe Waroquiers To: gdb-patches@sourceware.org Cc: Philippe Waroquiers Subject: [RFA_v4 8/8] Add a self-test for cli-utils.c Date: Tue, 10 Jul 2018 23:39:26 +0200 Message-Id: <20180710213926.32240-9-philippe.waroquiers@skynet.be> In-Reply-To: <20180710213926.32240-1-philippe.waroquiers@skynet.be> References: <20180710213926.32240-1-philippe.waroquiers@skynet.be> X-IsSubscribed: yes tests added for: * number_or_range_parser In particular, it tests the cur_tok when parsing is finished. * parse_flags * parse_flags_qcs gdb/ChangeLog 2018-07-09 Philippe Waroquiers * Makefile.in (SUBDIR_UNITTESTS_SRCS): Add unittests/cli-utils-selftests.c * unittests/cli-utils-selftests.c: New file. --- gdb/Makefile.in | 1 + gdb/unittests/cli-utils-selftests.c | 253 ++++++++++++++++++++++++++++ 2 files changed, 254 insertions(+) create mode 100644 gdb/unittests/cli-utils-selftests.c diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 9c0dbbfda5..24852702c4 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -420,6 +420,7 @@ SUBDIR_PYTHON_CFLAGS = SUBDIR_UNITTESTS_SRCS = \ unittests/array-view-selftests.c \ + unittests/cli-utils-selftests.c \ unittests/common-utils-selftests.c \ unittests/environ-selftests.c \ unittests/format_pieces-selftests.c \ diff --git a/gdb/unittests/cli-utils-selftests.c b/gdb/unittests/cli-utils-selftests.c new file mode 100644 index 0000000000..b952c7c77f --- /dev/null +++ b/gdb/unittests/cli-utils-selftests.c @@ -0,0 +1,253 @@ +/* Unit tests for the cli-utils.c file. + + Copyright (C) 2018 Free Software Foundation, Inc. + + This file is part of GDB. + + 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 . */ + +#include "defs.h" +#include "cli/cli-utils.h" +#include "selftest.h" + +namespace selftests { +namespace cli_utils { + +static void +test_number_or_range_parser () +{ + /* Test parsing a simple integer. */ + { + number_or_range_parser one ("1"); + + SELF_CHECK (!one.finished ()); + SELF_CHECK (one.get_number () == 1); + SELF_CHECK (one.finished ()); + SELF_CHECK (strcmp (one.cur_tok (), "") == 0); + } + + /* Test parsing an integer followed by a non integer. */ + { + number_or_range_parser one_after ("1 after"); + + SELF_CHECK (!one_after.finished ()); + SELF_CHECK (one_after.get_number () == 1); + SELF_CHECK (one_after.finished ()); + SELF_CHECK (strcmp (one_after.cur_tok (), "after") == 0); + } + + /* Test parsing a range. */ + { + number_or_range_parser one_three ("1-3"); + + for (int i = 1; i < 4; i++) + { + SELF_CHECK (!one_three.finished ()); + SELF_CHECK (one_three.get_number () == i); + } + SELF_CHECK (one_three.finished ()); + SELF_CHECK (strcmp (one_three.cur_tok (), "") == 0); + } + + /* Test parsing a range followed by a non-integer. */ + { + number_or_range_parser one_three_after ("1-3 after"); + + for (int i = 1; i < 4; i++) + { + SELF_CHECK (!one_three_after.finished ()); + SELF_CHECK (one_three_after.get_number () == i); + } + SELF_CHECK (one_three_after.finished ()); + SELF_CHECK (strcmp (one_three_after.cur_tok (), "after") == 0); + } + + /* Test a negative integer gives an error. */ + { + number_or_range_parser minus_one ("-1"); + + SELF_CHECK (!minus_one.finished ()); + TRY + { + minus_one.get_number (); + SELF_CHECK (false); + } + CATCH (ex, RETURN_MASK_ERROR) + { + SELF_CHECK (ex.reason == RETURN_ERROR); + SELF_CHECK (ex.error == GENERIC_ERROR); + SELF_CHECK (strcmp (ex.message, "negative value") == 0); + SELF_CHECK (strcmp (minus_one.cur_tok (), "-1") == 0); + } + END_CATCH; + } + + /* Test that a - followed by not a number does not give an error. */ + { + number_or_range_parser nan ("-whatever"); + + SELF_CHECK (nan.finished ()); + SELF_CHECK (strcmp (nan.cur_tok (), "-whatever") == 0); + } +} + +static void +test_parse_flags () +{ + const char *flags = "abc"; + const char *non_flags_args = "non flags args"; + + /* Extract twice the same flag, separated by one space. */ + { + const char *t1 = "-a -a non flags args"; + + SELF_CHECK (parse_flags (&t1, flags) == 1); + SELF_CHECK (parse_flags (&t1, flags) == 1); + SELF_CHECK (strcmp (t1, non_flags_args) == 0); + } + + /* Extract some flags, separated by one or more spaces. */ + { + const char *t2 = "-c -b -c -b -c non flags args"; + + SELF_CHECK (parse_flags (&t2, flags) == 3); + SELF_CHECK (parse_flags (&t2, flags) == 2); + SELF_CHECK (parse_flags (&t2, flags) == 3); + SELF_CHECK (parse_flags (&t2, flags) == 2); + SELF_CHECK (parse_flags (&t2, flags) == 3); + SELF_CHECK (strcmp (t2, non_flags_args) == 0); + } + + /* Check behaviour where there is no flag to extract. */ + { + const char *t3 = non_flags_args; + + SELF_CHECK (parse_flags (&t3, flags) == 0); + SELF_CHECK (strcmp (t3, non_flags_args) == 0); + } + + /* Extract 2 known flags in front of unknown flags. */ + { + const char *t4 = "-c -b -x -y -z -c"; + + SELF_CHECK (parse_flags (&t4, flags) == 3); + SELF_CHECK (parse_flags (&t4, flags) == 2); + SELF_CHECK (strcmp (t4, "-x -y -z -c") == 0); + SELF_CHECK (parse_flags (&t4, flags) == 0); + SELF_CHECK (strcmp (t4, "-x -y -z -c") == 0); + } + + /* Check combined flags are not recognised. */ + { + const char *t5 = "-c -cb -c"; + + SELF_CHECK (parse_flags (&t5, flags) == 3); + SELF_CHECK (parse_flags (&t5, flags) == 0); + SELF_CHECK (strcmp (t5, "-cb -c") == 0); + } +} + +static void +test_parse_flags_qcs () +{ + const char *non_flags_args = "non flags args"; + + /* Test parsing of 2 flags out of the known 3. */ + { + const char *t1 = "-q -s non flags args"; + qcs_flags flags; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t1.q", + &t1, + &flags) == 1); + SELF_CHECK (flags.quiet && !flags.cont && !flags.silent); + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t1.s", + &t1, + &flags) == 1); + SELF_CHECK (flags.quiet && !flags.cont && flags.silent); + SELF_CHECK (strcmp (t1, non_flags_args) == 0); + } + + /* Test parsing when there is no flag. */ + { + const char *t2 = "non flags args"; + qcs_flags flags; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t2", + &t2, + &flags) == 0); + SELF_CHECK (!flags.quiet && !flags.cont && !flags.silent); + SELF_CHECK (strcmp (t2, non_flags_args) == 0); + } + + /* Test parsing stops at a negative integer. */ + { + const char *t3 = "-123 non flags args"; + const char *orig_t3 = t3; + qcs_flags flags; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t3", + &t3, + &flags) == 0); + SELF_CHECK (!flags.quiet && !flags.cont && !flags.silent); + SELF_CHECK (strcmp (t3, orig_t3) == 0); + } + + /* Test mutual exclusion between -c and -s. */ + { + const char *t4 = "-c -s non flags args"; + qcs_flags flags; + + TRY + { + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t4.cs", + &t4, + &flags) == 1); + + (void) parse_flags_qcs ("test_parse_flags_qcs.t4.cs", + &t4, + &flags); + SELF_CHECK (false); + } + CATCH (ex, RETURN_MASK_ERROR) + { + SELF_CHECK (ex.reason == RETURN_ERROR); + SELF_CHECK (ex.error == GENERIC_ERROR); + SELF_CHECK + (strcmp (ex.message, + "test_parse_flags_qcs.t4.cs: " + "-c and -s are mutually exclusive") == 0); + } + END_CATCH; + } + +} + +static void +test_cli_utils () +{ + selftests::cli_utils::test_number_or_range_parser (); + selftests::cli_utils::test_parse_flags (); + selftests::cli_utils::test_parse_flags_qcs (); +} + +} +} + +void +_initialize_cli_utils_selftests () +{ + selftests::register_test ("cli_utils", + selftests::cli_utils::test_cli_utils); +}