[gem5-dev] Change in gem5/gem5[develop]: tests,ext: Fix so ex/include regex are applied after defaults

2022-03-18 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/56830 )


 (

2 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the  
submitted one.
 )Change subject: tests,ext: Fix so ex/include regex are applied after  
defaults

..

tests,ext: Fix so ex/include regex are applied after defaults

The seldom used '--include-tags' and '--exclude-tags' flags allows a
testing user to remove and include tags from the search used by TestLib
to select tests. For example, by default the 'quick' tag is included as
part of the search of tests to run. The '--exclude-tags' flag could then
be used to remove the 'quick' tags from the search.

The TestLib framework was applying the regex these flags input before
the default flags. This meant if the user wished to remove a flag, it
was impossible. This is now applied after.

Change-Id: I569e0f8d6093ff5e5cdc76faff89c15e75ff297a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/56830
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M ext/testlib/main.py
1 file changed, 24 insertions(+), 1 deletion(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/ext/testlib/main.py b/ext/testlib/main.py
index 6087a8e..b9d8e93 100644
--- a/ext/testlib/main.py
+++ b/ext/testlib/main.py
@@ -125,7 +125,7 @@
 if tags is None:
 tags = tuple()

-filters = list(itertools.chain(tags, final_tags))
+filters = list(itertools.chain(final_tags, tags))
 string = 'Filtering suites with tags as follows:\n'
 filter_string = '\t\n'.join((str(f) for f in filters))
 log.test_log.trace(string + filter_string)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/56830
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I569e0f8d6093ff5e5cdc76faff89c15e75ff297a
Gerrit-Change-Number: 56830
Gerrit-PatchSet: 5
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: tests,ext: Fix so ex/include regex are applied after defaults

2022-02-15 Thread Bobby Bruce (Gerrit) via gem5-dev
Bobby Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/56830 )



Change subject: tests,ext: Fix so ex/include regex are applied after  
defaults

..

tests,ext: Fix so ex/include regex are applied after defaults

The seldom used '--include-tags' and '--exclude-tags' flags allows a
testing user to remove and include tags from the search used by TestLib
to select tests. For example, by default the 'quick' tag is included as
part of the search of tests to run. The '--exclude-tags' flag could then
be used to remove the 'quick' tags from the search.

The TestLib framework was applying the regex these flags input before
the default flags. This meant if the user wished to remove a flag, it
was impossible. This is now applied after.

Change-Id: I569e0f8d6093ff5e5cdc76faff89c15e75ff297a
---
M ext/testlib/main.py
1 file changed, 20 insertions(+), 1 deletion(-)



diff --git a/ext/testlib/main.py b/ext/testlib/main.py
index 6087a8e..b9d8e93 100644
--- a/ext/testlib/main.py
+++ b/ext/testlib/main.py
@@ -125,7 +125,7 @@
 if tags is None:
 tags = tuple()

-filters = list(itertools.chain(tags, final_tags))
+filters = list(itertools.chain(final_tags, tags))
 string = 'Filtering suites with tags as follows:\n'
 filter_string = '\t\n'.join((str(f) for f in filters))
 log.test_log.trace(string + filter_string)

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/56830
To unsubscribe, or for help writing mail filters, visit  
https://gem5-review.googlesource.com/settings


Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I569e0f8d6093ff5e5cdc76faff89c15e75ff297a
Gerrit-Change-Number: 56830
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s