Qpid
  1. Qpid
  2. QPID-3857

Test ConnectionTest#testClientIDVerificationForSameUser fails occasionally

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.14
    • Fix Version/s: 0.15
    • Component/s: Java Broker, Java Tests
    • Labels:
      None

      Description

      Test ConnectionTest#testClientIDVerificationForSameUser fails occasionally with the following error:

      The client should throw a ConnectionException stating the client ID is not unique

      Stacktrace

      junit.framework.AssertionFailedError: The client should throw a ConnectionException stating the client ID is not unique
      at org.apache.qpid.test.unit.client.connection.ConnectionTest.testClientIDVerificationForSameUser(ConnectionTest.java:315)
      at org.apache.qpid.test.utils.QpidBrokerTestCase.runBare(QpidBrokerTestCase.java:237)
      at org.apache.qpid.test.utils.QpidTestCase.run(QpidTestCase.java:138)

      Test failed on profile java-mms-spawn.0-10

        Activity

        Alex Rudyy created issue -
        Alex Rudyy made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Alex Rudyy added a comment -

        A patch with a fix is attached

        Show
        Alex Rudyy added a comment - A patch with a fix is attached
        Alex Rudyy made changes -
        Alex Rudyy made changes -
        Status In Progress [ 3 ] Ready To Review [ 10006 ]
        Hide
        Alex Rudyy added a comment -

        Hi Robbie,

        Could you please review and commit the patch attached?

        Show
        Alex Rudyy added a comment - Hi Robbie, Could you please review and commit the patch attached?
        Alex Rudyy made changes -
        Assignee Alex Rudyy [ alex.rufous ] Robbie Gemmell [ gemmellr ]
        Robbie Gemmell made changes -
        Affects Version/s 0.14 [ 12316855 ]
        Affects Version/s 0.15 [ 12319043 ]
        Hide
        Robbie Gemmell added a comment -

        I have applied the patch, then gone a little further and removed the now-redundant impl method it updated.

        Show
        Robbie Gemmell added a comment - I have applied the patch, then gone a little further and removed the now-redundant impl method it updated.
        Robbie Gemmell made changes -
        Status Ready To Review [ 10006 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Rob Godfrey made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        20m 28s 1 Alex Rudyy 20/Feb/12 13:47
        In Progress In Progress Reviewable Reviewable
        55m 44s 1 Alex Rudyy 20/Feb/12 14:43
        Reviewable Reviewable Resolved Resolved
        1h 36m 1 Robbie Gemmell 20/Feb/12 16:19
        Resolved Resolved Closed Closed
        1087d 3h 47m 1 Rob Godfrey 11/Feb/15 20:06

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Alex Rudyy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development