diff mbox

[i-g-t,1/4] intel-ci: Add comments on test order to fast-feedback.testlist

Message ID 1497008372-5720-1-git-send-email-petri.latvala@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Petri Latvala June 9, 2017, 11:39 a.m. UTC
Document the test ordering choices in fast-feedback.testlist.

For comments in testlists, piglit commit

 commit 0c535186d624071098c10003fdafe8f475ed9ae7
 Author: Petri Latvala <petri.latvala@intel.com>
 Date:   Wed Feb 1 12:57:45 2017 +0200

    framework/programs/run.py: Allow comments in test-list files.

is required.

Signed-off-by: Petri Latvala <petri.latvala@intel.com>
---
 tests/intel-ci/fast-feedback.testlist | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Arkadiusz Hiler June 9, 2017, 12:17 p.m. UTC | #1
On Fri, Jun 09, 2017 at 02:39:28PM +0300, Petri Latvala wrote:
> Document the test ordering choices in fast-feedback.testlist.
> 
> For comments in testlists, piglit commit
> 
>  commit 0c535186d624071098c10003fdafe8f475ed9ae7
>  Author: Petri Latvala <petri.latvala@intel.com>
>  Date:   Wed Feb 1 12:57:45 2017 +0200
> 
>     framework/programs/run.py: Allow comments in test-list files.
> 
> is required.
> 
> Signed-off-by: Petri Latvala <petri.latvala@intel.com>

The whole series:
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
diff mbox

Patch

diff --git a/tests/intel-ci/fast-feedback.testlist b/tests/intel-ci/fast-feedback.testlist
index 5ffa2ce..ce7a41f 100644
--- a/tests/intel-ci/fast-feedback.testlist
+++ b/tests/intel-ci/fast-feedback.testlist
@@ -1,3 +1,5 @@ 
+# Keep alphabetically sorted by default
+
 igt@core_auth@basic-auth
 igt@core_prop_blob@basic
 igt@drv_getparams_basic@basic-eu-total
@@ -270,6 +272,11 @@  igt@vgem_basic@dmabuf-mmap
 igt@vgem_basic@mmap
 igt@vgem_basic@second-client
 igt@vgem_basic@sysfs
+
+# All tests that do module unloading and reloading are executed last.
+# They will sometimes reveal issues of earlier tests leaving the
+# driver in a broken state that is not otherwise noticed in that test.
+
 igt@vgem_basic@unload
 igt@drv_module_reload@basic-reload
 igt@drv_module_reload@basic-no-display