Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGitLab Bot <gitlab-bot@gitlab.com>2021-04-08 06:09:23 +0300
committerGitLab Bot <gitlab-bot@gitlab.com>2021-04-08 06:09:23 +0300
commit807003377d93b2cea9c1f79949a64bfce70202a8 (patch)
treeddacb167dae57646965d5e569e327c9acfcb1689 /spec/services/todos
parentb89bcf56ec86e9b510ff0ae09887408e747e55f2 (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.rb34
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