summaryrefslogtreecommitdiff
path: root/spec/services/merge_requests/update_service_spec.rb
Commit message (Collapse)AuthorAgeFilesLines
* Create system notes for MR too, improve doc + clean up codeJarka Kadlecova2017-09-141-0/+7
|
* Support discussion locking in the backendJarka Kadlecova2017-09-141-1/+3
|
* Enable the Layout/SpaceBeforeBlockBraces coprc/enable-the-Layout/SpaceBeforeBlockBraces-copRémy Coutable2017-08-091-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Don't include EmailHelpers manually, pick with rspecuse-rspec-support-helperLin Jen-Shin2017-08-031-3/+1
| | | | | | | | | `:mailer` is needed to pick it easily, while `type: :mailer` is needed for picking it automatically for tests located in spec/mailers/*_spec.rb It's a bit complicated in spec/services/notification_service_spec.rb but we'll leave it alone for now.
* Use described_class when possibleRémy Coutable2017-07-271-7/+7
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* 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>
* Native group milestonesFelipe Artur2017-07-071-3/+3
|
* Enable Style/DotPosition Rubocop :cop:Grzegorz Bizon2017-06-211-4/+4
|
* Correct RSpec/SingleLineHook cop offensesRobert Speicher2017-06-141-3/+9
|
* Only add a description change note when no tasks are updatedJared Deckard2017-06-121-0/+13
|
* Add transient head_pipeline_of to pipeline factoriesissue_32225Felipe Artur2017-05-221-5/+6
|
* [Spec optimization] Joining one-line 'it' blocksspec-joining-itValery Sizov2017-05-151-15/+23
|
* Move update_assignee_cache_counts to the serviceValery Sizov2017-05-121-0/+9
|
* Fix specs 2Felipe Artur2017-05-081-1/+1
|
* Fix SpecsFelipe Artur2017-05-081-1/+3
|
* Merge branch 'blackst0ne/gitlab-ce-add_system_note_for_editing_issuable'Sean McGivern2017-05-051-0/+7
|\
| * Updated specsblackst0ne2017-05-051-0/+7
| |
* | Backport of multiple_assignees_feature [ci skip]Valery Sizov2017-05-041-0/+48
|/
* Create todos only for new mentionsJarka Kadlecova2017-03-291-1/+10
|
* Use `:empty_project` where possible in service specsRobert Speicher2017-03-271-1/+1
|
* Address MR commentsJarka Kadlecova2017-01-131-1/+1
|
* support `/merge` slash comand for MRsJarka Kadlecova2017-01-111-0/+93
|
* Ensure issuable state changes only fire webhooks onceMark Fletcher2016-12-161-0/+5
| | | | | * Webhooks for close and reopen events now fired in respective services only * Prevents generic 'update' webhooks firing too
* Only include EmailHelpers in mailer specs and specs using themrs-include-emailhelpersRobert Speicher2016-12-011-0/+2
|
* Rephrase some system notes to be compatible with new system note stylerephrase-system-notesDouwe Maan2016-11-241-12/+12
|
* Remove default value for `project` argument on subscribable concernDouglas Barbosa Alexandre2016-11-171-1/+1
|
* Maintain "force_remove_source_branch" options on Merge Request unless specifiedMark Fletcher2016-10-131-0/+12
|
* Make guests unable to view MRsguests_cant_see_mrsValery Sizov2016-10-111-0/+6
|
* Add a spec for merge request metric caching while refreshing a merge request ↵Timothy Andrew2016-09-211-3/+6
| | | | from a forked project.
* Use the `IssuableBaseService` lifecycle hooks to cache ↵Timothy Andrew2016-09-211-3/+3
| | | | | | | `MergeRequestsClosingIssues` - Instead of overriding `create` and `update` in `MergeRequests::BaseService` - Get all merge request service specs passing
* Implement a second round of review comments from @DouweM.Timothy Andrew2016-09-211-3/+3
| | | | | | | | | | | | | | - Don't use `TableReferences` - using `.arel_table` is shorter! - Move some database-related code to `Gitlab::Database` - Remove the `MergeRequest#issues_closed` and `Issue#closed_by_merge_requests` associations. They were either shadowing or were too similar to existing methods. They are not being used anywhere, so it's better to remove them to reduce confusion. - Use Rails 3-style validations - Index for `MergeRequest::Metrics#first_deployed_to_production_at` - Only include `CycleAnalyticsHelpers::TestGeneration` for specs that need it. - Other minor refactorings.
* Fix rubocop spec.Timothy Andrew2016-09-201-1/+1
| | | | And `scss_lint`
* Test all cycle analytics pre-calculation code.Timothy Andrew2016-09-191-0/+34
| | | | | | | | | | | | | All the code that pre-calculates metrics for use in the cycle analytics page. - Ci::Pipeline -> build start/finish - Ci::Pipeline#merge_requests - Issue -> record default metrics after save - MergeRequest -> record default metrics after save - Deployment -> Update "first_deployed_to_production_at" for MR metrics - Git Push -> Update "first commit mention" for issue metrics - Merge request create/update/refresh -> Update "merge requests closing issues"
* DRY up the added update service specs, add two email helpersNick Thomas2016-08-161-31/+3
|
* Address minor review comments for testsNick Thomas2016-08-161-4/+4
|
* Add notifications for new user mentions in merge requestsNick Thomas2016-08-151-0/+33
|
* adds second batch of tests changed to active tenseactive-tense-test-coveragetiagonbotelho2016-08-091-4/+4
|
* Merge branch 'rm-source-branch' into 'master' Douwe Maan2016-05-201-1/+3
|\ | | | | | | | | | | | | | | | | | | | | | | | | Allows MR authors to have the source branch removed when merging the MR closes #13191 The location of the checkbox might not be optimal so any feedback is welcome. Any other feedback too obviously. Screenshot: ![Screenshot_2016-02-17_21.25.24](/uploads/a9c3eaafb39c6f5b4f0949a2278af6da/Screenshot_2016-02-17_21.25.24.png) See merge request !2801
| * Allows MR authors to have the source branch removed when merging the MRJeroen Jacobs2016-05-201-1/+3
| |
* | Create DiffFilter and change SystemNoteService#change_title to use ↵Adam Butler2016-05-181-2/+2
|/ | | | Gitlab::Diff::InlineDiff
* Update specs to accomodate issuable assign changesGrzegorz Bizon2016-04-221-3/+8
|
* Improving the original label-subscribing implementationRémy Coutable2016-03-151-33/+22
| | | | | | | 1. Make the "subscribed" text in Issuable sidebar reflect the labels subscription status 2. Current user mut be logged-in to toggle issue/MR/label subscription
* Original implementation to allow users to subscribe to labelsTimothy Andrew2016-03-151-1/+55
| | | | | | | | | | | | | | | | | | 1. Allow subscribing (the current user) to a label - Refactor the `Subscription` coffeescript class - The main change is that it accepts a container, and conducts all DOM queries within its scope. We need this because the labels page has multiple instances of `Subscription` on the same page. 2. Creating an issue or MR with labels notifies users subscribed to those labels - Label `has_many` subscribers through subscriptions. 3. Adding a label to an issue or MR notifies users subscribed to those labels - This only applies to subscribers of the label that has just been added, not all labels for the issue.
* Rename Tasks to TodosDouglas Barbosa Alexandre2016-02-201-18/+19
|
* Create a pending task when a user is mentioned when edit a issue/mr/noteDouglas Barbosa Alexandre2016-02-201-3/+1
|
* Marks pending tasks for an user as done when he edit a MRDouglas Barbosa Alexandre2016-02-201-1/+54
|
* Create a pending task when an MR is assigned to someoneDouglas Barbosa Alexandre2016-02-201-0/+25
|
* Tag service specsDouwe Maan2015-12-091-1/+1
|
* test fixValery Sizov2015-11-301-2/+4
|
* Generate system note after Task item has been updated on Issue or Merge ↵Ruben Davila2015-11-191-6/+45
| | | | | | | | | | | | Request. #2296 Everytime the User check or uncheck a Task Item from the Issue or Merge Request description, a new update is going to be added to the activity logs of the Issue or Merge Request. Note that when using the edit form, you can only update the Task item status or add/delete/modify existing ones. Doing both actions is not fully supported.
View Lorry Controller Status