Uploaded image for project: 'Derby'
  1. Derby
  2. DERBY-5815

Add replicationTests.ReplicationSuite to junit-core ant target

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: 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
        kristwaa Kristian Waagan added a comment -

        Closing issue.

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

        Thanks, Knut Anders.

        Committed patch 1a to trunk with revision 1350166.

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

        The patch looks good to me. +1

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

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

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development