Bug 6308 - [test] Skipped tests always say 'no reason given'
Summary: [test] Skipped tests always say 'no reason given'
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Regression Tests (show other bugs)
Version: 3.3.0
Hardware: All All
: P3 enhancement
Target Milestone: 3.4.2
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-26 12:38 UTC by Tom Schulz
Modified: 2018-09-04 15:09 UTC (History)
1 user (show)


Attachment Type Modified Status Actions Submitter/CLA Status
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tom Schulz 2010-01-26 12:38:12 UTC 
When tests are skipped, they say 'no reason given'. It would be helpful if
the reason for skipping a test could be given. The test that brings this
up for me is the sa_compile test. If a very old version of re2c in installed,
the output of the test is:

t/sa_compile......Argument "can't open -V" isn't numeric in addition (+)
at t/sa_compile.t line 18.
t/sa_compile......skipped: (no reason given)

It would be helpful if that test could say that the problem is the lack of a
usable re2c.

However, it would be good if all the tests could give a reason for being
skipped. When I first ran 'make test', I had to guess that the reason that
one test that I was interested in was being skipped was that the long tests
were not enabled.
Comment 1 Justin Mason 2010-03-23 16:33:16 UTC 
moving all open 3.3.1 bugs to 3.3.2
Comment 2 Karsten Bräckelmann 2010-03-23 17:42:30 UTC 
Moving back off of Security, which got changed by accident during the mass Target Milestone move.
Comment 3 Kevin A. McGrail 2013-06-21 16:11:59 UTC 
Moving all open bugs where target is defined and 3.4.0 or lower to 3.4.1 target
Comment 4 Kevin A. McGrail 2015-04-12 14:32:27 UTC 
Good idea, just need someone to contribute a patch.
Comment 5 Kevin A. McGrail 2018-09-04 15:09:12 UTC 
dependency info and tests have been improved for 3.4.2.  If there are more interest, it's likely not needed.  Read build/README.