There are approximately 30 # rubocop:disable Style/SafeNavigation comments within the codebase created during the resolution of #1787
Above these lines, there's another comment describing the type of coverage the following line lacks.
Acceptance Criteria
[ ] Add the missing type of test coverage (else branch, then branch, or line and branch coverage) for each LOC disabling the Style/SafeNavigation rubocop
[ ] Remove the # rubocop:disable Style/SafeNavigation comments and the comments about test coverage
[ ] Correct the lines of code to use the safe navigation operator (rubocop -A)
Design Consideration/Limitations
We could also consider making these changes as we edit the files for other purposes.
workato-integration[bot]
commented
1 year ago
Description
There are approximately 30
# rubocop:disable Style/SafeNavigationcomments within the codebase created during the resolution of #1787Above these lines, there's another comment describing the type of coverage the following line lacks.
Acceptance Criteria
elsebranch,thenbranch, or line and branch coverage) for each LOC disabling the Style/SafeNavigation rubocop# rubocop:disable Style/SafeNavigationcomments and the comments about test coveragerubocop -A)Design Consideration/Limitations
We could also consider making these changes as we edit the files for other purposes.
Dependencies
N/A
Estimates
M