Merge pull request #8643 from AlexSCorey/8642-InventoryonWFJTForm

Inventory Lookup on WFJT Form should not be required

Reviewed-by: https://github.com/apps/softwarefactory-project-zuul
This commit is contained in:
softwarefactory-project-zuul[bot] 2020-11-23 18:57:22 +00:00 committed by GitHub
commit 4882ca0481
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 1 deletions

View File

@ -126,7 +126,6 @@ function WorkflowJobTemplateForm({
onChange={value => {
inventoryHelpers.setValue(value);
}}
required={!askInventoryOnLaunchField.value}
touched={inventoryMeta.touched}
error={inventoryMeta.error}
/>

View File

@ -264,4 +264,8 @@ describe('<WorkflowJobTemplateForm/>', () => {
expect(handleCancel).toBeCalled();
});
test('should not show inventory field as required', () => {
expect(wrapper.find('InventoryLookup').prop('required')).toBe(false);
});
});