diff options
author | Douwe Maan <douwe@gitlab.com> | 2015-07-22 09:22:43 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2015-07-22 09:22:43 +0000 |
commit | 546731244a80fb44a7b1792615f2ccda767ad947 (patch) | |
tree | 05eb1aa1c6bd0876a1781cff93ae737d3d21dda4 /doc/gitlab-basics/command-line-commands.md | |
parent | 39dc39e33529ac3632e5d5a6fa06bf6908fb4fa4 (diff) | |
parent | b42422a7a97d376fb5f78e8dfa614857d6b8d1fa (diff) | |
download | gitlab-ce-546731244a80fb44a7b1792615f2ccda767ad947.tar.gz |
Merge branch 'rs-security-spec-speed' into 'master'
Speed up security feature specs
Before: `rspec spec/features/security/ 0.12s user 0.04s system 0% cpu 3:38.00 total`
After: `rspec spec/features/security/ 0.12s user 0.04s system 0% cpu 1:40.58 total`
The majority of the speed improvements is from two things:
1. Instead of using our standard `login_as` helper in the matchers, we take advantage
of the `Warden::Test::Helpers` version of the method which bypasses the login form
and logs the user in directly. We were essentially testing that filling out the login
form works hundreds of times.
2. There were many tests that verified if a user of a certain access level
(master, owner, reporter, guest) had access to a resource. Unfortunately we were
creating every type of user for each test even though a test was only verifying one of
them at a time. Now the tests only create the one user role they're testing.
See merge request !1023
Diffstat (limited to 'doc/gitlab-basics/command-line-commands.md')
0 files changed, 0 insertions, 0 deletions