Uploaded image for project: 'James Server'
  1. James Server
  2. JAMES-427

[PATCH] Introduce Unit Testing

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.3.0
    • None
    • None

    Description

      Patch, which incorporates Unit Testing into James project infrastructure.
      Since there was no reaction on the dev-list since I proposed it a few days ago, I'm uncertain if this is appreciated at all.

      I think unit testing could become vital for
      + helping QA
      + supporting refactorings (e.g. relaxing avalon dependencies)
      + testing while implementing new features (like IMAP)
      + reproducing bugs
      + testing fixes

      I know that it's a long way to go to reach very good test-coverage.
      But there is no risk in starting.
      My determination would be to add 2 to 3 test methods per week.

      It already helped me a lot understanding how parts of James work, although this is only my first test for this codebase.

      The patch utilizes JUnit 3.8.1 (junit.jar) and Ristretto (ristretto-1.0-all.jar, can be found under http://columbo.sourceforge.net) which the build script expects to find under /tools/lib but which are not attached here in the first place.

      tests are compiled and run invoking > build run-unit-tests

      I'd feel more than happy about comments how to improve this testing infrastructure.

      Attachments

        1. unittest.txt
          38 kB
          Bernd Fondermann
        2. unittest1.2.patch
          53 kB
          Bernd Fondermann
        3. unittest2.0.patch
          98 kB
          Bernd Fondermann
        4. unittest3.0.patch
          37 kB
          Bernd Fondermann
        5. unittest4.0.patch
          46 kB
          Bernd Fondermann
        6. unittest5.0.patch
          4 kB
          Bernd Fondermann
        7. unittest6.0.patch
          9 kB
          Bernd Fondermann
        8. website.patch
          0.9 kB
          Bernd Fondermann

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            bago Stefano Bagnara
            brainlounge Bernd Fondermann
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment