Recipe ABC to enable mandatory input workspaces #468
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of work
This updates the
Recipe
ABC to have a set of mandatory input workspaces.Explanation of work
The default
validateInputs
method on theRecipe
ABC was setup to check for the existence of every workspace in thegroceries
dictionary, even if it is an output.This specifies the set of mandatory inputs that must be in the
groceries
. The others can be Outputs or optional InOuts, or just optional workspaces.This allows implementing recipes to use the default without needing to override.
To test
Dev testing
Make sure all unit tests pass.
Make sure manual testing can get to save.
CIS testing
Enabler, no need
Link to EWM item
This came up as part of work on
EWM#7388
Verification
N/A
Acceptance Criteria
N/A