Qpid
  1. Qpid
  2. QPID-3595

Python Alternate Exchange tests fail against the Java Broker

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.12
    • Fix Version/s: 0.15
    • Labels:
      None

      Description

      Python Alternate Exchange tests fail against the Java Broker

      qpid_tests.broker_0_10.alternate_exchange.AlternateExchangeTests.test_modify_existing_exchange_alternate
      qpid_tests.broker_0_10.alternate_exchange.AlternateExchangeTests.test_queue_autodelete
      qpid_tests.broker_0_10.alternate_exchange.AlternateExchangeTests.test_queue_delete_no_match
      qpid_tests.broker_0_10.alternate_exchange.AlternateExchangeTests.test_reject_no_match
      qpid_tests.broker_0_10.alternate_exchange.AlternateExchangeTests.test_add_alternate_to_exchange

        Activity

        Hide
        Robbie Gemmell added a comment -

        A new test was added that also fails: qpid_tests.broker_0_10.new_api.GeneralTests.test_qpid_3481_acquired_to_alt_exchange

        Show
        Robbie Gemmell added a comment - A new test was added that also fails: qpid_tests.broker_0_10.new_api.GeneralTests.test_qpid_3481_acquired_to_alt_exchange
        Hide
        Rob Godfrey added a comment -

        Remove tests from exclusion as they actually pass

        Show
        Rob Godfrey added a comment - Remove tests from exclusion as they actually pass
        Hide
        Rob Godfrey added a comment -

        Fixed the broker to allow for better alternate exchange handling, added short pauses to tests which were failing due to race condition (this is not a good solution - the tests should be rewritten).

        Show
        Rob Godfrey added a comment - Fixed the broker to allow for better alternate exchange handling, added short pauses to tests which were failing due to race condition (this is not a good solution - the tests should be rewritten).
        Hide
        Rob Godfrey added a comment -

        robbie - can you review for me please?

        Show
        Rob Godfrey added a comment - robbie - can you review for me please?
        Hide
        Robbie Gemmell added a comment -

        Looks good to me.

        Show
        Robbie Gemmell added a comment - Looks good to me.

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Andrew MacBean
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development