Derby
  1. Derby
  2. DERBY-5871

Invoke logical statements with existing statement tests

    Details

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

      Description

      Code coverage of logical statements can be improved by invoking them in existing statement tests. This could be done by having a decorator that makes use of pooled connections.

      1. derby-5871-PrepStmtTst.patch
        2 kB
        Mohamed Nufail
      2. derby-5871-CallblTst.patch
        0.8 kB
        Mohamed Nufail
      3. derby-5871-CallblStmtTst.patch
        0.8 kB
        Mohamed Nufail

        Issue Links

          Activity

          Hide
          Knut Anders Hatlen added a comment -

          [bulk update: close all resolved issues that haven't had any activity the last year]

          Show
          Knut Anders Hatlen added a comment - [bulk update: close all resolved issues that haven't had any activity the last year]
          Hide
          Bryan Pendleton added a comment -

          I believe that all the work on this issue is complete. Marking resolved by revision 1364068.

          Show
          Bryan Pendleton added a comment - I believe that all the work on this issue is complete. Marking resolved by revision 1364068.
          Hide
          Knut Anders Hatlen added a comment -

          Thanks, Nufail. I actually need CallableStatementTest to run on logical statements to test the fix I'm working on for DERBY-5872. Great timing!

          The patches look good to me, and I've verified that the tests still pass. These tests don't take long to run, so running them in one more configuration doesn't add much to the total time of suites.All, and the extra testing seems worthwhile.

          Committed revision 1364068.

          Show
          Knut Anders Hatlen added a comment - Thanks, Nufail. I actually need CallableStatementTest to run on logical statements to test the fix I'm working on for DERBY-5872 . Great timing! The patches look good to me, and I've verified that the tests still pass. These tests don't take long to run, so running them in one more configuration doesn't add much to the total time of suites.All, and the extra testing seems worthwhile. Committed revision 1364068.
          Hide
          Mohamed Nufail added a comment -

          Attached,
          derby-5871-PrepdStmtTst.patch
          derby-5871-CallblStmtTst.patch
          derby-5871-CallblTst.patch

          These patches add a connectionCPDecorator to the following tests,

          • org.apache.derbyTesting.functionTests.tests.derbynet.PrepareStatementTest
          • org.apache.derbyTesting.functionTests.tests.jdbc4.CallableStatementTest
          • org.apache.derbyTesting.functionTests.tests.jdbcapi.CallableTest
          Show
          Mohamed Nufail added a comment - Attached, derby-5871-PrepdStmtTst.patch derby-5871-CallblStmtTst.patch derby-5871-CallblTst.patch These patches add a connectionCPDecorator to the following tests, org.apache.derbyTesting.functionTests.tests.derbynet.PrepareStatementTest org.apache.derbyTesting.functionTests.tests.jdbc4.CallableStatementTest org.apache.derbyTesting.functionTests.tests.jdbcapi.CallableTest

            People

            • Assignee:
              Mohamed Nufail
              Reporter:
              Mohamed Nufail
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development