summaryrefslogtreecommitdiff
path: root/spec/services/groups
Commit message (Collapse)AuthorAgeFilesLines
* 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