Qpid
  1. Qpid
  2. QPID-3513

Avoid use of shell script clean-dir during test cycle

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.13
    • Component/s: Java Tests
    • Labels:
      None

      Description

      Current the test harness provided by QpidBrokerTestCase utilises a UNIX shell script clean-dir to clean the broker's work
      directory. This approach is less than ideal: spawning a separate process is relativity slow and this delays the test cycle, it presents
      a barrier to running the test suite on Windows, and the approach is very sensitive to spaces within directory names (something that is preventing our use of Jenkins' matrix feature).

      This change will remove the use of the shellscript and replace with the existing FileUtils.delete utility method.

        Issue Links

          Activity

          Keith Wall created issue -
          Keith Wall made changes -
          Field Original Value New Value
          Description Current the test harness provided by QpidBrokerTestCase utilises a UNIX shell script clean-dir to clean the broker's work
          directory. This approach is less than ideal: spawning a separate process is relativity slow and this delays the test cycle, it presents
          a barrier to running the test suite on Windows, and the approach is very sensitive to spaces within directory names (something that is preventing out use of Jenkin's matrix feature).

          This change will remove the use of the shellsrcipt and replace with the existing FileUtils.delete utility method.

          Current the test harness provided by QpidBrokerTestCase utilises a UNIX shell script clean-dir to clean the broker's work
          directory. This approach is less than ideal: spawning a separate process is relativity slow and this delays the test cycle, it presents
          a barrier to running the test suite on Windows, and the approach is very sensitive to spaces within directory names (something that is preventing our use of Jenkins' matrix feature).

          This change will remove the use of the shellsrcipt and replace with the existing FileUtils.delete utility method.

          Keith Wall made changes -
          Description Current the test harness provided by QpidBrokerTestCase utilises a UNIX shell script clean-dir to clean the broker's work
          directory. This approach is less than ideal: spawning a separate process is relativity slow and this delays the test cycle, it presents
          a barrier to running the test suite on Windows, and the approach is very sensitive to spaces within directory names (something that is preventing our use of Jenkins' matrix feature).

          This change will remove the use of the shellsrcipt and replace with the existing FileUtils.delete utility method.

          Current the test harness provided by QpidBrokerTestCase utilises a UNIX shell script clean-dir to clean the broker's work
          directory. This approach is less than ideal: spawning a separate process is relativity slow and this delays the test cycle, it presents
          a barrier to running the test suite on Windows, and the approach is very sensitive to spaces within directory names (something that is preventing our use of Jenkins' matrix feature).

          This change will remove the use of the shellscript and replace with the existing FileUtils.delete utility method.

          Keith Wall made changes -
          Keith Wall made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Keith Wall made changes -
          Status In Progress [ 3 ] Ready To Review [ 10006 ]
          Keith Wall made changes -
          Assignee Keith Wall [ k-wall ] Robbie Gemmell [ gemmellr ]
          Robbie Gemmell made changes -
          Link This issue is related to QPID-3402 [ QPID-3402 ]
          Robbie Gemmell made changes -
          Status Ready To Review [ 10006 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              Robbie Gemmell
              Reporter:
              Keith Wall
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development