diff mbox

[i-g-t] tests: fix sub-test enumeration

Message ID 1392220467-12907-1-git-send-email-thomas.wood@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Wood Feb. 12, 2014, 3:54 p.m. UTC
Ensure sub-tests can be listed correctly by doing any test setup within
an igt_fixture block.

Signed-off-by: Thomas Wood <thomas.wood@intel.com>
---
 tests/gem_evict_alignment.c | 7 ++++---
 tests/gem_pwrite_pread.c    | 4 ++--
 2 files changed, 6 insertions(+), 5 deletions(-)

Comments

Daniel Vetter Feb. 12, 2014, 4:46 p.m. UTC | #1
On Wed, Feb 12, 2014 at 03:54:27PM +0000, Thomas Wood wrote:
> Ensure sub-tests can be listed correctly by doing any test setup within
> an igt_fixture block.
> 
> Signed-off-by: Thomas Wood <thomas.wood@intel.com>

Oops, I've missed some. Patch merged, thanks.
-Daniel
diff mbox

Patch

diff --git a/tests/gem_evict_alignment.c b/tests/gem_evict_alignment.c
index 15c4707..c478f18 100644
--- a/tests/gem_evict_alignment.c
+++ b/tests/gem_evict_alignment.c
@@ -189,11 +189,12 @@  igt_main
 
 	igt_skip_on_simulation();
 
-	igt_fixture
+	igt_fixture {
 		fd = drm_open_any();
 
-	igt_skip_on_f(intel_gen(intel_get_drm_devid(fd)) > 7,
-		      "BLIT commands not yet updated\n");
+		igt_skip_on_f(intel_gen(intel_get_drm_devid(fd)) > 7,
+			      "BLIT commands not yet updated\n");
+	}
 
 	igt_subtest("minor-normal") {
 		size = 1024 * 1024;
diff --git a/tests/gem_pwrite_pread.c b/tests/gem_pwrite_pread.c
index 1ae0820..8f768b3 100644
--- a/tests/gem_pwrite_pread.c
+++ b/tests/gem_pwrite_pread.c
@@ -380,9 +380,9 @@  int main(int argc, char **argv)
 
 		gem_set_caching(fd, src, 0);
 		gem_set_caching(fd, dst, 0);
-	}
 
-	devid = intel_get_drm_devid(fd);
+		devid = intel_get_drm_devid(fd);
+	}
 
 	igt_subtest("uncached-copy-correctness")
 		test_copy(fd, src, dst, tmp, object_size);