Solr
  1. Solr
  2. SOLR-5863

Asserts wrong way around in TestSolrXml

    Details

    • Type: Test Test
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.7
    • Fix Version/s: 4.8, Trunk
    • Component/s: Tests
    • Labels:
      None

      Description

      the assertEquals tests in this class have switched the "expected" and "actual" arguments which may be confusing if the assert fails.

        Activity

        Hide
        Uwe Schindler added a comment -

        Close issue after release of 4.8.0

        Show
        Uwe Schindler added a comment - Close issue after release of 4.8.0
        Hide
        Jan Høydahl added a comment -

        For information: I did not find this small patch in test-land worthy of a CHANGES entry.

        Show
        Jan Høydahl added a comment - For information: I did not find this small patch in test-land worthy of a CHANGES entry.
        Hide
        ASF subversion and git services added a comment -

        Commit 1577542 from janhoy@apache.org in branch 'dev/branches/branch_4x'
        [ https://svn.apache.org/r1577542 ]

        SOLR-5863: Asserts wrong way around in TestSolrXml and a few other classes (merge from trunk)

        Show
        ASF subversion and git services added a comment - Commit 1577542 from janhoy@apache.org in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1577542 ] SOLR-5863 : Asserts wrong way around in TestSolrXml and a few other classes (merge from trunk)
        Hide
        ASF subversion and git services added a comment -

        Commit 1577539 from janhoy@apache.org in branch 'dev/trunk'
        [ https://svn.apache.org/r1577539 ]

        SOLR-5863: Asserts wrong way around in TestSolrXml and a few other classes

        Show
        ASF subversion and git services added a comment - Commit 1577539 from janhoy@apache.org in branch 'dev/trunk' [ https://svn.apache.org/r1577539 ] SOLR-5863 : Asserts wrong way around in TestSolrXml and a few other classes

          People

          • Assignee:
            Jan Høydahl
            Reporter:
            Jan Høydahl
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development