From patchwork Sun Jun 24 18:37:08 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Philippe Waroquiers X-Patchwork-Id: 28008 Received: (qmail 41541 invoked by alias); 24 Jun 2018 18:37:50 -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 41409 invoked by uid 89); 24 Jun 2018 18:37:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-26.6 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=whatever, makefilein, Makefile.in, Makefilein X-HELO: mailsec106.isp.belgacom.be Received: from mailsec106.isp.belgacom.be (HELO mailsec106.isp.belgacom.be) (195.238.20.102) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 24 Jun 2018 18:37:46 +0000 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A2BGCwCk5C9b/+ApQFdcHQEBBQELAYNJJ?= =?us-ascii?q?D5tEiiMXYtgAYI1AZRsgWYLHwwBgUuCdQKDAiI3FQECAQEBAQEBAgFsHAyCNSK?= =?us-ascii?q?CUgZWIxA/EjkeGQmDHAGCA65ohFuDZoECikI/hWmBQoEyhhECmGxDBwKBa4QTi?= =?us-ascii?q?QgLggKLRyuLGYYpgVcigVJtgzwJghoXhhuCPoVAPTB4ARoBjVUBAQ?= X-IPAS-Result: =?us-ascii?q?A2BGCwCk5C9b/+ApQFdcHQEBBQELAYNJJD5tEiiMXYtgAYI?= =?us-ascii?q?1AZRsgWYLHwwBgUuCdQKDAiI3FQECAQEBAQEBAgFsHAyCNSKCUgZWIxA/EjkeG?= =?us-ascii?q?QmDHAGCA65ohFuDZoECikI/hWmBQoEyhhECmGxDBwKBa4QTiQgLggKLRyuLGYY?= =?us-ascii?q?pgVcigVJtgzwJghoXhhuCPoVAPTB4ARoBjVUBAQ?= Received: from 224.41-64-87.adsl-dyn.isp.belgacom.be (HELO md.home) ([87.64.41.224]) by relay.skynet.be with ESMTP/TLS/DHE-RSA-AES128-GCM-SHA256; 24 Jun 2018 20:37:17 +0200 From: Philippe Waroquiers To: gdb-patches@sourceware.org Cc: Philippe Waroquiers Subject: [RFA_v3 8/8] Add a self-test for cli-utils.c Date: Sun, 24 Jun 2018 20:37:08 +0200 Message-Id: <20180624183708.888-9-philippe.waroquiers@skynet.be> In-Reply-To: <20180624183708.888-1-philippe.waroquiers@skynet.be> References: <20180624183708.888-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-06-05 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 | 220 ++++++++++++++++++++++++++++ 2 files changed, 221 insertions(+) create mode 100644 gdb/unittests/cli-utils-selftests.c diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 354a6361b7..f8cdf9a560 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -416,6 +416,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..99414f097e --- /dev/null +++ b/gdb/unittests/cli-utils-selftests.c @@ -0,0 +1,220 @@ +/* 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 () +{ + number_or_range_parser one ("1"); + + SELF_CHECK (one.finished () == false); + SELF_CHECK (one.get_number () == 1); + SELF_CHECK (one.finished () == true); + SELF_CHECK (strcmp (one.cur_tok (), "") == 0); + + number_or_range_parser one_after ("1 after"); + + SELF_CHECK (one_after.finished () == false); + SELF_CHECK (one_after.get_number () == 1); + SELF_CHECK (one_after.finished () == true); + SELF_CHECK (strcmp (one_after.cur_tok (), "after") == 0); + + number_or_range_parser one_three ("1-3"); + + for (int i = 1; i < 4; i++) { + SELF_CHECK (one_three.finished () == false); + SELF_CHECK (one_three.get_number () == i); + } + SELF_CHECK (one_three.finished () == true); + SELF_CHECK (strcmp (one_three.cur_tok (), "") == 0); + + number_or_range_parser one_three_after ("1-3 after"); + + for (int i = 1; i < 4; i++) { + SELF_CHECK (one_three_after.finished () == false); + SELF_CHECK (one_three_after.get_number () == i); + } + SELF_CHECK (one_three_after.finished () == true); + SELF_CHECK (strcmp (one_three_after.cur_tok (), "after") == 0); + + number_or_range_parser minus_one ("-1"); + + SELF_CHECK (minus_one.finished () == false); + 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; + + number_or_range_parser nan ("-whatever"); + + SELF_CHECK (nan.finished () == true); + SELF_CHECK (one_three_after.get_number () == 0); + 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"; + int res; + + 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); + + 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); + + const char *t3 = non_flags_args; + + SELF_CHECK (parse_flags (&t3, flags) == 0); + SELF_CHECK (strcmp (t3, non_flags_args) == 0); + + const char *t4 = "-c -b -x -y -z -c"; + const char *orig_t4 = t4; + + SELF_CHECK (parse_flags (&t4, flags) == 3); + SELF_CHECK (parse_flags (&t4, flags) == 2); + SELF_CHECK (strcmp (t4, "-x -y -z -c") == 0); + + const char *t5 = "-c -cb -c"; + const char *orig_t5 = t5; + + 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 () +{ + bool quiet; + bool cont; + bool silent; + + const char *non_flags_args = "non flags args"; + + const char *t1 = "-q -s non flags args"; + quiet = false; + cont = false; + silent = false; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t1.q", + &t1, + &quiet, &cont, &silent) == 1); + SELF_CHECK (quiet == true && cont == false && silent == false); + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t1.s", + &t1, + &quiet, &cont, &silent) == 1); + SELF_CHECK (quiet == true && cont == false && silent == true); + SELF_CHECK (strcmp (t1, non_flags_args) == 0); + + const char *t2 = "non flags args"; + quiet = false; + cont = false; + silent = false; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t2", + &t2, + &quiet, &cont, &silent) == 0); + SELF_CHECK (quiet == false && cont == false && silent == false); + SELF_CHECK (strcmp (t2, non_flags_args) == 0); + + const char *t3 = "-123 non flags args"; + const char *orig_t3 = t3; + quiet = false; + cont = false; + silent = false; + + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t3", + &t3, + &quiet, &cont, &silent) == 0); + SELF_CHECK (quiet == false && cont == false && silent == false); + SELF_CHECK (strcmp (t3, orig_t3) == 0); + + const char *t4 = "-c -s non flags args"; + const char *orig_t4 = t4; + quiet = false; + cont = false; + silent = false; + TRY + { + SELF_CHECK (parse_flags_qcs ("test_parse_flags_qcs.t4.cs", + &t4, + &quiet, &cont, &silent) == 1); + + (void) parse_flags_qcs ("test_parse_flags_qcs.t4.cs", + &t4, + &quiet, &cont, &silent); + 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); +}