Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-6306

Executable test in utils needs to test an executable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: utils
    • Labels:
      None
    • Story Points:
      1
    • Team:
      Architecture

      Description

      In DM-4036 all the test binaries were removed as no longer being needed. This had the unfortunate side effect that the testExecutables.py test no longer tests anything. This ticket will be used for adding a test file.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Russell Owen would you mind taking a look at this? Some minor clean up, a sanity check to ensure that automated executable test discovery found something and a dummy shell script to test.

            Show
            tjenness Tim Jenness added a comment - Russell Owen would you mind taking a look at this? Some minor clean up, a sanity check to ensure that automated executable test discovery found something and a dummy shell script to test.
            Hide
            rowen Russell Owen added a comment - - edited

            This looks fine to me. Thanks for the extensive comments.

            Show
            rowen Russell Owen added a comment - - edited This looks fine to me. Thanks for the extensive comments.
            Hide
            tjenness Tim Jenness added a comment -

            Thanks. Merged.

            Show
            tjenness Tim Jenness added a comment - Thanks. Merged.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Russell Owen
                Watchers:
                Russell Owen, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: