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


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


      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.


        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


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


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




                Issue deployment