summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2018-07-02 18:06:11 +0300
committerDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>2018-07-02 18:06:11 +0300
commitf86d31c1b0429c890adee7edee554d052816a91f (patch)
treee25ff3e2d396448b957fa31b6f72e59399b3d653
parenta0481203d3ae2551325a8c197a1b9c090ba66b53 (diff)
downloadgitlab-ce-f86d31c1b0429c890adee7edee554d052816a91f.tar.gz
DRY group creation code in nested groups fixtures
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-rw-r--r--db/fixtures/development/20_nested_groups.rb28
1 files changed, 2 insertions, 26 deletions
diff --git a/db/fixtures/development/20_nested_groups.rb b/db/fixtures/development/20_nested_groups.rb
index 2bc78e120a5..3d95e243f8a 100644
--- a/db/fixtures/development/20_nested_groups.rb
+++ b/db/fixtures/development/20_nested_groups.rb
@@ -1,30 +1,5 @@
require './spec/support/sidekiq'
-def create_group_with_parents(user, full_path)
- parent_path = nil
- group = nil
-
- until full_path.blank?
- path, _, full_path = full_path.partition('/')
-
- if parent_path
- parent = Group.find_by_full_path(parent_path)
-
- parent_path += '/'
- parent_path += path
-
- group = Groups::CreateService.new(user, path: path, parent_id: parent.id).execute
- else
- parent_path = path
-
- group = Group.find_by_full_path(parent_path) ||
- Groups::CreateService.new(user, path: path).execute
- end
- end
-
- group
-end
-
Sidekiq::Testing.inline! do
Gitlab::Seeder.quiet do
flag = 'SEED_NESTED_GROUPS'
@@ -48,7 +23,8 @@ Sidekiq::Testing.inline! do
full_path = url.sub('https://android.googlesource.com/', '')
full_path = full_path.sub(/\.git\z/, '')
full_path, _, project_path = full_path.rpartition('/')
- group = Group.find_by_full_path(full_path) || create_group_with_parents(user, full_path)
+ group = Group.find_by_full_path(full_path) ||
+ Groups::NestedCreateService.new(user, group_path: full_path).execute
params = {
import_url: url,