Derby
  1. Derby
  2. DERBY-5114

Assert failure in SavepointJdbc30Test.testReleaseSqlSavepointAndRollback

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 10.8.1.2
    • Fix Version/s: 10.8.1.2
    • Component/s: Test
    • Labels:
      None
    • Bug behavior facts:
      Regression Test Failure

      Description

      Seen in nightly testing on Java 7:

      http://dbtg.foundry.sun.com/derby/test/Daily/jvm1.7/testing/testlog/vista-64/1078855-suitesAll_diff.txt

      23) testReleaseSqlSavepointAndRollback(org.apache.derbyTesting.functionTests.tests.jdbcapi.SavepointJdbc30Test)junit.framework.AssertionFailedError: expected:<6> but was:<0>
      at org.apache.derbyTesting.functionTests.tests.jdbcapi.SavepointJdbc30Test.testReleaseSqlSavepointAndRollback(SavepointJdbc30Test.java:775)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      at org.apache.derbyTesting.junit.BaseTestCase.runBare(BaseTestCase.java:112)

      1. derby-5114-1a.diff
        0.9 kB
        Knut Anders Hatlen

        Activity

        Hide
        Knut Anders Hatlen added a comment -

        The problem appears to be that some of the test cases insert rows into the test tables and don't clean up the tables when they're done. Depending on the order in which the test cases run, testReleaseSqlSavepointAndRollback may or may not see rows inserted by preceding test cases. The attached patch makes setUp() clear the tables before each test case.

        Show
        Knut Anders Hatlen added a comment - The problem appears to be that some of the test cases insert rows into the test tables and don't clean up the tables when they're done. Depending on the order in which the test cases run, testReleaseSqlSavepointAndRollback may or may not see rows inserted by preceding test cases. The attached patch makes setUp() clear the tables before each test case.
        Hide
        Knut Anders Hatlen added a comment -

        Committed revision 1079736.

        Show
        Knut Anders Hatlen added a comment - Committed revision 1079736.

          People

          • Assignee:
            Knut Anders Hatlen
            Reporter:
            Knut Anders Hatlen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development