Qpid Proton
  1. Qpid Proton
  2. PROTON-431

Ruby integration test have incorrect failure messages

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.6
    • Component/s: proton-c
    • Labels:
      None

      Description

      The test current returns the value of the incoming and outgoing messenger queues. But the assert library wants either a text message or a Proc object, not a Fixnum. So the test suite consistently fails even when the actual test succeeded.

      Additionally, there are changes in the APIs that are causing the spec tests to fail as well.

        Activity

        Hide
        ASF subversion and git services added a comment -

        Commit 1528217 from Darryl L. Pierce in branch 'proton/trunk'
        [ https://svn.apache.org/r1528217 ]

        PROTON-431: Fixed the assertions in the Ruby unit tests.

        Show
        ASF subversion and git services added a comment - Commit 1528217 from Darryl L. Pierce in branch 'proton/trunk' [ https://svn.apache.org/r1528217 ] PROTON-431 : Fixed the assertions in the Ruby unit tests.
        Hide
        ASF subversion and git services added a comment -

        Commit 1528218 from Darryl L. Pierce in branch 'proton/trunk'
        [ https://svn.apache.org/r1528218 ]

        PROTON-431: Fixed the Ruby spec tests.

        They weren't updated with the changes to the Messenger.accept and
        Messenger.settle API changes.

        Show
        ASF subversion and git services added a comment - Commit 1528218 from Darryl L. Pierce in branch 'proton/trunk' [ https://svn.apache.org/r1528218 ] PROTON-431 : Fixed the Ruby spec tests. They weren't updated with the changes to the Messenger.accept and Messenger.settle API changes.

          People

          • Assignee:
            Darryl L. Pierce
            Reporter:
            Darryl L. Pierce
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development