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

Add ways to test a PipelineTask's init inputs/outputs

    XMLWordPrintable

    Details

    • Story Points:
      6
    • Sprint:
      AP S21-3 (February), AP S21-4 (March)
    • Team:
      Alert Production

      Description

      DM-22599 created a test framework for validating many parts of a PipelineTask, including its normal inputs and outputs. However, it does not support InitInputs or InitOutputs. Add analogous test methods for these connections.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Deferring this until we get some more user experience with the existing code. `InitOutputs` could probably be handled much like `Outputs` (see the current `assertValidOutput`), but I'm not sure about the best way to handle `InitInputs`. The main thing that can go wrong there is a bug in any optional inputs, or a mismatch between what the config thinks is needed and what the task code assumes.

            Show
            krzys Krzysztof Findeisen added a comment - Deferring this until we get some more user experience with the existing code. `InitOutputs` could probably be handled much like `Outputs` (see the current `assertValidOutput`), but I'm not sure about the best way to handle `InitInputs`. The main thing that can go wrong there is a bug in any optional inputs, or a mismatch between what the config thinks is needed and what the task code assumes.
            Hide
            Parejkoj John Parejko added a comment -

            One small comment, otherwise this looks useful and I'm now thinking about how to apply it to the gen3 jointcal code.

            Show
            Parejkoj John Parejko added a comment - One small comment, otherwise this looks useful and I'm now thinking about how to apply it to the gen3 jointcal code.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: