Skip to content

Commit

Permalink
[CST] [Frontend] Added required prop to 5103 document upload form (#3…
Browse files Browse the repository at this point in the history
…0887)

* Added 'required' true attribute to checkbox, and updated unit test
  • Loading branch information
samcoforma authored Jul 17, 2024
1 parent 944814f commit e8472e7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ function Automated5103Notice({
className="vads-u-margin-y--3"
checked={addedEvidence}
error={checkboxErrorMessage}
required
onVaChange={event => {
setAddedEvidence(event.detail.checked);
}}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,9 @@ describe('<Automated5103Notice>', () => {
expect($('va-checkbox', container).getAttribute('checked')).to.equal(
'false',
);
expect($('va-checkbox', container).getAttribute('required')).to.equal(
'true',
);
expect(submitRequest.called).to.be.false;
expect(submit5103.called).to.be.false;
expect(navigate.called).to.be.false;
Expand Down

0 comments on commit e8472e7

Please sign in to comment.