Skip to content

Improve merge_request_finder_spec

Alina Mihaila requested to merge am-improve-merge-request-finder-spec into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Related #373974 (closed) Before

Finished in 5 minutes 16 seconds (files took 8.75 seconds to load)
167 examples, 0 failures

[TEST PROF INFO] Time spent in factories: 03:10.059 (59.93% of total time)
[TEST PROF INFO] Factories usage

 Total: 2875
 Total top-level: 2129
 Total time: 03:10.059 (out of 05:21.653)
 Total uniq factories: 17

   total   top-level     total time      time per call      top-level time               name

     727         727       57.6585s            0.0793s            57.6585s      merge_request
     405         390      113.1528s            0.2794s           104.0986s            project
     322         322        4.3613s            0.0135s             4.3613s              label
     310           0        0.8190s            0.0026s             0.0000s namespace_ci_cd_settings
     310         310       13.2193s            0.0426s            13.2193s              group
     310           0        0.9337s            0.0030s             0.0000s namespace_settings
     308         308        0.9113s            0.0030s             0.9113s         label_link
      97           0        6.2844s            0.0648s             0.0000s          namespace
      27          22        1.5839s            0.0587s             1.3582s               user
      17          14        7.2473s            0.4263s             0.0960s        environment
      17          17        7.9106s            0.4653s             7.9106s         deployment
      10          10        0.0466s            0.0047s             0.0466s           approval
       6           6        0.3524s            0.0587s             0.3524s labeled_merge_request
       3           0        0.0711s            0.0237s             0.0000s        ci_pipeline
       3           0        0.1363s            0.0454s             0.0000s           ci_build
       2           2        0.0365s            0.0182s             0.0365s          milestone
       1           1        0.0099s            0.0099s             0.0099s            license

After

Finished in 2 minutes 53.6 seconds (files took 12.45 seconds to load)
167 examples, 0 failures

[TEST PROF INFO] Time spent in factories: 01:55.080 (65.98% of total time)
[TEST PROF INFO] Factories usage

 Total: 2375
 Total top-level: 1706
 Total time: 01:55.080 (out of 02:59.902)
 Total uniq factories: 17

   total   top-level     total time      time per call      top-level time               name

     715         715       51.1404s            0.0715s            51.1404s      merge_request
     334         325       47.4380s            0.1420s            39.4817s            project
     310           0        0.9776s            0.0032s             0.0000s namespace_settings
     310           0        0.8039s            0.0026s             0.0000s namespace_ci_cd_settings
     310         310       13.8735s            0.0448s            13.8735s              group
     308         308        0.8863s            0.0029s             0.8863s         label_link
      26           0        1.4340s            0.0552s             0.0000s          namespace
      20          15        1.2255s            0.0613s             0.9905s               user
      10          10        0.0485s            0.0049s             0.0485s           approval
      10          10        0.8642s            0.0864s             0.8642s              label
       5           5        7.5725s            1.5145s             7.5725s         deployment
       5           2        7.2157s            1.4431s             0.0114s        environment
       3           3        0.1601s            0.0534s             0.1601s labeled_merge_request
       3           0        0.0663s            0.0221s             0.0000s        ci_pipeline
       3           0        0.1303s            0.0434s             0.0000s           ci_build
       2           2        0.0441s            0.0221s             0.0441s          milestone
       1           1        0.0069s            0.0069s             0.0069s            license
➜  gitlab git:(am-improve-merge-request-finder-spec) ✗

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Kozono

Merge request reports