[Intel-gfx] [PATCH i-g-t 1/2] lib: Add igt_subtest_group
Daniel Vetter
daniel.vetter at ffwll.ch
Sat Mar 19 09:33:45 UTC 2016
Useful for creating common setup code in igt_fixture which is only
needed by a subset of tests. And since I'm a good citizen it comes
with a library testcase/example included.
v2: Make testcase nastier to ensure a subtest group SKIPS when it's
parent is skipping already. I accidentally got this right, but let's
make sure.
Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
---
NEWS | 6 +++++
lib/igt_core.c | 14 +++++++++-
lib/igt_core.h | 24 +++++++++++++++++
lib/tests/.gitignore | 1 +
lib/tests/Makefile.sources | 1 +
lib/tests/igt_subtest_group.c | 63 +++++++++++++++++++++++++++++++++++++++++++
6 files changed, 108 insertions(+), 1 deletion(-)
create mode 100644 lib/tests/igt_subtest_group.c
diff --git a/NEWS b/NEWS
index a42382f3f6d3..6a3fac77a4d9 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,9 @@
+Release 1.15 (XXXX-XX-XX)
+-------------------------
+
+- Add igt_subtest_group to allow igt_fixture for only a subset of subtests
+ without skipping/failing all subsequent subtests.
+
Release 1.14 (2016-03-01)
-------------------------
diff --git a/lib/igt_core.c b/lib/igt_core.c
index 3c092a5f1651..947a9ca37690 100644
--- a/lib/igt_core.c
+++ b/lib/igt_core.c
@@ -97,7 +97,9 @@
*
* To allow this i-g-t provides #igt_fixture code blocks for setup code outside
* of subtests and automatically skips the subtest code blocks themselves. For
- * special cases igt_only_list_subtests() is also provided.
+ * special cases igt_only_list_subtests() is also provided. For setup code only
+ * shared by a group of subtest encapsulate the #igt_fixture block and all the
+ * subtestest in a #igt_subtest_group block.
*
* # Magic Control Blocks
*
@@ -889,6 +891,16 @@ bool igt_only_list_subtests(void)
return list_subtests;
}
+void __igt_subtest_group_save(int *save)
+{
+ *save = skip_subtests_henceforth;
+}
+
+void __igt_subtest_group_restore(int save)
+{
+ skip_subtests_henceforth = save;
+}
+
static bool skipped_one = false;
static bool succeeded_one = false;
static bool failed_one = false;
diff --git a/lib/igt_core.h b/lib/igt_core.h
index 124ae0d90f2f..b3fa7356e473 100644
--- a/lib/igt_core.h
+++ b/lib/igt_core.h
@@ -197,6 +197,30 @@ bool __igt_run_subtest(const char *subtest_name);
const char *igt_subtest_name(void);
bool igt_only_list_subtests(void);
+void __igt_subtest_group_save(int *);
+void __igt_subtest_group_restore(int);
+/**
+ * igt_subtest_group:
+ *
+ * Group a set of subtests together with their common setup code
+ *
+ * Testcase with subtests often need to set up a bunch of shared state as the
+ * common test fixture. But if there are multiple with different requirements
+ * the commont setup code can't be extracted, since a test condition failure in
+ * e.g. igt_require() would result in all subsequent tests skipping. Even those
+ * from a different group.
+ *
+ * This macro allows to group together a set of #igt_fixture and #igt_subtest
+ * clauses. If any common setup in a fixture fails, only the subtests in this
+ * group will fail or skip. Subtest groups can be arbitrarily nested.
+ */
+#define igt_subtest_group for (int igt_tokencat(__tmpint,__LINE__) = 0, \
+ igt_tokencat(__save,__LINE__) = 0; \
+ igt_tokencat(__tmpint,__LINE__) < 1 && \
+ (__igt_subtest_group_save(& igt_tokencat(__save,__LINE__) ), true); \
+ igt_tokencat(__tmpint,__LINE__) ++, \
+ __igt_subtest_group_restore(igt_tokencat(__save,__LINE__) ))
+
/**
* igt_main:
*
diff --git a/lib/tests/.gitignore b/lib/tests/.gitignore
index 661386a38dc8..842482c0a12c 100644
--- a/lib/tests/.gitignore
+++ b/lib/tests/.gitignore
@@ -11,4 +11,5 @@ igt_segfault
igt_simple_test_subtests
igt_simulation
igt_stats
+igt_subtest_group
igt_timeout
diff --git a/lib/tests/Makefile.sources b/lib/tests/Makefile.sources
index 09baeb127d22..707c445afca6 100644
--- a/lib/tests/Makefile.sources
+++ b/lib/tests/Makefile.sources
@@ -10,6 +10,7 @@ check_PROGRAMS = \
igt_timeout \
igt_invalid_subtest_name \
igt_segfault \
+ igt_subtest_group \
igt_assert \
igt_exit_handler \
$(NULL)
diff --git a/lib/tests/igt_subtest_group.c b/lib/tests/igt_subtest_group.c
new file mode 100644
index 000000000000..2b17955c5ac8
--- /dev/null
+++ b/lib/tests/igt_subtest_group.c
@@ -0,0 +1,63 @@
+/*
+ * Copyright © 2016 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
+ * 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 <assert.h>
+#include "igt_core.h"
+
+igt_main
+{
+ bool t1 = false;
+
+ igt_subtest_group {
+ igt_fixture {
+ igt_require(true);
+ }
+
+ igt_subtest_group {
+ igt_fixture {
+ igt_require(false);
+ }
+
+ igt_subtest("not-run") {
+ assert(0);
+ }
+
+ igt_subtest_group {
+ /* need to make sure we don't accidentally
+ * restore to "run testcases" when an outer
+ * group is already in SKIP state. */
+ igt_subtest("still-not-run") {
+ assert(0);
+ }
+ }
+ }
+
+ igt_subtest("run") {
+ t1 = true;
+ assert(1);
+ }
+ }
+
+ assert(t1);
+}
--
2.7.0
More information about the Intel-gfx
mailing list