summaryrefslogtreecommitdiff
path: root/spec/services/merge_requests/reopen_service_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2023-03-101-1/+5
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2023-03-091-1/+1
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2021-05-121-6/+6
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2021-01-081-0/+1
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2021-01-071-0/+8
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2020-11-161-0/+5
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2020-11-121-0/+5
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2020-09-181-17/+3
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2020-06-241-1/+1
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2020-05-221-3/+19
|
* Add latest changes from gitlab-org/gitlab@masterGitLab Bot2019-10-231-1/+1
|
* Add frozen_string_literal to spec/servicesfrozen_string_literal_spec_servicesThong Kuah2019-04-121-0/+2
| | | | Probably useful as we often move these files to "new" files.
* [CE] Support multiple assignees for merge requestsosw-multi-assignees-merge-requestsOswaldo Ferreira2019-04-081-1/+1
| | | | | Backports https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/10161 (code out of ee/ folder).
* Retrieve merge request closing issues from database cacheFelipe Artur2018-08-061-0/+6
|
* Resolve "Rename the `Master` role to `Maintainer`" BackendMark Chao2018-07-111-1/+1
|
* Make user/author use project.creator in most factoriesRémy Coutable2018-01-311-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Cache merged and closed events data in merge_request_metrics tableOswaldo Ferreira2018-01-021-0/+13
|
* Replace '.team << [user, role]' with 'add_role(user)' in specs36782-replace-team-user-role-with-add_role-user-in-specsblackst0ne2017-12-221-3/+3
|
* Cache the number of open issues and merge requestsYorick Peterse2017-08-231-0/+7
| | | | | | | | | | | | | | | | | | | | | Every project page displays a navigation menu that in turn displays the number of open issues and merge requests. This means that for every project page we run two COUNT(*) queries, each taking up roughly 30 milliseconds on GitLab.com. By caching these numbers and refreshing them whenever necessary we can reduce loading times of all these pages by up to roughly 60 milliseconds. The number of open issues does not include confidential issues. This is a trade-off to keep the code simple and to ensure refreshing the data only needs 2 COUNT(*) queries instead of 3. A downside is that if a project only has 5 confidential issues the counter will be set to 0. Because we now have 3 similar counting service classes the code previously used in Projects::ForksCountService has mostly been moved to Projects::CountService, which in turn is reused by the various service classes. Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/36622
* Merge issuable "reopened" state into "opened"merge-issuable-reopened-into-opened-stateYorick Peterse2017-07-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Having two states that essentially mean the same thing is very much like having a boolean "true" and boolean "mostly-true": it's rather silly. This commit merges the "reopened" state into the "opened" state while taking care of system notes still showing messages along the lines of "Alice reopened this issue". A big benefit from having only two states (opened and closed) is that indexing and querying becomes simpler and more performant. For example, to get all the opened queries we no longer have to query both states: SELECT * FROM issues WHERE project_id = 2 AND state IN ('opened', 'reopened'); Instead we can query a single state directly, which can be much faster: SELECT * FROM issues WHERE project_id = 2 AND state = 'opened'; Further, only having two states makes indexing easier as we will only ever filter (and thus scan an index) using a single value. Partial indexes could help but aren't supported on MySQL, complicating the development process and not being helpful for MySQL.
* Remove superfluous lib: true, type: redis, service: true, models: true, ↵Rémy Coutable2017-07-271-1/+1
| | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me>
* Enable Style/DotPosition Rubocop :cop:Grzegorz Bizon2017-06-211-2/+2
|
* Fix counters cache invalidation for Issues and Merge RequestsValery Sizov2017-05-261-0/+2
|
* Rephrase some system notes to be compatible with new system note stylerephrase-system-notesDouwe Maan2016-11-241-1/+1
|
* Enforce permissions in `{Issues,MergeRequests}::{Close,Reopen}Service`Rémy Coutable2016-08-131-3/+16
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* adds second batch of tests changed to active tenseactive-tense-test-coveragetiagonbotelho2016-08-091-3/+3
|
* Avoid `describe`-ing symbols in specsrs-describe-symbolsRobert Speicher2016-07-121-1/+1
|
* Tag service specsDouwe Maan2015-12-091-1/+1
|
* test fixValery Sizov2015-11-301-1/+3
|
* Updated rspec to rspec 3.x syntaxJeroen van Baarsen2015-02-121-6/+6
| | | | Signed-off-by: Jeroen van Baarsen <jeroenvanbaarsen@gmail.com>
* Add action property to merge request hookjubianchi2015-01-201-0/+45