Fix todo-ed rubocop violations

What does this MR do?

This MR contains a number of TODO-ed rubocop fixes encountered during work on !40088 (merged) due to the use of REVEAL_RUBOCOP_TODO=1.

None of these changes should have any functional impact.

Does this MR meet the acceptance criteria?

Conformity

• 📋 Does this MR need a changelog?
  • I have included a changelog entry.
  • I have not included a changelog entry because this is a set of developer facing changes.
• Documentation (if required)
• Code review guidelines
• Merge request performance guidelines
• Style guides
• Database guides
• Separation of EE specific content

Availability and Testing

This should have no impact at all on any behavior.