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

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

    Details

      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

            krzys Krzysztof Findeisen created issue -
            krzys Krzysztof Findeisen made changes -
            Field Original Value New Value
            Link This issue relates to DM-22599 [ DM-22599 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S20-3 (February) [ 984 ]
            krzys Krzysztof Findeisen made changes -
            Assignee Krzysztof Findeisen [ krzys ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S20-3 (February) [ 984 ] AP S20-4 (March) [ 985 ]
            krzys Krzysztof Findeisen made changes -
            Rank Ranked lower
            krzys Krzysztof Findeisen made changes -
            Epic Link DM-22633 [ 427742 ]
            swinbank John Swinbank made changes -
            Sprint AP S20-4 (March) [ 985 ] AP S20-5 (April) [ 986 ]
            swinbank John Swinbank made changes -
            Rank Ranked lower
            swinbank John Swinbank made changes -
            Epic Link DM-22633 [ 427742 ] DM-24341 [ 433028 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S20-5 (April) [ 986 ] AP S20-6 (May) [ 987 ]
            krzys Krzysztof Findeisen made changes -
            Rank Ranked higher
            krzys Krzysztof Findeisen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S20-6 (May) [ 987 ]
            krzys Krzysztof Findeisen made changes -
            Rank Ranked lower
            krzys Krzysztof Findeisen made changes -
            Status In Progress [ 3 ] To Do [ 10001 ]
            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.
            swinbank John Swinbank made changes -
            Epic Link DM-24341 [ 433028 ] DM-25145 [ 435263 ]
            swinbank John Swinbank made changes -
            Epic Link DM-25145 [ 435263 ] DM-26803 [ 439755 ]

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                krzys Krzysztof Findeisen
                Watchers:
                Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Summary Panel