Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.1, 4.0-ALPHA
    • Component/s: modules/other
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      The contrib/remote tests will fail randomly.

      This is because they use this _TestUtil.getRandomSocketPort() which
      simply generates a random number, but if this is already in use, it will fail.

      Additionally there is duplicate RMI logic across all 3 test classes.

        Activity

        Hide
        Robert Muir added a comment -

        Patch that creates a RemoteTestCaseJ4 that binds to an ephemeral port (0), with a custom
        RMIServerSocketFactory that has access to the port number.

        _TestUtil.getRandomSocketPort() is removed, and all remote tests extend this new class.

        Show
        Robert Muir added a comment - Patch that creates a RemoteTestCaseJ4 that binds to an ephemeral port (0), with a custom RMIServerSocketFactory that has access to the port number. _TestUtil.getRandomSocketPort() is removed, and all remote tests extend this new class.
        Hide
        Robert Muir added a comment -

        Committed revision 932795.

        Show
        Robert Muir added a comment - Committed revision 932795.
        Hide
        Uwe Schindler added a comment -

        Looks fine! +1 to commit!

        Show
        Uwe Schindler added a comment - Looks fine! +1 to commit!
        Hide
        Robert Muir added a comment -

        backported: revision 941659

        Show
        Robert Muir added a comment - backported: revision 941659
        Hide
        Grant Ingersoll added a comment -

        Bulk close for 3.1

        Show
        Grant Ingersoll added a comment - Bulk close for 3.1

          People

          • Assignee:
            Robert Muir
            Reporter:
            Robert Muir
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development