Skip to content

Add missing coverage for disabled Style/SafeNavigation lines #1851

Open
@kaylareopelle

Description

Description

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.

Dependencies

N/A

Estimates

M

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions