RedJocker
commented
2 years ago - always be very mindful with assertions inside ifs
- ideally we should not do it
- sometimes we have no other way, but it really has to be the case
- what happens if implementation incorrectly has all widget.error values null?
- hole for incorrect solutions
- possible fix
- now that checkForErrorInputFieldsError is a fun you can pass arguments to that fun stating if field should be tested or not
- on line 110-111 assertion will never fail because if clause test same condition as assertion
- ideally we should not do it