Derby
  1. Derby
  2. DERBY-5318

Use assertDirectoryDeleted in ReplicationRun and remove dead code

    Details

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

      Description

      The error reporting in the replication test when deleting files to initialize the master/slave is insufficient. A message is printed to the console, but the test continues to run. This can cause the test to fail for what appears to be a different reason the left-over files.

        Issue Links

          Activity

          Kristian Waagan created issue -
          Kristian Waagan made changes -
          Field Original Value New Value
          Assignee Kristian Waagan [ kristwaa ]
          Hide
          Kristian Waagan added a comment -

          Attaching patch 1a, which makes use of BaseJDBCTestCase.assertDirectoryDeleted to make the test fail if some of the files in the database directory can't be deleted. Note that the root directory will now be deleted and recreated, whereas before the patch the files inside the root directory were deleted.
          Also removed dead code and pruned out some unused method arguments.

          The replication test ran fine on Solaris 11, will test on Windows too.
          Patch ready for review.

          Show
          Kristian Waagan added a comment - Attaching patch 1a, which makes use of BaseJDBCTestCase.assertDirectoryDeleted to make the test fail if some of the files in the database directory can't be deleted. Note that the root directory will now be deleted and recreated, whereas before the patch the files inside the root directory were deleted. Also removed dead code and pruned out some unused method arguments. The replication test ran fine on Solaris 11, will test on Windows too. Patch ready for review.
          Kristian Waagan made changes -
          Attachment derby-5318-1a-cleanup.diff [ 12485562 ]
          Hide
          Kristian Waagan added a comment -

          Verified on Windows Vista.
          All subsequent tests will fail with the same error if one or more files cannot be deleted.
          If the file(s) can be deleted again for some reason, the test will "recover". If this happens, something kept a file open for some time, but then closed the file handle.

          Show
          Kristian Waagan added a comment - Verified on Windows Vista. All subsequent tests will fail with the same error if one or more files cannot be deleted. If the file(s) can be deleted again for some reason, the test will "recover". If this happens, something kept a file open for some time, but then closed the file handle.
          Kristian Waagan made changes -
          Link This issue is related to DERBY-4635 [ DERBY-4635 ]
          Hide
          Kristian Waagan added a comment -

          Committed to trunk with revision 1146644.

          Show
          Kristian Waagan added a comment - Committed to trunk with revision 1146644.
          Kristian Waagan made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 10.9.0.0 [ 12316344 ]
          Resolution Fixed [ 1 ]
          Hide
          Kristian Waagan added a comment -

          Backported to 10.8 with revision 1156997.

          Show
          Kristian Waagan added a comment - Backported to 10.8 with revision 1156997.
          Kristian Waagan made changes -
          Fix Version/s 10.8.1.6 [ 12316676 ]
          Hide
          Kristian Waagan added a comment -

          Closing issue.

          Show
          Kristian Waagan added a comment - Closing issue.
          Kristian Waagan made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Fix Version/s 10.8.1.6 [ 12316676 ]
          Myrna van Lunteren made changes -
          Fix Version/s 10.8.2.2 [ 12317968 ]
          Fix Version/s 10.8.2.0 [ 12317955 ]
          Gavin made changes -
          Workflow jira [ 12619425 ] Default workflow, editable Closed status [ 12801159 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          7d 2h 1m 1 Kristian Waagan 14/Jul/11 11:53
          Resolved Resolved Closed Closed
          28d 21h 18m 1 Kristian Waagan 12/Aug/11 09:12

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development