Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Nightly Builds
    • Fix Version/s: None
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      here is the updated tests.

      I think I have finally conquered dumbster. (Basically the problem was waiting
      for the dumbster thread to start up properly)

      There are a few other issues, style complaints, etc. changed as well.

      I have tested applying it, and it ran fine.

      Any problems please let me know.

      PS. If we are all happy with this, then I will add futher validation of the
      sent emails as we have previously discussed.

        Activity

        Siegfried Goeschl made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Henri Yandell made changes -
        Affects Version/s 1.0 Alpha [ 12311833 ]
        Henri Yandell made changes -
        Affects Version/s 1.0 Alpha [ 12311665 ]
        Key COM-1691 EMAIL-43
        Assignee Jakarta Commons Developers Mailing List [ commons-dev@jakarta.apache.org ]
        Component/s Email [ 12311114 ]
        Project Commons [ 12310458 ] Commons Email [ 12310474 ]
        Henri Yandell made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 31970 12341843
        Hide
        David Eric Pugh added a comment -

        Heck yeah! It all worked. It is clearly the unicode problem messing up all
        the patches! Now that you changed the characters used, I think we are beyond
        that. If we aren't, then I will move those characters to a seperate Interface.

        Very happy to report every test passed.

        Show
        David Eric Pugh added a comment - Heck yeah! It all worked. It is clearly the unicode problem messing up all the patches! Now that you changed the characters used, I think we are beyond that. If we aren't, then I will move those characters to a seperate Interface. Very happy to report every test passed.
        Hide
        Corey Scott added a comment -

        last patch should get us down to 8 checkstyle complaints, I dont know what to
        do with these, any ideas?

        all unicode issues should be resolved.
        the dumbster start up problem, part of COM-1693 is still not resolved, the
        other 2 errors should be fixed by this patch and the previous one.

        Show
        Corey Scott added a comment - last patch should get us down to 8 checkstyle complaints, I dont know what to do with these, any ideas? all unicode issues should be resolved. the dumbster start up problem, part of COM-1693 is still not resolved, the other 2 errors should be fixed by this patch and the previous one.
        Hide
        Corey Scott added a comment -

        Created an attachment (id=13303)
        new patch, includes missing pieces from previous and extended tests for testing receved email contents

        Show
        Corey Scott added a comment - Created an attachment (id=13303) new patch, includes missing pieces from previous and extended tests for testing receved email contents
        Hide
        David Eric Pugh added a comment -

        Ok, I realized half the patch issues are the weird non unicode formatted
        characters in the array. I could apply everything but that! In the future,
        send me that directly in an email, maybe that way will apply better? Not sure...

        At any rate, everything went in. I am down to two errors, which are related to
        the aforementioned issue. Check it out!

        Show
        David Eric Pugh added a comment - Ok, I realized half the patch issues are the weird non unicode formatted characters in the array. I could apply everything but that! In the future, send me that directly in an email, maybe that way will apply better? Not sure... At any rate, everything went in. I am down to two errors, which are related to the aforementioned issue. Check it out!
        Hide
        Corey Scott added a comment -

        Created an attachment (id=13263)
        pls see bug desc

        Show
        Corey Scott added a comment - Created an attachment (id=13263) pls see bug desc
        Corey Scott created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Corey Scott
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development