Qpid
  1. Qpid
  2. QPID-2910

Small fix for python timeout tests

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.7
    • Component/s: None
    • Labels:
      None
    • Environment:

      Python test suite

      Description

      Currently the Connection class in qpid/messaging/endpoints initializes the transport with AMQP(S) first if an AMQP(S) url type is provided, but on the timeout tests the transport is proxied via the hangable so using an amqp:// URL breaks the timeout tests. This small patch changes the transport setup order, if a transport is passed in the options it gets set, only further we are checking for AMQP(S) url types and ultimately default to TCP.

        Activity

        Hide
        Sorin Suciu added a comment -

        Andrew, could you review and commit if all good please?

        Show
        Sorin Suciu added a comment - Andrew, could you review and commit if all good please?
        Hide
        Andrew Kennedy added a comment -

        Reviewed and committed patch from Sorin S <ssuciu@gmail.com>

        Show
        Andrew Kennedy added a comment - Reviewed and committed patch from Sorin S <ssuciu@gmail.com>

          People

          • Assignee:
            Andrew Kennedy
            Reporter:
            Sorin Suciu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development