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

Add tests for junit TODOs

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Code Review (Initial)
    • Priority: Should
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • 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.

        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:

                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