Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41773 )
Change subject: tests: Always run all unit tests
......................................................................
Patch Set 1:
Jan, I'm most unsure of this change: is there any dependency assumed between tests that would require us not to run "later" tests if "earlier" tests failed? Because that's what would change here.
On the other hand, make is free to reorder dependencies and run them in parallel, so I don't think we could have relied on that in the first place.
Anyway, something I'd like your input on.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/41773
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I407715f85513c2c95a1cf89cfb427317dff9fbab
Gerrit-Change-Number: 41773
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: Jan Dabros
jsd@semihalf.com
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 27 May 2020 11:38:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment