summaryrefslogtreecommitdiff
path: root/spec/services
Commit message (Collapse)AuthorAgeFilesLines
* Enable RSpec/FilePath copenable-spec-file-name-copSean McGivern2017-04-262-12/+12
| | | | | | - Ignore JS fixtures - Ignore qa directory - Rewrite concern specs to put concern name first
* Merge branch 'tc-realtime-every-pipeline-on-mr' into 'master' Kamil Trzciński2017-04-241-27/+0
|\ | | | | | | | | | | | | Properly expire cache for **all** MRs of a pipeline Closes #31040 See merge request !10770
| * Properly expire cache for **all** MRs of a pipelineToon Claes2017-04-241-27/+0
| | | | | | | | | | Turn ExpirePipelineCacheService into Worker so it can fetch all the merge requests for which the pipeline runs or did run against.
* | Move records to the ghost user in a transaction.Timothy Andrew2017-04-241-0/+18
|/ | | | | | | | | | | | | | | | - While deleting a user, some of the user's associated records are moved to the ghost user so they aren't deleted. The user is blocked before these records are moved, to prevent the user from creating new records while the migration is happening, and so preventing a data race. - Previously, if the migration failed, the user would _remain_ blocked, which is not the expected behavior. On the other hand, we can't just stick the block + migration into a transaction, because we want the block to be committed before the migration starts (for the data race reason mentioned above). - One solution (implemented in this commit) is to block the user in a parent transaction, migrate the associated records in a nested sub-transaction, and then unblock the user in the parent transaction if the sub-transaction fails.
* Merge branch 'uassign_on_member_removing' into 'master' Sean McGivern2017-04-201-0/+45
|\ | | | | | | | | | | | | Unassign all Issues and Merge Requests when member leaves a team Closes #30768 and #24117 See merge request !10755
| * Remove helpers assigned_issuables_count and cached_assigned_issuables_countuassign_on_member_removingValery Sizov2017-04-201-4/+4
| |
| * Remove assigned_(issues|merge_requests) methods from the issuableValery Sizov2017-04-181-2/+1
| |
| * Unassign all Issues and Merge Requests when member leaves teamValery Sizov2017-04-181-0/+46
| |
* | Refactor changing files in web UIDouwe Maan2017-04-201-3/+3
| |
* | Merge branch 'sh-issue-29247-fix' into 'master' Douwe Maan2017-04-191-0/+13
|\ \ | | | | | | | | | | | | | | | | | | Don't delete a branch involved in an open merge request in "Delete all merged branches" service Closes #29427 See merge request !10727
| * | Don't delete a branch involved in an open merge request in "Delete all ↵sh-issue-29247-fixStan Hu2017-04-191-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | merged branches" service Customers were surprised by the previous behavior, which destroyed branches even though an open merge request existed for it. Closes #29427
* | | Merge branch 'usage-ping-port' into 'master' Sean McGivern2017-04-193-0/+162
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Usage ping port Closes #27750 See merge request !10481
| * | Use last_activity_on in cohortsSean McGivern2017-04-141-2/+2
| | |
| * | Use serializer for formatting cohorts dataSean McGivern2017-04-141-15/+76
| | |
| * | Rename user cohorts -> cohortsSean McGivern2017-04-141-1/+1
| | |
| * | Make UserCohortsService more understandableSean McGivern2017-04-141-5/+1
| | | | | | | | | | | | | | | 1. Extract out into several methods. 2. Add more comments describing the data and the shape of the data.
| * | Add user cohorts table to admin areaSean McGivern2017-04-141-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This table shows the percentage of users who registered in the last twelve months, who last signed in during or later than each of those twelve months, by month. It is only enabled when the usage ping is enabled, and the page also shows pretty-printed usage ping data. The cohorts table is generated in Ruby from some basic SQL queries, because performing the gap-filling and running sums needed in both MySQL and Postgres is painful.
| * | Add a new Gitlab::UserActivities class to track user activitiesRémy Coutable2017-04-142-5/+11
| | | | | | | | | | | | | | | | | | This new class uses a Redis Hash instead of a Sorted Set. Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | Remove user activities table and use redis instead of PG for recording ↵James Lopez2017-04-142-22/+40
| | | | | | | | | | | | | | | | | | activities Refactored specs and added a post deployment migration to remove the activity users table.
| * | Add user activity service and spec. Also added relevant - NOT offline - ↵James Lopez2017-04-142-0/+39
| |/ | | | | | | | | | | | | migration It uses a user activity table instead of a column in users. Tested with mySQL and postgreSQL
* | Merge branch 'fix/orphan-notification-settings' into 'master' Yorick Peterse2017-04-191-0/+2
|\ \ | | | | | | | | | | | | | | | | | | Fix orphaned notification settings Closes #29688 See merge request !10763
| * | add spec and changelogfix/orphan-notification-settingsJames Lopez2017-04-191-0/+2
| | |
* | | Refactor group search out of global searchNick Thomas2017-04-182-21/+40
|/ /
* | Does not remove the GitHub remote when importing from GitHubDouglas Barbosa Alexandre2017-04-181-0/+9
| |
* | Hard delete users' associated records deleted from AbuseReportssh-add-option-to-hard-deleteStan Hu2017-04-161-0/+6
|/ | | | | | In the case of spammers, we really want a hard delete to avoid retaining spam. Closes #31021
* Merge branch '30349-create-users-build-service' into 'master' Rémy Coutable2017-04-142-66/+67
|\ | | | | | | | | | | | | Implement Users::BuildService Closes #30349 See merge request !10675
| * Refactor build_service_specGeorge Andrinopoulos2017-04-131-4/+2
| |
| * Implement Users::BuildServicegeoandri2017-04-132-66/+69
| |
* | Improve error messages when a record is unable to be created for a projectStan Hu2017-04-131-0/+14
|/
* Fix email not being sent after project exportmhasbini2017-04-091-0/+16
|
* Merge branch 'master' into new-resolvable-discussionLuke "Jared" Bennett2017-04-074-0/+80
|\
| * Merge branch 'bvl-fix-project-ci-status-cache' into 'master' Kamil Trzciński2017-04-071-0/+7
| |\ | | | | | | | | | | | | Fix invalidating Project build status cache to often See merge request !10313
| | * Use `Ci::ExpirePipelineCacheService` to set `ProjectPipelinestatus`bvl-fix-project-ci-status-cacheBob Van Landuyt2017-04-071-0/+7
| | |
| * | Merge branch '18471-restrict-tag-pushes-protected-tags' into 'master' Douwe Maan2017-04-073-0/+73
| |\ \ | | |/ | |/| | | | | | | | | | | | | Protected Tags Closes #18471 See merge request !10356
| | * Merge branch 'master' into '18471-restrict-tag-pushes-protected-tags'Kushal Pandya2017-04-065-52/+220
| | |\ | | | | | | | | | | | | # Conflicts: # spec/lib/gitlab/import_export/all_models.yml
| | * \ Merge branch 'last-green-master' into 18471-restrict-tag-pushes-protected-tagsJames Edwards-Jones2017-04-061-0/+27
| | |\ \
| | * | | Protected Tags backend review changesJames Edwards-Jones2017-04-062-0/+52
| | | | | | | | | | | | | | | Added changelog
| | * | | Merge branch 'master' into '18471-restrict-tag-pushes-protected-tags'Kushal Pandya2017-04-0674-384/+1778
| | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/assets/javascripts/dispatcher.js # app/assets/stylesheets/pages/projects.scss
| | * | | | Renamed ProtectedTag push_access_levels to create_access_levelsJames Edwards-Jones2017-04-041-2/+2
| | | | | |
| | * | | | Extract ProtectedRef ConcernJames Edwards-Jones2017-04-031-2/+0
| | | | | |
| | * | | | Protected tags copy/paste from protected branchesJames Edwards-Jones2017-03-311-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | Should provide basic CRUD backend for frontend to work from. Doesn’t include frontend, API, or the internal API used from gitlab-shell
* | | | | | Merge branch 'update-droplab-to-webpack-version' into new-resolvable-discussionLuke "Jared" Bennett2017-04-074-99/+153
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge branch '8998_skip_pending_commits_if_not_head' into 'master' Kamil Trzciński2017-04-072-98/+130
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add auto-cancel for pending pipelines on branch, if they are not HEAD See merge request !9362
| | * \ \ \ \ Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2017-04-071-1/+3
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8998_skip_pending_commits_if_not_head * upstream/master: (58 commits) Fix icon name error Rewrite system note helper Change edit icon Leave icon area blank if legacy note; remove diamond icon Fix positioning of note icons Fix newline errors Add remaining system note icons Add system notes icon helper; add icons Fixed specs Updated JS that was causing the hints to appear & then disappear Update tests Fix broken spinach test Reuse code Improve shortcuts code Adds ShortcutsDashboardNavigation to globals comment since its a global variable Fix shortcut specs Reorganize shortcut help menu Change todos shortcut to shift Change shortcuts Switch global shortcuts to shift; reuse key styles from help menu Map bindings to lowercase letters; only show key bindings when using keyboard shortcut ...
| | * \ \ \ \ \ Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2017-04-075-52/+220
| | |\ \ \ \ \ \ | | | | |_|_|_|/ | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8998_skip_pending_commits_if_not_head * upstream/master: (197 commits) Add text to break up diagrams Implement review comments from @DouweM for !10467. Fix rubocop offence Linking to edit file directly Optimise trace handling code to use streaming instead of full read Use config.toml to configure Gitaly Fix indexes in container repositories table Recent search history for issues Fix rubocop Use change direction in spec Use be_pending Improve trigger_schedule.rb Implement a offset calculation on cron_parser_spec Clean up trigger_schedule_worker_spec.rb Improve instantiate recursion in cron_parser.rb Fix unnecessary changes in schema.rb Add empty line in cron_parser.rb Use parenthesis for respond_to :ref Define next_time as let in trigger_schedule_spec Remove next_run_at: nil from trigger_schedule_spec ...
| | * | | | | | Fix test by ignoring auto_canceled_by for jobsLin Jen-Shin2017-04-071-3/+4
| | | | | | | |
| | * | | | | | Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2017-04-061-0/+27
| | |\ \ \ \ \ \ | | | | |_|_|_|/ | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8998_skip_pending_commits_if_not_head * upstream/master: Show CI status as Favicon on Pipelines, Job and MR pages STL file viewer Wait for the PDF to be loaded before doing anything remove unnecessary lease as cron job Search for opened MRs - include reopened MRs ProjectsFinder should handle more options Clearly show who triggered the pipeline in email Make it possible to preview pipeline success/failed emails Add remove_concurrent_index to database helper fix project authorizations migration issue attempt to fix migration Revert schema.rb attempt to fix db failure Periodically mark projects that are stuck in importing as failed Fix html structure to prevent tooltip from not hidding Enable creation of deploy keys with write access via the API
| | * | | | | | Merge remote-tracking branch 'upstream/master' into ↵Lin Jen-Shin2017-04-062-1/+67
| | |\ \ \ \ \ \ | | | | |_|_|_|/ | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8998_skip_pending_commits_if_not_head * upstream/master: (64 commits) Merge branch 'open-redirect-fix-continue-to' into 'security' Merge branch 'open-redirect-host-fix' into 'security' Merge branch 'path-disclosure-proj-import-export' into 'security' Merge branch '29364-private-projects-mr-fix' Merge branch '30125-markdown-security' Issue title realtime Update CHANGELOG.md for 8.16.9 Update CHANGELOG.md for 8.17.5 Update CHANGELOG.md for 9.0.4 Add "search" optional param and docs for V4 Use PDFLab to render PDFs in GitLab Separate Scala from Java in CI examples Fix broken link Reorganize CI examples, add more links Refactor CI index page Remove deprecated field from workhorse response Use gitlab-workhorse 1.4.3 Document how ETag caching middleware handles query parameters Make group skip validation in the frontend Use NamespaceValidator::WILDCARD_ROUTES in ETag caching middleware ...
| | * | | | | | Add a test to make sure it's not auto-cancelingLin Jen-Shin2017-04-061-10/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | whenever the feature is disabled in the project.
| | * | | | | | refactor specRydkin Maxim2017-04-041-10/+7
| | | | | | | |
View Lorry Controller Status