MRUnit
  1. MRUnit
  2. MRUNIT-74

Matching output but at incorrect position is not returned as an Error in the message string thrown in the RuntimeException

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.8.1
    • Fix Version/s: 0.9.0
    • Labels:
      None

      Description

      It seems strange to me to not return "0 Error(s): ()" in the now RuntimeException but soon to be AssertionError when the output is correctly matched but in the incorrect order

        Activity

        Jim Donofrio created issue -
        Jim Donofrio made changes -
        Field Original Value New Value
        Fix Version/s 1.0.0 [ 12316360 ]
        Jim Donofrio made changes -
        Assignee Jim Donofrio [ jdonofrio ]
        Hide
        Jim Donofrio added a comment -

        Edited lookupExpectedValue to also take a List<String> errors in order to add the logged errors relating to "Received unexpected output" and "Matched expected output but at incorrect position" to the exception message

        Also corresponding with MRUNIT-48 throw more meaningful error for Received unexpected output when key, value actual class doesnt match any of the expected classes under the assumption that only one concrete class (no subclasses) is used for the key and one class for the value.

        Updated tests to pass based on the exception message changing

        committed in 1299567

        Show
        Jim Donofrio added a comment - Edited lookupExpectedValue to also take a List<String> errors in order to add the logged errors relating to "Received unexpected output" and "Matched expected output but at incorrect position" to the exception message Also corresponding with MRUNIT-48 throw more meaningful error for Received unexpected output when key, value actual class doesnt match any of the expected classes under the assumption that only one concrete class (no subclasses) is used for the key and one class for the value. Updated tests to pass based on the exception message changing committed in 1299567
        Jim Donofrio made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4d 4h 4m 1 Jim Donofrio 12/Mar/12 07:08

          People

          • Assignee:
            Jim Donofrio
            Reporter:
            Jim Donofrio
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development