Commons Email
  1. Commons Email
  2. EMAIL-32

[email] tests fail on Unix c/o port settings

        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: Mac OS X 10.0
          Platform: Macintosh

          Description

          this bug was patched and has sneaked in again. Port testings during tests weren't being set to the
          values set in EmailConfiguration. Submitting a patch that fixes this.

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Mark Lowe added a comment -

            Created an attachment (id=13554)
            [email] smtp port setting *nix during tests

            Show
            Mark Lowe added a comment - Created an attachment (id=13554) [email] smtp port setting *nix during tests
            Hide
            Permalink
            Corey Scott added a comment -

            Created an attachment (id=13569)
            Changes the port number to 2500 from 25 (as this is reserved on unix)

            Show
            Corey Scott added a comment - Created an attachment (id=13569) Changes the port number to 2500 from 25 (as this is reserved on unix)
            Hide
            Permalink
            Corey Scott added a comment -
                • COM-1693 has been marked as a duplicate of this bug. ***
            Show
            Corey Scott added a comment - COM-1693 has been marked as a duplicate of this bug. ***
            Hide
            Permalink
            Corey Scott added a comment -

            Created an attachment (id=13571)
            previous patch was invalid, resubmitting

            Show
            Corey Scott added a comment - Created an attachment (id=13571) previous patch was invalid, resubmitting
            Hide
            Permalink
            David Eric Pugh added a comment -

            Okay, I've applied this patch. When CVS is working I'll commit it.

            Show
            David Eric Pugh added a comment - Okay, I've applied this patch. When CVS is working I'll commit it.
            Hide
            Permalink
            David Eric Pugh added a comment -

            Committed. Can someone test this and verify the fix?

            Show
            David Eric Pugh added a comment - Committed. Can someone test this and verify the fix?
            Hide
            Permalink
            Joe Germuska added a comment -

            I have run the tests on Mac OS X and verified that they work without being 'root'.

            Show
            Joe Germuska added a comment - I have run the tests on Mac OS X and verified that they work without being 'root'.

              People

              • Assignee:
                Unassigned
                Reporter:
                Mark Lowe
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development