Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: None
-
Labels:None
-
Environment:v 1.0rc1
-
Number of attachments :
Description
Maven Dev Team,
I've discovered that using the goal "test:match" ignores the setting of
the property "maven.test.failure.ignore" whereas "test:test" abides by
the setting properly.
I was able to fix it in the maven-test-plugin-1.4/plugin.jelly file of
my
environment by using the line as stated in the test:test goal:
failureProperty="maven.test.failure"
and eliminating the hardcoded "halt on failure stuff". Here's the diff:
190,191c190
< haltonfailure="yes"
< haltonerror="yes"
—
> failureProperty="maven.test.failure"
254,255c253
< haltonfailure="yes"
< haltonerror="yes"
—
> failureProperty="maven.test.failure"
fixed. please be careful with patches - this one caused it never to fail because it isn't checking the failure property.
Also, patches should be attached, not inline, and should be in unified diff format (-u)