[PATCH 4/4] Added tests for the vertex clipping code.
Sam Spilsbury
smspillaz at gmail.com
Thu Sep 12 19:01:22 PDT 2013
This tests (via the table-driven testing method) that the correct
number of vertices and also the correct vertices themselves
are generated for an clip box and polygon of up to eight vertices.
Also add a libshared-test.la so that we don't have to build weston-test-runner
all the time
---
tests/Makefile.am | 37 +++++---
tests/vertex-clip-test.c | 219 +++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 245 insertions(+), 11 deletions(-)
create mode 100644 tests/vertex-clip-test.c
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 3a36619..5be52c6 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -1,7 +1,8 @@
TESTS = $(shared_tests) $(module_tests) $(weston_tests)
shared_tests = \
- config-parser.test
+ config-parser.test \
+ vertex-clip.test
module_tests = \
surface-test.la \
@@ -48,13 +49,6 @@ AM_CPPFLAGS = \
-DUNIT_TEST \
$(COMPOSITOR_CFLAGS)
-
-config_parser_test_LDADD = \
- ../shared/libshared.la \
- $(COMPOSITOR_LIBS)
-config_parser_test_SOURCES = \
- config-parser-test.c
-
surface_global_test_la_SOURCES = surface-global-test.c
surface_global_test_la_LDFLAGS = -module -avoid-version -rpath $(libdir)
surface_test_la_SOURCES = surface-test.c
@@ -72,17 +66,38 @@ weston_test_la_SOURCES = \
weston_test_runner_src = \
weston-test-runner.c \
weston-test-runner.h
+
+check_LTLIBRARIES = libshared-test.la
+
+libshared_test_la_SOURCES = \
+ $(weston_test_runner_src)
+libshared_test_la_CFLAGS = $(GCC_CFLAGS) $(COMPOSITOR_CFLAGS)
+
+config_parser_test_LDADD = \
+ ../shared/libshared.la \
+ libshared-test.la \
+ $(COMPOSITOR_LIBS)
+config_parser_test_SOURCES = \
+ config-parser-test.c
+vertex_clip_test_SOURCES = \
+ vertex-clip-test.c \
+ ../src/vertex-clipping.c \
+ ../src/vertex-clipping.h
+vertex_clip_test_LDADD = \
+ libshared-test.la \
+ -lm -lrt
+
weston_test_client_src = \
weston-test-client-helper.c \
weston-test-client-helper.h \
wayland-test-protocol.c \
wayland-test-client-protocol.h \
subsurface-protocol.c \
- subsurface-client-protocol.h \
- $(weston_test_runner_src)
+ subsurface-client-protocol.h
weston_test_client_libs = \
$(SIMPLE_CLIENT_LIBS) \
- ../shared/libshared.la
+ ../shared/libshared.la \
+ libshared-test.la
keyboard_weston_SOURCES = keyboard-test.c $(weston_test_client_src)
keyboard_weston_LDADD = $(weston_test_client_libs)
diff --git a/tests/vertex-clip-test.c b/tests/vertex-clip-test.c
new file mode 100644
index 0000000..5b2e08c
--- /dev/null
+++ b/tests/vertex-clip-test.c
@@ -0,0 +1,219 @@
+/*
+ * Copyright © 2013 Sam Spilsbury <smspillaz at gmail.com>
+ *
+ * Permission to use, copy, modify, distribute, and sell this software and
+ * its documentation for any purpose is hereby granted without fee, provided
+ * that the above copyright notice appear in all copies and that both that
+ * copyright notice and this permission notice appear in supporting
+ * documentation, and that the name of the copyright holders not be used in
+ * advertising or publicity pertaining to distribution of the software
+ * without specific, written prior permission. The copyright holders make
+ * no representations about the suitability of this software for any
+ * purpose. It is provided "as is" without express or implied warranty.
+ *
+ * THE COPYRIGHT HOLDERS DISCLAIM ALL WARRANTIES WITH REGARD TO THIS
+ * SOFTWARE, INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND
+ * FITNESS, IN NO EVENT SHALL THE COPYRIGHT HOLDERS BE LIABLE FOR ANY
+ * SPECIAL, INDIRECT OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER
+ * RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF
+ * CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
+ * CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+ */
+#include <assert.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <math.h>
+
+#include "weston-test-runner.h"
+
+#include "../src/vertex-clipping.h"
+
+#define BOUNDING_BOX_TOP_Y 100.0f
+#define BOUNDING_BOX_LEFT_X 50.0f
+#define BOUNDING_BOX_RIGHT_X 100.0f
+#define BOUNDING_BOX_BOTTOM_Y 50.0f
+
+#define INSIDE_X1 (BOUNDING_BOX_LEFT_X + 1.0f)
+#define INSIDE_X2 (BOUNDING_BOX_RIGHT_X - 1.0f)
+#define INSIDE_Y1 (BOUNDING_BOX_BOTTOM_Y + 1.0f)
+#define INSIDE_Y2 (BOUNDING_BOX_TOP_Y - 1.0f)
+
+#define OUTSIDE_X1 (BOUNDING_BOX_LEFT_X - 1.0f)
+#define OUTSIDE_X2 (BOUNDING_BOX_RIGHT_X + 1.0f)
+#define OUTSIDE_Y1 (BOUNDING_BOX_BOTTOM_Y - 1.0f)
+#define OUTSIDE_Y2 (BOUNDING_BOX_TOP_Y + 1.0f)
+
+static void
+populate_clip_context (struct clip_context *ctx)
+{
+ ctx->clip.x1 = BOUNDING_BOX_LEFT_X;
+ ctx->clip.y1 = BOUNDING_BOX_BOTTOM_Y;
+ ctx->clip.x2 = BOUNDING_BOX_RIGHT_X;
+ ctx->clip.y2 = BOUNDING_BOX_TOP_Y;
+}
+
+static int
+clip_polygon (struct clip_context *ctx,
+ struct polygon8 *polygon,
+ GLfloat *vertices_x,
+ GLfloat *vertices_y)
+{
+ populate_clip_context(ctx);
+ return clip_transformed(ctx, polygon, vertices_x, vertices_y);
+}
+
+struct vertex_clip_test_data
+{
+ struct polygon8 surface;
+ struct polygon8 expected;
+};
+
+const struct vertex_clip_test_data test_data[] =
+{
+ /* All inside */
+ {
+ {
+ { INSIDE_X1, INSIDE_X2, INSIDE_X2, INSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ },
+ {
+ { INSIDE_X1, INSIDE_X2, INSIDE_X2, INSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ }
+ },
+ /* Top outside */
+ {
+ {
+ { INSIDE_X1, INSIDE_X2, INSIDE_X2, INSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, OUTSIDE_Y2, OUTSIDE_Y2 },
+ 4
+ },
+ {
+ { INSIDE_X1, INSIDE_X1, INSIDE_X2, INSIDE_X2 },
+ { BOUNDING_BOX_TOP_Y, INSIDE_Y1, INSIDE_Y1, BOUNDING_BOX_TOP_Y },
+ 4
+ }
+ },
+ /* Bottom outside */
+ {
+ {
+ { INSIDE_X1, INSIDE_X2, INSIDE_X2, INSIDE_X1 },
+ { OUTSIDE_Y1, OUTSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ },
+ {
+ { INSIDE_X1, INSIDE_X2, INSIDE_X2, INSIDE_X1 },
+ { BOUNDING_BOX_BOTTOM_Y, BOUNDING_BOX_BOTTOM_Y, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ }
+ },
+ /* Left outside */
+ {
+ {
+ { OUTSIDE_X1, INSIDE_X2, INSIDE_X2, OUTSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ },
+ {
+ { BOUNDING_BOX_LEFT_X, INSIDE_X2, INSIDE_X2, BOUNDING_BOX_LEFT_X },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ }
+ },
+ /* Right outside */
+ {
+ {
+ { INSIDE_X1, OUTSIDE_X2, OUTSIDE_X2, INSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ },
+ {
+ { INSIDE_X1, BOUNDING_BOX_RIGHT_X, BOUNDING_BOX_RIGHT_X, INSIDE_X1 },
+ { INSIDE_Y1, INSIDE_Y1, INSIDE_Y2, INSIDE_Y2 },
+ 4
+ }
+ },
+ /* Diamond extending from bounding box edges, clip to bounding box */
+ {
+ {
+ { BOUNDING_BOX_LEFT_X - 25, BOUNDING_BOX_LEFT_X + 25, BOUNDING_BOX_RIGHT_X + 25, BOUNDING_BOX_RIGHT_X - 25 },
+ { BOUNDING_BOX_BOTTOM_Y + 25, BOUNDING_BOX_TOP_Y + 25, BOUNDING_BOX_TOP_Y - 25, BOUNDING_BOX_BOTTOM_Y - 25 },
+ 4
+ },
+ {
+ { BOUNDING_BOX_LEFT_X, BOUNDING_BOX_LEFT_X, BOUNDING_BOX_RIGHT_X, BOUNDING_BOX_RIGHT_X },
+ { BOUNDING_BOX_BOTTOM_Y, BOUNDING_BOX_TOP_Y, BOUNDING_BOX_TOP_Y, BOUNDING_BOX_BOTTOM_Y },
+ 4
+ }
+ },
+ /* Diamond inside of bounding box edges, clip t bounding box, 8 resulting vertices */
+ {
+ {
+ { BOUNDING_BOX_LEFT_X - 12.5, BOUNDING_BOX_LEFT_X + 25, BOUNDING_BOX_RIGHT_X + 12.5, BOUNDING_BOX_RIGHT_X - 25 },
+ { BOUNDING_BOX_BOTTOM_Y + 25, BOUNDING_BOX_TOP_Y + 12.5, BOUNDING_BOX_TOP_Y - 25, BOUNDING_BOX_BOTTOM_Y - 12.5 },
+ 4
+ },
+ {
+ { BOUNDING_BOX_LEFT_X + 12.5, BOUNDING_BOX_LEFT_X, BOUNDING_BOX_LEFT_X, BOUNDING_BOX_LEFT_X + 12.5,
+ BOUNDING_BOX_RIGHT_X - 12.5, BOUNDING_BOX_RIGHT_X, BOUNDING_BOX_RIGHT_X, BOUNDING_BOX_RIGHT_X - 12.5 },
+ { BOUNDING_BOX_BOTTOM_Y, BOUNDING_BOX_BOTTOM_Y + 12.5, BOUNDING_BOX_TOP_Y - 12.5, BOUNDING_BOX_TOP_Y,
+ BOUNDING_BOX_TOP_Y, BOUNDING_BOX_TOP_Y - 12.5, BOUNDING_BOX_BOTTOM_Y + 12.5, BOUNDING_BOX_BOTTOM_Y },
+ 8
+ }
+ }
+};
+
+/* clip_polygon modifies the source operand and the test data must
+ * be const, so we need to deep copy it */
+static void
+deep_copy_polygon8(const struct polygon8 *src, struct polygon8 *dst)
+{
+ dst->n = src->n;
+ memcpy((void *) dst->x, src->x, sizeof (src->x));
+ memcpy((void *) dst->y, src->y, sizeof (src->y));
+}
+
+TEST_P(clip_polygon_n_vertices_emitted, test_data)
+{
+ struct vertex_clip_test_data *tdata = data;
+ struct clip_context ctx;
+ struct polygon8 polygon;
+ GLfloat vertices_x[8];
+ GLfloat vertices_y[8];
+ deep_copy_polygon8(&tdata->surface, &polygon);
+ int emitted = clip_polygon(&ctx, &polygon, vertices_x, vertices_y);
+
+ assert(emitted == tdata->expected.n);
+}
+
+TEST_P(clip_polygon_expected_vertices, test_data)
+{
+ struct vertex_clip_test_data *tdata = data;
+ struct clip_context ctx;
+ struct polygon8 polygon;
+ GLfloat vertices_x[8];
+ GLfloat vertices_y[8];
+ deep_copy_polygon8(&tdata->surface, &polygon);
+ int emitted = clip_polygon(&ctx, &polygon, vertices_x, vertices_y);
+ int i = 0;
+
+ for (; i < emitted; ++i)
+ {
+ assert(vertices_x[i] == tdata->expected.x[i]);
+ assert(vertices_y[i] == tdata->expected.y[i]);
+ }
+}
+
+TEST(float_difference_different)
+{
+ assert(float_difference(1.0f, 0.0f) == 1.0f);
+}
+
+TEST(float_difference_same)
+{
+ assert(float_difference(1.0f, 1.0f) == 0.0f);
+}
+
--
1.8.3.2
More information about the wayland-devel
mailing list