[1/3] Add unit test for xml_escape_text

Message ID 1505214048-13445-1-git-send-email-simon.marchi@ericsson.com
State New, archived
Headers

Commit Message

Simon Marchi Sept. 12, 2017, 11 a.m. UTC
  The following patch modifies xml_escape_text, so I took the opportunity
to write a unit test for it.

gdb/ChangeLog:

	* Makefile.in (SUBDIR_UNITTESTS_SRCS): Add new source file.
	(SUBDIR_UNITTESTS_OBS): Add new object file.
	* unittests/xml-utils-selftests.c: New file.
---
 gdb/Makefile.in                     |  6 +++--
 gdb/unittests/xml-utils-selftests.c | 45 +++++++++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 2 deletions(-)
 create mode 100644 gdb/unittests/xml-utils-selftests.c
  

Comments

Simon Marchi Sept. 16, 2017, 12:23 p.m. UTC | #1
On 2017-09-12 13:00, Simon Marchi wrote:
> The following patch modifies xml_escape_text, so I took the opportunity
> to write a unit test for it.
> 
> gdb/ChangeLog:
> 
> 	* Makefile.in (SUBDIR_UNITTESTS_SRCS): Add new source file.
> 	(SUBDIR_UNITTESTS_OBS): Add new object file.
> 	* unittests/xml-utils-selftests.c: New file.
> ---
>  gdb/Makefile.in                     |  6 +++--
>  gdb/unittests/xml-utils-selftests.c | 45 
> +++++++++++++++++++++++++++++++++++++
>  2 files changed, 49 insertions(+), 2 deletions(-)
>  create mode 100644 gdb/unittests/xml-utils-selftests.c
> 
> diff --git a/gdb/Makefile.in b/gdb/Makefile.in
> index 2aa474e..9dfc117 100644
> --- a/gdb/Makefile.in
> +++ b/gdb/Makefile.in
> @@ -532,7 +532,8 @@ SUBDIR_UNITTESTS_SRCS = \
>  	unittests/offset-type-selftests.c \
>  	unittests/optional-selftests.c \
>  	unittests/ptid-selftests.c \
> -	unittests/scoped_restore-selftests.c
> +	unittests/scoped_restore-selftests.c \
> +	unittests/xml-utils-selftests.c
> 
>  SUBDIR_UNITTESTS_OBS = \
>  	array-view-selftests.o \
> @@ -541,7 +542,8 @@ SUBDIR_UNITTESTS_OBS = \
>  	offset-type-selftests.o \
>  	optional-selftests.o \
>  	ptid-selftests.o \
> -	scoped_restore-selftests.o
> +	scoped_restore-selftests.o \
> +	xml-utils-selftests.o
> 
>  # Opcodes currently live in one of two places.  Either they are in the
>  # opcode library, typically ../opcodes, or they are in a header file
> diff --git a/gdb/unittests/xml-utils-selftests.c
> b/gdb/unittests/xml-utils-selftests.c
> new file mode 100644
> index 0000000..14cc853
> --- /dev/null
> +++ b/gdb/unittests/xml-utils-selftests.c
> @@ -0,0 +1,45 @@
> +/* Unit tests for the xml-utils.c file.
> +
> +   Copyright (C) 2017 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 
> <http://www.gnu.org/licenses/>.  */
> +
> +#include "defs.h"
> +#include "xml-utils.h"
> +#include "selftest.h"
> +
> +namespace selftests {
> +namespace xml_utils {
> +
> +static void test_xml_escape_text ()
> +{
> +  const char *input = "<this isn't=\"xml\"> &";
> +  const char *expected_output = "&lt;this
> isn&apos;t=&quot;xml&quot;&gt; &amp;";
> +  char *actual_output = xml_escape_text (input);
> +
> +  SELF_CHECK (strcmp (actual_output, expected_output) == 0);
> +
> +  xfree (actual_output);
> +}
> +
> +}
> +}
> +
> +void
> +_initialize_xml_utils ()
> +{
> +  selftests::register_test 
> (selftests::xml_utils::test_xml_escape_text);
> +}

These patches are now pushed in.

Simon
  

Patch

diff --git a/gdb/Makefile.in b/gdb/Makefile.in
index 2aa474e..9dfc117 100644
--- a/gdb/Makefile.in
+++ b/gdb/Makefile.in
@@ -532,7 +532,8 @@  SUBDIR_UNITTESTS_SRCS = \
 	unittests/offset-type-selftests.c \
 	unittests/optional-selftests.c \
 	unittests/ptid-selftests.c \
-	unittests/scoped_restore-selftests.c
+	unittests/scoped_restore-selftests.c \
+	unittests/xml-utils-selftests.c
 
 SUBDIR_UNITTESTS_OBS = \
 	array-view-selftests.o \
@@ -541,7 +542,8 @@  SUBDIR_UNITTESTS_OBS = \
 	offset-type-selftests.o \
 	optional-selftests.o \
 	ptid-selftests.o \
-	scoped_restore-selftests.o
+	scoped_restore-selftests.o \
+	xml-utils-selftests.o
 
 # Opcodes currently live in one of two places.  Either they are in the
 # opcode library, typically ../opcodes, or they are in a header file
diff --git a/gdb/unittests/xml-utils-selftests.c b/gdb/unittests/xml-utils-selftests.c
new file mode 100644
index 0000000..14cc853
--- /dev/null
+++ b/gdb/unittests/xml-utils-selftests.c
@@ -0,0 +1,45 @@ 
+/* Unit tests for the xml-utils.c file.
+
+   Copyright (C) 2017 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 <http://www.gnu.org/licenses/>.  */
+
+#include "defs.h"
+#include "xml-utils.h"
+#include "selftest.h"
+
+namespace selftests {
+namespace xml_utils {
+
+static void test_xml_escape_text ()
+{
+  const char *input = "<this isn't=\"xml\"> &";
+  const char *expected_output = "&lt;this isn&apos;t=&quot;xml&quot;&gt; &amp;";
+  char *actual_output = xml_escape_text (input);
+
+  SELF_CHECK (strcmp (actual_output, expected_output) == 0);
+
+  xfree (actual_output);
+}
+
+}
+}
+
+void
+_initialize_xml_utils ()
+{
+  selftests::register_test (selftests::xml_utils::test_xml_escape_text);
+}