diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-04-08 06:09:23 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-04-08 06:09:23 +0300 |
commit | 807003377d93b2cea9c1f79949a64bfce70202a8 (patch) | |
tree | ddacb167dae57646965d5e569e327c9acfcb1689 /spec/services/todos | |
parent | b89bcf56ec86e9b510ff0ae09887408e747e55f2 (diff) |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'spec/services/todos')
-rw-r--r-- | spec/services/todos/destroy/destroyed_issuable_service_spec.rb | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/spec/services/todos/destroy/destroyed_issuable_service_spec.rb b/spec/services/todos/destroy/destroyed_issuable_service_spec.rb new file mode 100644 index 00000000000..24f74bae7c8 --- /dev/null +++ b/spec/services/todos/destroy/destroyed_issuable_service_spec.rb @@ -0,0 +1,34 @@ +# frozen_string_literal: true + +require 'spec_helper' + +RSpec.describe Todos::Destroy::DestroyedIssuableService do + describe '#execute' do + let_it_be(:target) { create(:merge_request) } + let_it_be(:pending_todo) { create(:todo, :pending, project: target.project, target: target, user: create(:user)) } + let_it_be(:done_todo) { create(:todo, :done, project: target.project, target: target, user: create(:user)) } + + def execute + described_class.new(target.id, target.class.name).execute + end + + it 'deletes todos for specified target ID and type' do + control_count = ActiveRecord::QueryRecorder.new { execute }.count + + # Create more todos for the target + create(:todo, :pending, project: target.project, target: target, user: create(:user)) + create(:todo, :pending, project: target.project, target: target, user: create(:user)) + create(:todo, :done, project: target.project, target: target, user: create(:user)) + create(:todo, :done, project: target.project, target: target, user: create(:user)) + + expect { execute }.not_to exceed_query_limit(control_count) + expect(target.reload.todos.count).to eq(0) + end + + it 'invalidates todos cache counts of todo users', :use_clean_rails_redis_caching do + expect { execute } + .to change { pending_todo.user.todos_pending_count }.from(1).to(0) + .and change { done_todo.user.todos_done_count }.from(1).to(0) + end + end +end |