MRUnit
  1. MRUnit
  2. MRUNIT-15

runTests() should throw meaningful error messages

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.5.0
    • Fix Version/s: 0.8.0
    • Labels:
      None

      Description

      When runTests() fails to validate the output generated by a mapper or reducer, it throws a RuntimeException with no message and relies on logging output to indicate what was wrong. Instead it should throw meaningful exceptions (although they can still be unchecked).

      1. MRUNIT-15-1.patch
        2 kB
        Brock Noland
      2. MRUNIT-15-2.patch
        4 kB
        Brock Noland

        Activity

        E. Sammer created issue -
        E. Sammer made changes -
        Field Original Value New Value
        Fix Version/s 0.8.0 [ 12316359 ]
        Hide
        Brock Noland added a comment -

        Attached is a patch which implements this. All tests pass.

        Show
        Brock Noland added a comment - Attached is a patch which implements this. All tests pass.
        Brock Noland made changes -
        Attachment MRUNIT-15-1.patch [ 12496597 ]
        Hide
        Brock Noland added a comment -

        Forgot about the other drivers. Updated patch is attached.

        Show
        Brock Noland added a comment - Forgot about the other drivers. Updated patch is attached.
        Brock Noland made changes -
        Attachment MRUNIT-15-2.patch [ 12496599 ]
        Chris A. Mattmann made changes -
        Assignee Chris A. Mattmann [ chrismattmann ]
        Chris A. Mattmann made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Chris A. Mattmann added a comment -
        • patch committed in r1178100. Thanks Brock!
        Show
        Chris A. Mattmann added a comment - patch committed in r1178100. Thanks Brock!
        Chris A. Mattmann made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        127d 23h 8m 1 Chris A. Mattmann 01/Oct/11 23:30
        In Progress In Progress Resolved Resolved
        1m 10s 1 Chris A. Mattmann 01/Oct/11 23:31

          People

          • Assignee:
            Chris A. Mattmann
            Reporter:
            E. Sammer
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development