provide id for option props

flag=none

test plan:
  - specs pass

qa risk: low

Change-Id: Id7548234277366abe9e528fd6982ca36fafbedee
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/274196
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
Reviewed-by: Omar Soto-Fortuño <omar.soto@instructure.com>
QA-Review: Omar Soto-Fortuño <omar.soto@instructure.com>
Product-Review: Omar Soto-Fortuño <omar.soto@instructure.com>
This commit is contained in:
Davis Hyer 2021-09-22 13:26:07 -06:00
parent 725711463b
commit 0069377ce7
2 changed files with 3 additions and 4 deletions

View File

@ -80,7 +80,6 @@ const ignoredErrors = [
/Invariant Violation/,
/It looks like you're using the wrong act/,
/modalProps.onDismiss is not a function/,
/Must provide id for each option via `getOptionProps`/,
/Prop `children` should be supplied unless/,
/props.setRCEOpen is not a function/,
/React does not recognize the `%s` prop on a DOM element/,

View File

@ -128,17 +128,17 @@ describe('CreateCourseModal', () => {
const accountsPage1 = []
for (let i = 0; i < 50; i++) {
accountsPage1.push({
id: i,
id: String(i),
name: String(i)
})
}
const accountsPage2 = [
{
id: 51,
id: '51',
name: '51'
},
{
id: 52,
id: '52',
name: '52'
}
]