diff options
| author | Peter Leitzen <pleitzen@gitlab.com> | 2019-07-24 08:19:15 +0000 |
|---|---|---|
| committer | Grzegorz Bizon <grzegorz@gitlab.com> | 2019-07-24 08:19:15 +0000 |
| commit | 163a43629c9018beb5f2a474ce63a0065445471b (patch) | |
| tree | 3081baef63fcb1bdc30c35972793e49b750be859 /spec/controllers/projects | |
| parent | 2524cb1016cc54b82bcbf01e4df5fad46b980912 (diff) | |
| download | gitlab-ce-163a43629c9018beb5f2a474ce63a0065445471b.tar.gz | |
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/controllers/projects')
| -rw-r--r-- | spec/controllers/projects/jobs_controller_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/controllers/projects/jobs_controller_spec.rb b/spec/controllers/projects/jobs_controller_spec.rb index 901402aa5fd..e428aa3c7b7 100644 --- a/spec/controllers/projects/jobs_controller_spec.rb +++ b/spec/controllers/projects/jobs_controller_spec.rb @@ -158,7 +158,7 @@ describe Projects::JobsController, :clean_gitlab_redis_shared_state do get_show_json json_response.dig('pipeline', 'details', 'stages').tap do |stages| - expect(stages.map(&:keys).flatten) + expect(stages.flat_map(&:keys)) .to eq %w[name title status path dropdown_path] end end |