Maven Shared Components

Verifier executeGoal method is OS-dependent in case of failure

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: maven-verifier-1.0
  • Fix Version/s: maven-verifier-1.2
  • Component/s: maven-verifier
  • Labels:
    None
  • Number of attachments :
    0

Description

On windows Verifier.executeGoal() runs ok even if the underneath execution fails (this is good since there is a special method to make sure that everything went ok).

On linux/macOSx, a VerificationException is thrown (Exit code was non-zero)

This OS-dependent behavior is an issue when one wants to test that a given project is actually expected to fail (see project-026 in the EAR plugin for instance).

Issue Links

duplicates

Bug - A problem which impairs or prevents the functions of the product. MSHARED-73 Verifier doesn't throw VerificationException upon non-zero exit code of mvn.bat on Windows

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Stéphane Nicoll made changes -
Field Original Value New Value
Assignee Jason van Zyl [ jason ]
Jason van Zyl made changes -
Fix Version/s Reviewed [ 13555 ]
Benjamin Bentmann made changes -
Key MNG-2718 MSHARED-76
Affects Version/s maven-verifier 1.0 [ 14581 ]
Component/s Integration Tests [ 11866 ]
Fix Version/s Reviewed Pending Version Assignment [ 13555 ]
Component/s maven-verifier [ 13281 ]
Project Maven 2 [ 10500 ] Maven Shared Components [ 11761 ]
Fix Version/s maven-verifier 1.2 [ 14582 ]
Affects Version/s 2.0.4 [ 12527 ]
Benjamin Bentmann made changes -
Link This issue duplicates MSHARED-73 [ MSHARED-73 ]
Hide
Permalink
Benjamin Bentmann added a comment -

Fixed as per MSHARED-73.

Show
Benjamin Bentmann added a comment - Fixed as per MSHARED-73 .
Benjamin Bentmann made changes -
Assignee Jason van Zyl [ jason ] Benjamin Bentmann [ bentmann ]
Status Open [ 1 ] Closed [ 6 ]
Resolution Duplicate [ 3 ]

People

  • Assignee:
    Benjamin Bentmann
    Reporter:
    Stéphane Nicoll
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: