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
        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
        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
        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
        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

          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