Directory ApacheDS
  1. Directory ApacheDS
  2. DIRSERVER-1022

Determine if ReferralTest and ReferralITest in server-unit are the same test

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.4
    • Component/s: None
    • Labels:
      None

      Description

      I think when we renamed ReferralTest to ReferralITest so it is an integration test (ITest) we may not have deleted the old test and this will make the integration runs take an extra 20-30 seconds unnecessarily as well as clutter the project. Let's determine if it is a duplicate and make sure we get rid of it if so.

        Activity

        Alex Karasulu created issue -
        Emmanuel Lecharny made changes -
        Field Original Value New Value
        Assignee Emmanuel Lecharny [ elecharny ]
        Hide
        Emmanuel Lecharny added a comment -

        ReferralITest was created on purpose, to check some pb we have with referrals (mainly related to the STANDARD compliance) test. The test is based on the page http://cwiki.apache.org/confluence/display/DIRxDEV/Referral+Unit+Tests describing the different scenarii (this page is not complete yet ...)

        Show
        Emmanuel Lecharny added a comment - ReferralITest was created on purpose, to check some pb we have with referrals (mainly related to the STANDARD compliance) test. The test is based on the page http://cwiki.apache.org/confluence/display/DIRxDEV/Referral+Unit+Tests describing the different scenarii (this page is not complete yet ...)
        Emmanuel Lecharny made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Hide
        Emmanuel Lecharny added a comment -

        The initial issue was valid. We should merge both tests.

        Show
        Emmanuel Lecharny added a comment - The initial issue was valid. We should merge both tests.
        Emmanuel Lecharny made changes -
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Invalid [ 6 ]
        Hide
        Emmanuel Lecharny added a comment -

        I think we must use the same set of data for all the tests, being it maxwell/fermi/hertz or france/usa. I suggest that tests are migrated from one file to the other, and that the wiki page is completed with the tested scenarii.

        Show
        Emmanuel Lecharny added a comment - I think we must use the same set of data for all the tests, being it maxwell/fermi/hertz or france/usa. I suggest that tests are migrated from one file to the other, and that the wiki page is completed with the tested scenarii.
        Hide
        Emmanuel Lecharny added a comment -

        Cowardly postponing this issue to 1.5.2

        This is definitively not critical, but must be fixed.

        the idea is to capture the scenarii from one of the two tests and to reproduce them into the other test, using the correct data.

        Show
        Emmanuel Lecharny added a comment - Cowardly postponing this issue to 1.5.2 This is definitively not critical, but must be fixed. the idea is to capture the scenarii from one of the two tests and to reproduce them into the other test, using the correct data.
        Emmanuel Lecharny made changes -
        Fix Version/s 1.5.2 [ 12310793 ]
        Fix Version/s 1.5.1 [ 12310792 ]
        Hide
        Emmanuel Lecharny added a comment -

        Postponed.

        Show
        Emmanuel Lecharny added a comment - Postponed.
        Emmanuel Lecharny made changes -
        Fix Version/s 1.5.3 [ 12312693 ]
        Fix Version/s 1.5.2 [ 12310793 ]
        Alex Karasulu made changes -
        Fix Version/s 1.5.3 [ 12312693 ]
        Fix Version/s 1.5.4 [ 12313147 ]
        Alex Karasulu made changes -
        Assignee Emmanuel Lecharny [ elecharny ] Alex Karasulu [ akarasulu ]
        Hide
        Alex Karasulu added a comment -

        Same test. Integrated and converted tests to use server-integ framework.

        Show
        Alex Karasulu added a comment - Same test. Integrated and converted tests to use server-integ framework.
        Alex Karasulu made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Alex Karasulu
            Reporter:
            Alex Karasulu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development