tests: Removed the ignoring of tests
This commit reverts
https://gem5-review.googlesource.com/c/public/gem5/+/23023.
It has proven to be an unpopular piece of functionality which makes it
too easy to silently ignore failing tests. The new policy will be to
remove/comment-out failing tests in the testing source and tag with Jira
issues as to why.
Change-Id: I17d69fc57a9171ce3702e019615390a9aa3da250
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/33139
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
diff --git a/tests/gem5/suite.py b/tests/gem5/suite.py
index 329e31f..cba3d43 100644
--- a/tests/gem5/suite.py
+++ b/tests/gem5/suite.py
@@ -88,15 +88,6 @@
fixtures = list(fixtures)
testsuites = []
- # Obtain the set of tests to ignore. This is found in the
- # ".testignore" file.
- __location__ = os.path.realpath(
- os.path.join(os.getcwd(), os.path.dirname(__file__)))
- _test_ignore_file_loc = os.path.join(__location__,".testignore")
- ignore = set()
- if os.path.exists(_test_ignore_file_loc):
- ignore.update(open(_test_ignore_file_loc).read().splitlines())
-
for host in valid_hosts:
for opt in valid_variants:
for isa in valid_isas:
@@ -115,11 +106,6 @@
if protocol:
_name += '-'+protocol
- # We check to see if this test suite is to be ignored. If so,
- # we skip it.
- if _name in ignore:
- continue
-
# Create the running of gem5 subtest. NOTE: We specifically
# create this test before our verifiers so this is listed
# first.