Gracefully handle puglRealize() being called twice

1 files changed, 100 insertions, 0 deletions

diff --git a/test/test_realize.c b/test/test_realize.c
new file mode 100644
index 0000000..4a12d1d
--- /dev/null
+++ b/test/test_realize.c
@@ -0,0 +1,100 @@
+/*
+  Copyright 2020 David Robillard <d@drobilla.net>
+
+  Permission to use, copy, modify, and/or distribute this software for any
+  purpose with or without fee is hereby granted, provided that the above
+  copyright notice and this permission notice appear in all copies.
+
+  THIS SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+  ANY SPECIAL, DIRECT, 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.
+*/
+
+/*
+  Tests that realize sends a create event, and can safely be called twice.
+
+  Without handling this case, an application that accidentally calls realize
+  twice could end up in a very confusing situation where multiple windows have
+  been allocated (and ultimately leaked) for a view.
+*/
+
+#undef NDEBUG
+
+#include "test_utils.h"
+
+#include "pugl/pugl.h"
+#include "pugl/pugl_stub.h"
+
+#include <assert.h>
+#include <stdbool.h>
+#include <stddef.h>
+
+typedef enum {
+	START,
+	CREATED,
+} State;
+
+typedef struct {
+	PuglWorld*      world;
+	PuglView*       view;
+	PuglTestOptions opts;
+	State           state;
+} PuglTest;
+
+static PuglStatus
+onEvent(PuglView* view, const PuglEvent* event)
+{
+	PuglTest* test = (PuglTest*)puglGetHandle(view);
+
+	if (test->opts.verbose) {
+		printEvent(event, "Event: ", true);
+	}
+
+	switch (event->type) {
+	case PUGL_CREATE:
+		assert(test->state == START);
+		test->state = CREATED;
+		break;
+	default:
+		break;
+	}
+
+	return PUGL_SUCCESS;
+}
+
+int
+main(int argc, char** argv)
+{
+	PuglTest test = {puglNewWorld(PUGL_PROGRAM, 0),
+	                 NULL,
+	                 puglParseTestOptions(&argc, &argv),
+	                 START};
+
+	// Set up view
+	test.view = puglNewView(test.world);
+	puglSetClassName(test.world, "Pugl Test");
+	puglSetBackend(test.view, puglStubBackend());
+	puglSetHandle(test.view, &test);
+	puglSetEventFunc(test.view, onEvent);
+	puglSetDefaultSize(test.view, 512, 512);
+
+	// Create initially invisible window
+	assert(!puglRealize(test.view));
+	assert(!puglGetVisible(test.view));
+	while (test.state < CREATED) {
+		assert(!puglUpdate(test.world, -1.0));
+	}
+
+	// Check that calling realize() again is okay
+	assert(puglRealize(test.view) == PUGL_FAILURE);
+
+	// Tear down
+	puglFreeView(test.view);
+	puglFreeWorld(test.world);
+
+	return 0;
+}