Trivial patch attached.
Created attachment 129363 [details] [review] test: Add missing error handling to some name-test tests Also take the opportunity to tweak the test-threads-init messages slightly to make it more TAP-compliant. It is not entirely TAP compliant because it doesn’t print a test plan before starting its tests. Coverity IDs: 54701, 54714, 54726 Signed-off-by: Philip Withnall <withnall@endlessm.com>
Comment on attachment 129363 [details] [review] test: Add missing error handling to some name-test tests Review of attachment 129363 [details] [review]: ----------------------------------------------------------------- Looks fine, thanks. > It is not entirely TAP compliant because it doesn't print a test plan before starting its tests. Actually, printing the test plan at the end is TAP-compliant, although planning in advance is weakly preferred: > The plan is optional but if there is a plan before the test points it must be the first non-diagnostic line output by the test file. In certain instances a test file may not know how many test points it will ultimately be running. In this case the plan can be the last non-diagnostic line in the output. > —http://testanything.org/tap-specification.html §(The plan)
Fixed in git for 1.11.10, thanks
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.