Derby
  1. Derby
  2. DERBY-5815

Add replicationTests.ReplicationSuite to junit-core ant target

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.10.1.1
    • Fix Version/s: 10.10.1.1
    • Component/s: Test
    • Labels:
      None

      Description

      The replication tests aren't run as part of the junit-core ant target (nor in any of the other relevant targets).
      I suggest adding the suite such that is is run as part of the top-level ant targets for testing Derby.

      Note that it would have been simpler to simply run suites.All, but the special logic for JDBC 4 code may still be relevant for some platforms (JSR169). It is also nice to get some more feedback as the test cycle proceeds. The downside of not using suites.All is that we may forget adding new suites to the ant target(s), but we're adding so few new suites that it shouldn't be much of a problem.

        Activity

        Hide
        Kristian Waagan added a comment -

        Closing issue.

        Show
        Kristian Waagan added a comment - Closing issue.
        Hide
        Kristian Waagan added a comment -

        Thanks, Knut Anders.

        Committed patch 1a to trunk with revision 1350166.

        Show
        Kristian Waagan added a comment - Thanks, Knut Anders. Committed patch 1a to trunk with revision 1350166.
        Hide
        Knut Anders Hatlen added a comment -

        The patch looks good to me. +1

        Show
        Knut Anders Hatlen added a comment - The patch looks good to me. +1
        Hide
        Kristian Waagan added a comment -

        Attaching patch 1a, adding the replication suite to the junit-core ant target.

        Show
        Kristian Waagan added a comment - Attaching patch 1a, adding the replication suite to the junit-core ant target.

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development