Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-248

Add tests for junit TODOs

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Should
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Core 2.3.0
    • Component/s: None
    • Labels:
    • Complexity:
      Low

      Description

      There are TODO comments spread throughout the code that contain the word 'junit'. These represent where devs have thought "hmm, this could benefit from a junit test", but didn't have time/energy to write it at that moment.

      These tests need to actually be written, the TODOs removed.

      It might help to run code coverage locally to spot the untested line the TODO was for. Run mvn clean package && mvn jacoco:report report is then in api/target/site/jacoco/index.html.

        Gliffy Diagrams

          Attachments

            Activity

              People

              Assignee:
              jwnasambu Juliet Wamalwa
              Reporter:
              bwolfe Ben Wolfe
              Votes:
              0 Vote for this issue
              Watchers:
              15 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 3 days Original Estimate - 3 days
                  3d
                  Remaining:
                  Time Spent - 5 hours Remaining Estimate - 3 days
                  3d
                  Logged:
                  Time Spent - 5 hours Remaining Estimate - 3 days
                  5h