fiveam-devel

Download

fiveam-devel@common-lisp.net

September 2019

2 participants
1 discussions

PATCH: SIGNAL a condition on test failures, for use with ASDF:TEST-SYSTEM
by Vladimir Sedach 15 Sep '19

15 Sep '19

Hello! I have a proposed patch to FiveAM on GitHub: https://github.com/sionescu/fiveam/pull/58 I recently wrote a script to run FiveAM tests for one of my libraries on many different implementations on your own machine: https://gitlab.common-lisp.net/uri-template/uri-template2/blob/master/run-t… I do not want to copy-paste that script for all of my libraries. It would be nice to contribute a generalized version of the script to Roswell (on which the script is based), and just be able to say "ros test my-system in all installed implementations" for any system. The first step to doing that is to get ASDF:TEST-SYSTEM to report errors using a common interface. Because of the way ASDF is designed, ASDF:TEST-SYSTEM needs to use conditions to signal test failures. I like this idea. Since you do not have to handle conditions raised by SIGNAL, the functionality can be added without impacting existing use or interfaces of FiveAM (the use of WARNING instead of ERROR as the parent condition of TEST-SPEC-FAILURE also ensures this, as many people tend to abuse HANDLER-CASE to indiscriminately catch ERRORs). With this patch, all projects that use FiveAM and define ASDF test-op should be testable from a generic version of the Roswell script without any changes. The basic mechanism for test automation with FiveAM is then very simple: (handler-case (asdf:test-system "system") (fiveam:test-spec-failure (condition) (princ condition uiop:*stderr*) (uiop:quit 1))) Other test libraries can have supported added as well: (handler-case (asdf:test-system "system") ((or fiveam:test-spec-failure prove:test-failure) (condition) (princ condition uiop:*stderr*) (uiop:quit 1))) The goal is to introduce a parent condition in ASDF that FiveAM, etc., will then inherit from: (handler-case (asdf:test-system "system") (asdf:test-failure (condition) (princ condition uiop:*stderr*) (uiop:quit 1))) Ultimately, any system using any test library should be testable this way, ensuring that test automation becomes trivial to build. I am going to propose adding the parent condition to ASDF, but in the meantime (until the ASDF patch makes it to the repository, another release comes out, release becomes widely available…), I would like to get this code into FiveAM. Vladimir

2 2