summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Added some more commentsFrancisco Lopez2017-11-171-1/+1
|
* Added UserAuthFinders specFrancisco Lopez2017-11-172-2/+196
|
* Added RequestAuthenticator specFrancisco Lopez2017-11-171-0/+67
|
* Homogenising the type of the request handled by UserAuthFinder. Also tests fixedFrancisco Lopez2017-11-171-16/+21
|
* Removing private tokenFrancisco Lopez2017-11-171-27/+0
|
* Updated refactor and pushing to see if test failsFrancisco Lopez2017-11-171-26/+1
|
* Pushing to see how this tests behavesFrancisco Lopez2017-11-171-7/+7
|
* First refactorFrancisco Lopez2017-11-171-0/+1
|
* Add link to discussionMichael Kozono2017-11-171-0/+4
|
* Remove extra attemptsMichael Kozono2017-11-171-7/+22
| | | | | | Get the tests deterministic or bust! Also, explain what the constants are doing and why. And don’t output the log message if it doesn’t apply.
* Dry up URLs and get args constructionMichael Kozono2017-11-171-36/+48
|
* Prevent affecting other testsMichael Kozono2017-11-171-4/+4
|
* Refactor application setting managementMichael Kozono2017-11-171-41/+35
|
* Fix Rubocop offensesMichael Kozono2017-11-171-13/+13
|
* Fix OAuth API and RSS rate limitingMichael Kozono2017-11-171-70/+178
|
* Allow throttling code in test environmentMichael Kozono2017-11-171-3/+0
| | | | This code should not break other tests because the rate limits are off by default.
* Add Rack::Attack throttle specsMichael Kozono2017-11-171-0/+286
|
* Resolve "lock/confidential issuable sidebar custom svg icons iteration"Luke "Jared" Bennett2017-11-161-3/+3
|
* Merge branch 'bvl-isolate-fork-network-background-migrations' into 'master'Rémy Coutable2017-11-161-11/+11
|\ | | | | | | | | Isolate the fork network background migrations See merge request gitlab-org/gitlab-ce!15393
| * Isolate the fork network background migrationsbvl-isolate-fork-network-background-migrationsBob Van Landuyt2017-11-151-11/+11
| | | | | | | | | | | | | | | | Before the `PopulateForkNetworksRange` spec would also call the `CreateForkNetworkMemberships` which we would count on in the spec. With this, I'm isolating that, and counting only records created in this particular migration instead.
* | Merge branch 'cache-user-keys-count' into 'master'Douwe Maan2017-11-164-1/+176
|\ \ | | | | | | | | | | | | Cache the number of user SSH keys See merge request gitlab-org/gitlab-ce!15401
| * | Cache the number of user SSH keyscache-user-keys-countYorick Peterse2017-11-164-1/+176
| | | | | | | | | | | | | | | | | | | | | | | | | | | By caching the number of personal SSH keys we reduce the number of queries necessary on pages such as ProjectsController#show (which can end up querying this data multiple times). The cache is refreshed/flushed whenever an SSH key is added, removed, or when a user is removed.
* | | Merge branch 'dm-notes-actions-noteable-for-update' into 'master'Sean McGivern2017-11-161-0/+23
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Make sure NotesActions#noteable returns a Noteable in the update action Closes #40208 See merge request gitlab-org/gitlab-ce!15421
| * | | Make sure NotesActions#noteable returns a Noteable in the update actiondm-notes-actions-noteable-for-updateDouwe Maan2017-11-161-0/+23
| |/ /
* | | Merge branch 'bvl-adjust-openid_connect_spec' into 'master'Sean McGivern2017-11-161-2/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Adjust openid_connect_spec to use `raise_error` Closes #40093 See merge request gitlab-org/gitlab-ce!15420
| * | | Adjust openid_connect_spec to use `raise_error`Bob Van Landuyt2017-11-161-2/+11
| | | | | | | | | | | | | | | | | | | | Using the `raise_error`-matcher instead of `throw_symbol` makes sure our after blocks get called in the test suite.
* | | | Merge branch 'backport-delete-epic' into 'master'Filipa Lacerda2017-11-163-14/+63
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Backport delete epic changes Closes #40162 See merge request gitlab-org/gitlab-ce!15414
| * | | | Backport delete epic changesbackport-delete-epicClement Ho2017-11-153-14/+63
| | | | |
* | | | | Merge branch 'fix/git-env-repo-paths' into 'master'Douwe Maan2017-11-161-10/+36
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | Use relative git object paths to construct absolute ones before setting Env See merge request gitlab-org/gitlab-ce!15376
| * | | | Use relative git object paths to construct absolute ones before setting Envfix/git-env-repo-pathsAhmad Sherif2017-11-141-10/+36
| | | | |
* | | | | Update container repository path referenceGrzegorz Bizon2017-11-161-0/+18
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | We should allow to use double underscore in the path, and it seems that our container repository path regexp was outdated. See https://github.com/docker/distribution/blob/master/reference/regexp.go
* | | | Merge branch ↵Mike Greiling2017-11-151-0/+17
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '39861-follow-up-from-enable-multiple-queries-per-prometheus-graph' into 'master' Resolve "Follow-up from "Enable multiple queries per Prometheus graph"" Closes #39861 See merge request gitlab-org/gitlab-ce!15363
| * | | | Add missing spec for computed prop strokeDashArray39861-follow-up-from-enable-multiple-queries-per-prometheus-graphJose Ivan Vargas2017-11-131-0/+17
| | | | |
* | | | | Merge branch 'update-emoji-digests-with-latest-from-gemojione' into 'master'Filipa Lacerda2017-11-152-7/+31
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update Emoji digests and assets with latest from Gemojione v3.3.0 Closes #32634 See merge request gitlab-org/gitlab-ce!15390
| * | | | | Add rainbow_flag to emoji support mapEric Eastwood2017-11-152-7/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tested on the following platforms that split the emoji into separate characters currently. - Windows 10 FCU - Chrome 62.0.3202.89 - Firefox 57.0b14 - MS Edge 41.16299.15.0 - macOS 10.12.6 - Chrome 63.0.3239.40 - Safari 11.0.1 (12604.3.5.1.1) - Firefox 57.0 - Ubuntu 16.04 - Chromium 62.0.3202.75 - Firefox 56.0
* | | | | | Merge branch 'tree_item_limit' into 'master'Douwe Maan2017-11-151-3/+29
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Truncate tree to max 1,000 items and display notice to users Closes #39746 See merge request gitlab-org/gitlab-ce!15333
| * | | | | | Truncate tree to max 1,000 items and display notice to usersDrew Blessing2017-11-151-3/+29
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rendering ten thousands of tree items consumes a lot of server time and can cause timeouts in extreme cases. Realistically, displaying more than 1,000 files is probably not useful so truncate and show the user a notice instead. 'Find files' can be used to locate specific files beyond the 1,000 limit.
* | | | | | Merge branch ↵Douwe Maan2017-11-153-13/+33
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '40122-only-one-note-webhook-is-triggered-when-a-comment-with-time-spent-is-added' into 'master' Resolve "Only one note webhook is triggered when a comment with time spent is added" Closes #40122 See merge request gitlab-org/gitlab-ce!15381
| * | | | | | Add total_time_spent to the `changes` hash in issuable Webhook payloads40122-only-one-note-webhook-is-triggered-when-a-comment-with-time-spent-is-addedRémy Coutable2017-11-153-13/+33
| |/ / / / / | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | | | Fix dumping hashed storage based repositoryGabriel Mazetto2017-11-151-0/+4
|/ / / / /
* | | | | Merge branch 'update-merge-worker-metrics' into 'master'Sean McGivern2017-11-151-0/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add simple logging to UpdateMergeRequestsWorker#perform See merge request gitlab-org/gitlab-ce!15360
| * | | | | rework the logging to be simpler and add a thresholdmicael.bergeron2017-11-141-3/+9
| | | | | |
| * | | | | add simple logging to UpdateMergeRequestsWorker#performmicael.bergeron2017-11-131-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | this is to try to debug #35914
* | | | | | Merge branch 'ce-backport-39009-fix-approvals-showing-up-in-empty-mr-widget' ↵Phil Hughes2017-11-151-9/+10
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Remove approvals when in empty MR widget state -- CE backport See merge request gitlab-org/gitlab-ce!15362
| * | | | | Backport spec changes from EE remove approvals in empty statece-backport-39009-fix-approvals-showing-up-in-empty-mr-widgetEric Eastwood2017-11-131-9/+10
| |/ / / / | | | | | | | | | | | | | | | See https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3376
* | | | | Merge branch 'gitaly-fix-broken-specs' into 'master'Robert Speicher2017-11-141-2/+6
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | skip_gitaly_mock disables Gitaly in specs See merge request gitlab-org/gitlab-ce!15383
| * | | | we need to disable gitaly for some testsKim "BKC" Carlbäcker2017-11-141-2/+6
| | | | |
* | | | | Merge branch 'bvl-subgroup-in-dropdowns' into 'master'Sean McGivern2017-11-145-8/+141
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include child projects a user can manage in namespace dropdowns Closes #39987 See merge request gitlab-org/gitlab-ce!15294
| * | | | | Include child projects a user can manage in namespace dropdownsBob Van Landuyt2017-11-145-8/+141
| | | | | | | | | | | | | | | | | | | | | | | | These dropdown options are used for creating and transfering projects.
* | | | | | Stops page reload when changing tabs or pages - uses API requests insteadFilipa Lacerda2017-11-146-132/+199
| |_|_|_|/ |/| | | |