Derby
  1. Derby
  2. DERBY-2100

Convert derbynet/prepStmt.java to Junit

    Details

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

      Description

      Convert derbynet/prepStmt.java to Junit.
      Should also make it run both in embedded and client/server frameworks.

      1. derby2100-stat.txt
        0.8 kB
        Øystein Grøvlen
      2. derby2100-diff.txt
        110 kB
        Øystein Grøvlen

        Issue Links

          Activity

          Hide
          Øystein Grøvlen added a comment -

          derby2100-diff.txt contains the patch for converting the prepStmt test to Junit. The test is now called PrepareStatementTest.java. While prepStmt only ran in the client/server suites, the new test now runs in both embedded and client/server. However, I have chosen to place the new test in the derbynet directory since its test cases are mostly testing border cases that are mainly relevant to client/server.

          In addition to the new test, the patch contains the changes needed to remove the old test and its associated master files. The patch also contains a change to AllPackages.java in order to run the derbynet suite only once. (I found that an entry for this suite had been added twice.)

          I have run the Junit All suite with no errors and verified that the new test was run. I have also run derbyall with no errors and verified that the old test was not run.

          Show
          Øystein Grøvlen added a comment - derby2100-diff.txt contains the patch for converting the prepStmt test to Junit. The test is now called PrepareStatementTest.java. While prepStmt only ran in the client/server suites, the new test now runs in both embedded and client/server. However, I have chosen to place the new test in the derbynet directory since its test cases are mostly testing border cases that are mainly relevant to client/server. In addition to the new test, the patch contains the changes needed to remove the old test and its associated master files. The patch also contains a change to AllPackages.java in order to run the derbynet suite only once. (I found that an entry for this suite had been added twice.) I have run the Junit All suite with no errors and verified that the new test was run. I have also run derbyall with no errors and verified that the old test was not run.
          Hide
          Knut Anders Hatlen added a comment -

          Committed revision 491768.

          Show
          Knut Anders Hatlen added a comment - Committed revision 491768.
          Hide
          Øystein Grøvlen added a comment -

          I have verified that the new test now runs successfully in the Junit test suite.

          Show
          Øystein Grøvlen added a comment - I have verified that the new test now runs successfully in the Junit test suite.

            People

            • Assignee:
              Øystein Grøvlen
              Reporter:
              Øystein Grøvlen
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development