BigW Consortium Gitlab

Commit 6db238de by Douwe Maan

Merge branch 'workaround-seed-fu-issue' into 'master'

Fixes GDK issue where repos would not be imported properly Seed-Fu runs this entire fixture in a transaction, so the `after_commit` hook won't run until after the fixture is loaded. That is too late since the Sidekiq::Testing block has already exited. Force clearing the `after_commit` queue to ensure the job is run now. See: gitlab-org/gitlab-development-kit#58 See merge request !1513
parents 1115caf8 9ccd8e26
...@@ -73,8 +73,13 @@ Sidekiq::Testing.inline! do ...@@ -73,8 +73,13 @@ Sidekiq::Testing.inline! do
} }
project = Projects::CreateService.new(User.first, params).execute project = Projects::CreateService.new(User.first, params).execute
# Seed-Fu runs this entire fixture in a transaction, so the `after_commit`
# hook won't run until after the fixture is loaded. That is too late
# since the Sidekiq::Testing block has already exited. Force clearing
# the `after_commit` queue to ensure the job is run now.
project.send(:_run_after_commit_queue)
if project.valid? if project.valid? && project.valid_repo?
print '.' print '.'
else else
puts project.errors.full_messages puts project.errors.full_messages
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment