[PATCH i-g-t 1/2] lib: Add igt_subtest_group

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxx>
---
 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

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux