diff options
| author | Peter Leitzen <pleitzen@gitlab.com> | 2019-07-24 11:19:15 +0300 |
|---|---|---|
| committer | Grzegorz Bizon <grzegorz@gitlab.com> | 2019-07-24 11:19:15 +0300 |
| commit | 163a43629c9018beb5f2a474ce63a0065445471b (patch) | |
| tree | 3081baef63fcb1bdc30c35972793e49b750be859 /spec/services/issues/update_service_spec.rb | |
| parent | 2524cb1016cc54b82bcbf01e4df5fad46b980912 (diff) | |
Prefer `flat_map` over `map` + `flatten` in specs
Although `flat_map` is equivalent to `map` + `flatten(1)`
(note the level 1) we can apply this same refactoring to all cases.
Diffstat (limited to 'spec/services/issues/update_service_spec.rb')
| -rw-r--r-- | spec/services/issues/update_service_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/services/issues/update_service_spec.rb b/spec/services/issues/update_service_spec.rb index 468e7c286d5..68b79132096 100644 --- a/spec/services/issues/update_service_spec.rb +++ b/spec/services/issues/update_service_spec.rb @@ -179,7 +179,7 @@ describe Issues::UpdateService, :mailer do it 'sends email to user2 about assign of new issue and email to user3 about issue unassignment' do deliveries = ActionMailer::Base.deliveries email = deliveries.last - recipients = deliveries.last(2).map(&:to).flatten + recipients = deliveries.last(2).flat_map(&:to) expect(recipients).to include(user2.email, user3.email) expect(email.subject).to include(issue.title) end |