summaryrefslogtreecommitdiff
path: root/spec/services
Commit message (Collapse)AuthorAgeFilesLines
* Smarter flushing of branch statistics cachesYorick Peterse2016-02-091-3/+3
| | | | | | | | | | | | | | | | | Instead of flushing the behind/ahead counts for all branches upon every push we now only flush the cache of branches that actually need to have these statistics recalculated. There are now basically 2 scenarios and their effects: 1. A user pushes a commit to the default branch, this results in the cache being flushed for all branches. 2. A user pushes to a non default branch, this results in _only_ the cache for that branch being flushed. The existing code (Repository#expire_cache) remains backwards compatible with the previous behaviour, the new behaviour is only applied when a branch name is passed as an argument. This ensures that when for example a project is deleted the cache for all branches is flushed.
* Cache various Repository Git operationscaching-repository-git-operationsYorick Peterse2016-02-081-0/+30
| | | | | | | | | | | | | | This caches the output of the following methods: * Repository#empty? * Repository#has_visible_content? * Repository#root_ref The cache for Repository#has_visible_content? is flushed whenever a commit is pushed to a new branch or an existing branch is removed. The cache for Repository#root_ref is only flushed whenever a user changes the default branch of a project. The cache for Repository#empty? is never explicitly flushed as there's no need for it.
* Fix Rubocop errornote-background-jobJosh Frye2016-01-281-3/+3
|
* Update testsJosh Frye2016-01-282-3/+27
|
* Extract Projects::ImportService service from RepositoryImportWorkerDouglas Barbosa Alexandre2016-01-251-0/+106
|
* Use a MR with commits.Rubén Dávila2016-01-201-1/+1
|
* Fix creator should be added as a master of the project on creationDouglas Barbosa Alexandre2016-01-191-0/+1
|
* This will ensure previous assignee gets an email even if his notif level is ↵fix-consider-re-assign-as-a-mentionRémy Coutable2016-01-151-0/+40
| | | | "on mention"
* Consider re-assign as a mention from a notification point of viewRémy Coutable2016-01-151-1/+17
| | | | | This will ensure new assignee gets an email even if his notif level is "on mention".
* Merge branch 'feature/ldap-sync-edgecases' into 'master' Douwe Maan2016-01-141-0/+23
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | LDAP Sync blocked user edgecases Allow GitLab admins to block otherwise valid GitLab LDAP users (https://gitlab.com/gitlab-org/gitlab-ce/issues/3462) Based on the discussion on the original issue, we are going to differentiate "normal" block operations to the ldap automatic ones in order to make some decisions when its one or the other. Expected behavior: - [x] "ldap_blocked" users respond to both `blocked?` and `ldap_blocked?` - [x] "ldap_blocked" users can't be unblocked by the Admin UI - [x] "ldap_blocked" users can't be unblocked by the API - [x] Block operations that are originated from LDAP synchronization will flag user as "ldap_blocked" - [x] Only "ldap_blocked" users will be automatically unblocked by LDAP synchronization - [x] When LDAP identity is removed, we should convert `ldap_blocked` into `blocked` Mockup for the Admin UI with both "ldap_blocked" and normal "blocked" users: ![image](/uploads/4f56fc17b73cb2c9e2a154a22e7ad291/image.png) There will be another MR for the EE version. See merge request !2242
| * Code style fixes and some code simplifiedGabriel Mazetto2016-01-081-1/+1
| |
| * Repair ldap_blocked state when no ldap identity exist anymoreGabriel Mazetto2016-01-081-0/+23
| |
* | Merge branch 'add_username_to_user_system_hooks' of ↵Dmitriy Zaporozhets2016-01-131-16/+16
|\ \ | | | | | | | | | | | | | | | https://github.com/tbeadle/gitlabhq Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Include user_username in user_(add_to/remove_from)_(project/group) system hooks.Tommy Beadle2016-01-071-4/+4
| | |
| * | Include the username in user_create/destroy system hooks.Tommy Beadle2016-01-071-2/+2
| | |
* | | Merge pull request #9952 from tbeadle/user_id_in_member_system_hookDmitriy Zaporozhets2016-01-131-2/+2
|\ \ \ | | | | | | | | Include the user_id in user_*_team system hooks.
| * | | Include the user_id in user_*_team system hooks.Tommy Beadle2016-01-061-2/+2
| |/ / | | | | | | | | | | | | | | | This fixes an issue where the user_id is not included in the data for user_add_to_team and user_remove_from_team system hooks. The documentation already states that the user_id should be included.
* | | Merge branch 'api-project-upload' into 'master' Robert Speicher2016-01-081-12/+12
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | Add API project upload endpoint Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/4317 See merge request !2329
| * | DRY up upload and download servicesapi-project-uploadDouwe Maan2016-01-081-12/+12
| |/
* | Merge branch 'master' into milestone-refDouwe Maan2016-01-061-1/+1
|\ \ | |/
| * Fix notification specfix-notification-specDouwe Maan2016-01-051-1/+1
| |
* | Merge branch 'master' into milestone-refDouwe Maan2016-01-052-14/+51
|\ \ | |/
| * Merge branch 'pr-9938' into 'master' Douwe Maan2016-01-051-0/+3
| |\ | | | | | | | | | | | | | | | | | | Merge pull request GH-9938 from huacnlee/hotfix/note_mail_with_notification Hotfix note mail with notification See merge request !2288
| | * Merge pull request GH-9938 from huacnlee/hotfix/note_mail_with_notificationpr-9938Douwe Maan2016-01-041-0/+3
| | | | | | | | | | | | Hotfix note mail with notification
| * | Don't notify users twice if they are both project watchers and subscribersStan Hu2016-01-041-0/+12
| |/ | | | | | | Closes #4708
| * Added system hooks messages for renaming and transferring a projectSteve Norman2016-01-041-14/+33
| |
| * Fix specmention-allDouwe Maan2015-12-251-0/+2
| |
| * Fix specsDouwe Maan2015-12-241-0/+1
| |
* | Merge branch 'master' into milestone-refDouwe Maan2015-12-241-0/+39
|\ \ | |/
| * Merge branch 'fix/visibility-level-setting-in-forked-projects' into 'master' Douwe Maan2015-12-241-0/+39
| |\ | | | | | | | | | | | | | | | | | | | | | | | | Fix/visibility level setting in forked projects Fixes #3136 /cc @DouweM See merge request !1744
| | * Add some specs for forked project visibility_level casesTomasz Maczukin2015-12-241-0/+39
| | |
* | | Link to milestone in "Milestone changed" system noteDouwe Maan2015-12-241-1/+1
|/ /
* | Backport JIRA serviceDrew Blessing2015-12-182-0/+130
|/
* Don't create CI status for refs that doesn't have .gitlab-ci.yml, even if ↵Kamil Trzcinski2015-12-171-6/+11
| | | | the builds are enabled
* Fixed Rubocop offensesGabriel Mazetto2015-12-153-20/+20
|
* Merge branch 'ci-project-migrate' into 'master' Kamil Trzciński2015-12-147-219/+182
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ci Project migrate - This doesn't migrate: allow_git_fetch, coverage_regex, timeout. Since this are project configuration settings I would propose to migrate them to `.gitlab-ci.yml`. - This requires offline migrations. - It simplifies database models making all CI objects to be attached to: Project. - It removes Ci::Project, but makes /ci/projects working by adding method: Project.find_by_ci_id for backward compatibility (badges, triggers). - We should add default `timeout` to Application Settings. - It misses specs. - It is based on ci-services-migrate for now. - It removes CI events. - It removes administrator CI projects overview. - It removes CI application settings. In 8.4 or 8.5 we can remove redundant tables and columns. See merge request !1987
| * Fix specsKamil Trzcinski2015-12-111-2/+2
| |
| * Fix errorsKamil Trzcinski2015-12-112-2/+1
| |
| * Remove ci_ prefix from all ci related thingsKamil Trzcinski2015-12-112-172/+171
| |
| * Migrate CI::Project to ProjectKamil Trzcinski2015-12-115-48/+13
| |
* | Merge branch 'ci-services-migrate' into 'master' Kamil Trzciński2015-12-141-37/+0
|\ \ | |/ | | | | | | | | | | Ci Services migrate See merge request !1985
| * Migrate CI::Services and CI::WebHooks to Services and WebHooksKamil Trzcinski2015-12-101-37/+0
| |
* | Add spec for invalid options in project creationStan Hu2015-12-091-0/+7
|/
* Merge branch 'master' into split-up-buildsDouwe Maan2015-12-094-5/+120
|\
| * Merge branch 'master' into zj/gitlab-ce-merge-if-greenDouwe Maan2015-12-081-0/+53
| |\
| * \ Merge branch 'master' into merge-if-greenZeger-Jan van de Weg2015-12-0712-240/+412
| |\ \
| * | | Button fixZeger-Jan van de Weg2015-12-072-2/+2
| | | |
| * | | Incorporate feedbackZeger-Jan van de Weg2015-12-053-18/+25
| | | |
| * | | Specs for 'Merge When Build Succeeds'Zeger-Jan van de Weg2015-12-024-11/+102
| | | |
| * | | Merge branch 'master' into merge-if-greenZeger-Jan van de Weg2015-11-1810-80/+114
| |\ \ \