Skip to content

Enable RSpec/MatchArray cop

Abdul Wadoodrequested to merge
abdwdd-rspec-match-array into master

Description of the proposal

Enable the RSpec/MatchArray cop.

Examples

# bad
it { is_expected.to match_array([content1, content2]) }

# good
it { is_expected.to contain_exactly(content1, content2) }

# good
it { is_expected.to match_array([content] + array) }

# good
it { is_expected.to match_array(%w(tremble in fear foolish mortals)) }

Impact on gitlab-org/gitlab 

bundle exec rubocop --only RSpec/MatchArray
6267 files inspected, 1165 offenses detected, 1165 offenses autocorrectable

Check-list

  • Mention this proposal in the relevant Slack channels (e.g. #development, #backend, #frontend)
  • If there is a choice to make between two potential styles, set up an emoji vote in the MR:
    • CHOICE_A: 🅰
    • CHOICE_B: 🅱
    • Vote yourself for both choices so that people know these are the choices
  • The MR doesn't have significant objections, and is getting a majority of 👍 vs 👎 (remember that we don't need to reach a consensus)
  • (If applicable) One style is getting a majority of vote (compared to the other choice)
  • (If applicable) Update the MR with the chosen style
  • Follow the review process as usual

Merge request reports