Had a realization that list options that were in a group would not go through the registerOption routine (a fix for SLE Angular apps). While I'm not sure if there is any current use-case that would hit the problem, I went ahead and made the fix and the tests that demonstrate the problem.
👩💻 Implementation
Just made the private getListOptionOwner routine on ListOption a bit more robust.
🧪 Testing
Wrote component and Angular tests modeled after existing tests but with a group as the option owner.
✅ Checklist
[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.
Pull Request
🤨 Rationale
Had a realization that list options that were in a group would not go through the
registerOptionroutine (a fix for SLE Angular apps). While I'm not sure if there is any current use-case that would hit the problem, I went ahead and made the fix and the tests that demonstrate the problem.👩💻 Implementation
Just made the private
getListOptionOwnerroutine onListOptiona bit more robust.🧪 Testing
Wrote component and Angular tests modeled after existing tests but with a group as the option owner.
✅ Checklist