summaryrefslogtreecommitdiff
path: root/spec/services/groups
Commit message (Collapse)AuthorAgeFilesLines
* Handle creating a nested group on MySQL correctlybvl-fix-mysql-bare-repository-importerBob Van Landuyt2017-08-251-26/+61
| | | | | Since we don't support nested groups on MySQL, raise an error explaining that on import instead of trying anyway.
* Add `:nested_groups` metadata to `Groups::NestedCreateService` specsRobert Speicher2017-08-241-2/+2
|
* Migrate creation of nested groups into a serviceBob Van Landuyt2017-08-231-0/+53
|
* Add `:nested_groups` metadata to two subgroup-related specsRobert Speicher2017-08-221-1/+1
| | | | | | Prevents these from failing on MySQL. Closes #36811 and #36812.
* Merge branch '35845-improve-subgroup-creation-permissions' into 'master'Douwe Maan2017-08-221-1/+13
|\ | | | | | | | | | | | | Improves subgroup creation permissions Closes #35845 See merge request !13418
| * Improves subgroup creation permissions35845-improve-subgroup-creation-permissionsTiago Botelho2017-08-211-1/+13
| |
* | Make sure repository's removal work for legacy and hashed storages28283-uuid-storageGabriel Mazetto2017-08-221-2/+24
|/
* Change all `:empty_project` to `:project`rs-empty_project-defaultRobert Speicher2017-08-022-4/+4
|
* Merge branch 'zj-delete-mm-team' into 'master'Douwe Maan2017-07-281-0/+10
|\ | | | | | | | | | | | | Remove Mattermost team with GitLab group Closes #30023 See merge request !11362
| * Remove Mattermost team with GitLab groupZ.J. van de Weg2017-07-281-0/+10
| | | | | | | | | | | | | | When destroying a group, now an API call is made to the Mattermost server to request the deletion of the project. Actual team deletion on the Mattermost side happens async, so the runtime shouldn't increase by more than a second.
* | Use described_class when possibleRémy Coutable2017-07-271-1/+1
| | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | Remove superfluous lib: true, type: redis, service: true, models: true, ↵Rémy Coutable2017-07-273-3/+3
|/ | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me>
* Defer project destroys within a namespace in ↵Stan Hu2017-06-291-20/+32
| | | | | | | | | | | Groups::DestroyService#async_execute Group#destroy would actually hard-delete all associated projects even though the acts_as_paranoia gem is used, preventing Projects::DestroyService from doing any work. We first noticed this while trying to log all projects deletion to the Geo log.
* Correct RSpec/SingleLineHook cop offensesRobert Speicher2017-06-141-2/+6
|
* add spec and changelogfix/orphan-notification-settingsJames Lopez2017-04-191-0/+2
|
* Merge branch 'jej-group-name-disclosure' into 'security'Sean McGivern2017-03-291-0/+14
| | | | | | Prevent private group disclosure via parent_id See merge request !2077
* Add a new `stub_mattermost_setting` stub helper to properly stub Mattermost ↵Rémy Coutable2017-03-281-1/+1
| | | | | | settings Signed-off-by: Rémy Coutable <remy@rymai.me>
* Use `:empty_project` where possible in service specsRobert Speicher2017-03-272-4/+4
|
* Merge branch 'master' into zj-create-mattermost-teamZ.J. van de Weg2017-03-021-0/+2
|\
| * Include nested group into destroy_service_specsh-async-delete-childrenDmitriy Zaporozhets2017-02-271-0/+2
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Improve UXZ.J. van de Weg2017-03-021-6/+0
| |
* | Fix testsZ.J. van de Weg2017-02-241-1/+5
| |
* | Merge branch 'master' into zj-create-mattermost-teamZ.J. van de Weg2017-02-201-3/+16
|\ \ | |/
| * Fix a number of race conditions that can occur during namespace deletionsh-namespace-cleanup-deleted-projectsStan Hu2017-02-151-3/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are two problems in the current implementation: 1. If a project is marked for deletion via the `pending_delete` flag and then the namespace was quickly deleted, it's possible that the namespace skips over that project and leaves that project in an orphaned state. 2. Before namespace deletion, the namespace attempts to clean up all the relevant storage paths. However, if all projects have been removed synchronously, then the namespace will not be able to clean anything. To prevent this, we should load the paths to be deleted before actually destroying projects. The specs were missing this second case due to a permission issue that caused project removal never to happen.
* | Transactional mattermost team creationZ.J. van de Weg2017-02-201-2/+9
| | | | | | | | | | | | | | Before this commit, but still on this feature branch, the creation of mattermost teams where a background job. However, it was decided it was better that these happened as transaction so feedback could be displayed to the user.
* | Add tests for Mattermost team creationZ.J. van de Weg2017-02-161-4/+13
|/
* Fix inconsistent naming for services that delete thingsdixpac2017-02-081-0/+98
| | | | | | * Changed name of delete_user_service and worker to destroy * Move and change delete_group_service to Groups::DestroyService * Rename Notes::DeleteService to Notes::DestroyService
* Use random group name to prevent conflictsDouwe Maan2017-02-061-1/+1
|
* Add nested groups support to the Groups::CreateServiceDmitriy Zaporozhets2016-12-261-4/+27
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* added more specsJames Lopez2016-12-211-2/+4
|
* Fix error 500 renaming group. Also added specs and changelog.James Lopez2016-12-201-7/+42
|
* Update tests for the current_application_settings request store changesDJ Mountney2016-05-271-1/+1
|
* Enable RSpec/NotToNot cop and auto-correct offensesrs-rubocop-nottonotRobert Speicher2016-05-241-1/+1
| | | | Also removes the note from the development/testing.md guide
* Fix specsDouwe Maan2016-03-201-4/+7
|
* Fix more specsDouwe Maan2016-03-201-16/+14
|
* Improve group visibility level featureZeger-Jan van de Weg2016-03-182-20/+20
|
* Code fixesFelipe Artur2016-03-171-7/+5
|
* Code improvements and add Create group serviceFelipe Artur2016-03-161-0/+22
|
* Prevent projects to have higher visibility than groupsFelipe Artur2016-03-101-0/+51
Prevent Groups to have smaller visibility than projects Add default_group_visibility_level to configuration Code improvements