[Mesa-dev] [PATCH 2/8] util: Add tests for the string buffer
Nicolai Hähnle
nhaehnle at gmail.com
Wed Sep 6 08:44:53 UTC 2017
On 29.08.2017 21:56, Thomas Helland wrote:
> More tests could probably be added, but this should cover
> concatenation, resizing, clearing, formatted printing,
> and checking the length, so it should be quite complete.
>
> V2: Address review feedback from Timothy, plus fixes
> - Use a large enough char array
> - Actually test the formatted appending
> - Test that clear function resets string length
> ---
> configure.ac | 2 +
> src/util/Makefile.am | 3 +-
> src/util/tests/string_buffer/Makefile.am | 34 +++++++++
> src/util/tests/string_buffer/append_and_print.c | 99 +++++++++++++++++++++++++
> 4 files changed, 137 insertions(+), 1 deletion(-)
> create mode 100644 src/util/tests/string_buffer/Makefile.am
> create mode 100644 src/util/tests/string_buffer/append_and_print.c
>
> diff --git a/configure.ac b/configure.ac
> index 53d52f6d52..a0d7a67956 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2933,9 +2933,11 @@ AC_CONFIG_FILES([Makefile
> src/mesa/main/tests/Makefile
> src/util/Makefile
> src/util/tests/hash_table/Makefile
> + src/util/tests/string_buffer/Makefile
> src/util/xmlpool/Makefile
> src/vulkan/Makefile])
>
> +
> AC_OUTPUT
>
> # Fix up dependencies in *.Plo files, where we changed the extension of a
> diff --git a/src/util/Makefile.am b/src/util/Makefile.am
> index 4512dc99d5..7000202a36 100644
> --- a/src/util/Makefile.am
> +++ b/src/util/Makefile.am
> @@ -19,7 +19,8 @@
> # FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> # IN THE SOFTWARE.
>
> -SUBDIRS = xmlpool . tests/hash_table
> +SUBDIRS = . tests/hash_table \
> + . tests/string_buffer
xmlpool shouldn't be removed, and there should be no need to have '.' twice.
>
> include Makefile.sources
>
> diff --git a/src/util/tests/string_buffer/Makefile.am b/src/util/tests/string_buffer/Makefile.am
> new file mode 100644
> index 0000000000..60039a90d2
> --- /dev/null
> +++ b/src/util/tests/string_buffer/Makefile.am
> @@ -0,0 +1,34 @@
> +# Copyright © 2009 Intel Corporation
> +#
> +# Permission is hereby granted, free of charge, to any person obtaining a
> +# copy of this software and associated documentation files (the "Software"),
> +# to deal in the Software without restriction, including without limitation
> +# on the rights to use, copy, modify, merge, publish, distribute, sub
> +# license, and/or sell copies of the Software, and to permit persons to whom
> +# the Software is furnished to do so, subject to the following conditions:
> +#
> +# The above copyright notice and this permission notice (including the next
> +# paragraph) shall be included in all copies or substantial portions of the
> +# Software.
> +#
> +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +# FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL
> +# ADAM JACKSON BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
That's an odd version of the copyright statement :)
Cheers,
Nicolai
> +# IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> +# CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> +
> +AM_CPPFLAGS = \
> + -I$(top_srcdir)/include \
> + -I$(top_srcdir)/src/util \
> + $(DEFINES)
> +
> +LDADD = \
> + $(top_builddir)/src/util/libmesautil.la \
> + $(PTHREAD_LIBS) \
> + $(DLOPEN_LIBS)
> +
> +TESTS = append_and_print \
> + $()
> +
> +check_PROGRAMS = $(TESTS)
> diff --git a/src/util/tests/string_buffer/append_and_print.c b/src/util/tests/string_buffer/append_and_print.c
> new file mode 100644
> index 0000000000..71b857892f
> --- /dev/null
> +++ b/src/util/tests/string_buffer/append_and_print.c
> @@ -0,0 +1,99 @@
> +/*
> + * Copyright © 2017 Thomas Helland
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> + * IN THE SOFTWARE.
> + *
> + */
> +
> +#include <stdlib.h>
> +#include <stdio.h>
> +#include <string.h>
> +#include <assert.h>
> +#include "string_buffer.h"
> +
> +int
> +main(int argc, char **argv)
> +{
> + (void) argc;
> + (void) argv;
> +
> + struct _mesa_string_buffer *buf;
> + char *str1 = "test1";
> + char *str2 = "test2";
> + char *str3 = "test1test2";
> + char str4[80];
> + char str5[40];
> +
> + buf = _mesa_string_buffer_create(NULL, 6);
> +
> + /* The string terminator needs one byte, so there should be five left */
> + assert(_mesa_string_buffer_space_left(buf) == 5);
> +
> + _mesa_string_buffer_append(buf, str1);
> +
> + /* The string should now be full */
> + assert(_mesa_string_buffer_space_left(buf) == 0);
> +
> + /* The content should be equal */
> + assert(strcmp(buf->buf, str1) == 0);
> +
> + /* Add more, so that the string is resized */
> + _mesa_string_buffer_append(buf, str2);
> +
> + /* The size of the buffer should have been doubled.
> + * We have 5 + 5 + 1 characters, so there should be one left.
> + */
> + assert(_mesa_string_buffer_space_left(buf) == 1);
> +
> + /* The string should now be equal to str3 */
> + assert(strcmp(buf->buf, str3) == 0);
> +
> + /* Compile a string with some formatting */
> + sprintf(str4, "Testing formatting %d, %f", 100, 1.0);
> +
> + /* Clear the string buffer first */
> + _mesa_string_buffer_clear(buf);
> +
> + /* Check that the length of the string is reset */
> + assert(buf->length == 0);
> +
> + /* Then test printing some formatted text */
> + _mesa_string_buffer_printf(buf, "Testing formatting %d, %f", 100, 1.0);
> +
> + /* The string should now be equal to str4 */
> + assert(strcmp(buf->buf, &str4) == 0);
> +
> + /* Compile a string with some other formatting */
> + sprintf(str5, "Testing formatting %d, %x", 100, 0xDEADBEAF);
> +
> + /* Concatenate str5 to str4 */
> + strcat(str4, str5);
> +
> + /* Now use the formatted append function */
> + _mesa_string_buffer_printf(buf, "Testing formatting %d, %x", 100, 0xDEADBEAF);
> +
> + /* The string buffer should now be equal to str4 */
> + assert(strcmp(buf->buf, &str4) == 0);
> +
> + /* Finally, clean up after us */
> + _mesa_string_buffer_destroy(buf);
> +
> + return 0;
> +}
>
--
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.
More information about the mesa-dev
mailing list