DEV: Bump rubocop-discourse and correct violations (#409)

Author: davidtaylorhq

Gemfile

@@ -3,6 +3,6 @@
 source "https://rubygems.org"
 
 group :development do
-  gem "rubocop-discourse", git: "https://github.com/discourse/rubocop-discourse/", branch: "stree"
+  gem "rubocop-discourse"
   gem "syntax_tree"
 end

Gemfile.lock

@@ -1,37 +1,31 @@
-GIT
-  remote: https://github.com/discourse/rubocop-discourse/
-  revision: 8afca6460a423a11a2e0bf1f7051b18dd9a7231b
-  branch: stree
-  specs:
-    rubocop-discourse (2.5.0)
-      rubocop (>= 1.1.0)
-      rubocop-rspec (>= 2.0.0)
-
 GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
-    json (2.6.2)
+    json (2.6.3)
     parallel (1.22.1)
-    parser (3.1.2.1)
+    parser (3.1.3.0)
       ast (~> 2.4.1)
     prettier_print (0.1.0)
     rainbow (3.1.1)
-    regexp_parser (2.6.0)
+    regexp_parser (2.6.1)
     rexml (3.2.5)
-    rubocop (1.36.0)
+    rubocop (1.41.1)
       json (~> 2.3)
       parallel (~> 1.10)
       parser (>= 3.1.2.1)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.20.1, < 2.0)
+      rubocop-ast (>= 1.23.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.21.0)
+    rubocop-ast (1.24.0)
       parser (>= 3.1.1.0)
-    rubocop-rspec (2.13.2)
+    rubocop-discourse (3.0.1)
+      rubocop (>= 1.1.0)
+      rubocop-rspec (>= 2.0.0)
+    rubocop-rspec (2.16.0)
       rubocop (~> 1.33)
     ruby-progressbar (1.11.0)
     syntax_tree (3.2.1)
@@ -42,7 +36,7 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  rubocop-discourse!
+  rubocop-discourse
   syntax_tree
 
 BUNDLED WITH

lib/random_assign_utils.rb

@@ -23,9 +23,9 @@ def self.automation_script!(context, fields, automation)
     min_hours = fields.dig("minimum_time_between_assignments", "value").presence
     if min_hours &&
          TopicCustomField
-           .where(name: "assigned_to_id", topic_id: topic_id)
-           .where("created_at < ?", min_hours.to_i.hours.ago)
-           .exists?
+             .where(name: "assigned_to_id", topic_id: topic_id)
+             .where("created_at < ?", min_hours.to_i.hours.ago)
+             .exists?
       log_info(automation, "Topic(#{topic_id}) has already been assigned recently")
       return
     end

spec/components/search_spec.rb

@@ -12,8 +12,8 @@
     SiteSetting.assign_enabled = true
   end
 
-  context "Advanced search" do
-    include_context "A group that is allowed to assign"
+  describe "Advanced search" do
+    include_context "with group that is allowed to assign"
 
     let(:post1) { Fabricate(:post) }
     let(:post2) { Fabricate(:post) }

spec/components/topic_query_spec.rb

@@ -11,7 +11,7 @@
   fab!(:user3) { Fabricate(:user) }
   fab!(:user4) { Fabricate(:user) }
 
-  include_context "A group that is allowed to assign"
+  include_context "with group that is allowed to assign"
 
   before do
     add_to_assign_allowed_group(user)
@@ -183,7 +183,7 @@
     end
   end
 
-  context "assigned filter" do
+  describe "assigned filter" do
     it "filters topics assigned to a user" do
       assigned_topic = Fabricate(:post).topic
       assigned_topic2 = Fabricate(:post).topic

spec/components/topics_bulk_action_spec.rb

@@ -13,7 +13,7 @@
   let(:user) { Fabricate(:user) }
   let(:user2) { Fabricate(:user) }
 
-  include_context "A group that is allowed to assign"
+  include_context "with group that is allowed to assign"
 
   before { add_to_assign_allowed_group(user) }
 

spec/integration/assign_spec.rb

@@ -23,7 +23,7 @@
     let(:channel) { "/private-messages/assigned" }
     fab!(:group) { Fabricate(:group, assignable_level: Group::ALIAS_LEVELS[:everyone]) }
 
-    include_context "A group that is allowed to assign"
+    include_context "with group that is allowed to assign"
 
     before do
       add_to_assign_allowed_group(user)
@@ -102,7 +102,7 @@ def assert_publish_topic_state(topic, user: nil, group: nil)
     let(:user1) { Fabricate(:user) }
     let(:user2) { Fabricate(:user) }
 
-    include_context "A group that is allowed to assign"
+    include_context "with group that is allowed to assign"
 
     before do
       add_to_assign_allowed_group(user1)
@@ -138,13 +138,13 @@ def assert_publish_topic_state(topic, user: nil, group: nil)
     end
   end
 
-  context "already assigned" do
+  context "when already assigned" do
     fab!(:post) { Fabricate(:post) }
     fab!(:post_2) { Fabricate(:post, topic: post.topic) }
     let(:topic) { post.topic }
     fab!(:user) { Fabricate(:user) }
 
-    include_context "A group that is allowed to assign"
+    include_context "with group that is allowed to assign"
 
     it "does not allow to assign topic if post is already assigned" do
       add_to_assign_allowed_group(user)
@@ -164,7 +164,7 @@ def assert_publish_topic_state(topic, user: nil, group: nil)
     end
   end
 
-  context "move post" do
+  describe "move post" do
     fab!(:old_topic) { Fabricate(:topic) }
     fab!(:post) { Fabricate(:post, topic: old_topic) }
     fab!(:user) { Fabricate(:user) }

spec/jobs/regular/assign_notification_spec.rb

@@ -21,7 +21,7 @@ def assert_publish_topic_state(topic, user)
 
     before { assign_allowed_group.add(user1) }
 
-    context "User" do
+    describe "User" do
       it "sends notification alert" do
         messages =
           MessageBus.track_publish("/notification-alert/#{user2.id}") do
@@ -89,7 +89,7 @@ def assert_publish_topic_state(topic, user)
       end
     end
 
-    context "Group" do
+    describe "Group" do
       fab!(:user3) { Fabricate(:user, last_seen_at: 1.day.ago) }
       fab!(:user4) { Fabricate(:user, suspended_till: 1.year.from_now) }
       fab!(:group) { Fabricate(:group, name: "Developers") }

spec/jobs/regular/unassign_notification_spec.rb

@@ -21,7 +21,7 @@ def assert_publish_topic_state(topic, user)
       expect(message.user_ids).to eq([user.id])
     end
 
-    context "User" do
+    describe "User" do
       it "deletes notifications" do
         Jobs::AssignNotification.new.execute(
           {
@@ -64,7 +64,7 @@ def assert_publish_topic_state(topic, user)
       end
     end
 
-    context "Group" do
+    describe "Group" do
       fab!(:assign_allowed_group) { Group.find_by(name: "staff") }
       fab!(:user3) { Fabricate(:user) }
       fab!(:group) { Fabricate(:group) }

spec/lib/assigner_spec.rb

@@ -12,7 +12,7 @@
   let(:pm_post) { Fabricate(:private_message_post) }
   let(:pm) { pm_post.topic }
 
-  context "assigning and unassigning" do
+  describe "assigning and unassigning" do
     let(:post) { Fabricate(:post) }
     let(:topic) { post.topic }
     let(:secure_category) { Fabricate(:private_category, group: Fabricate(:group)) }
@@ -53,7 +53,7 @@
       expect {
         described_class.new(topic, admin).assign(moderator_2)
       }.to change { moderator.notifications.count }.by(-1)
-      .and change { moderator_2.notifications.count }.by(1)
+        .and change { moderator_2.notifications.count }.by(1)
     end
 
     it "can assign with note" do
@@ -186,7 +186,7 @@ def assigned_to?(assignee)
       assigner.assign(assignee).fetch(:success)
     end
 
-    context "forbidden reasons" do
+    describe "forbidden reasons" do
       it "doesn't assign if the topic has more than 5 assignments" do
         other_post = nil
 
@@ -487,7 +487,7 @@ def assigned_to?(assignee)
     end
   end
 
-  context "assign_self_regex" do
+  describe "assign_self_regex" do
     fab!(:me) { Fabricate(:admin) }
     fab!(:op) { Fabricate(:post) }
     fab!(:reply) do
@@ -528,7 +528,7 @@ def assigned_to?(assignee)
     end
   end
 
-  context "assign_other_regex" do
+  describe "assign_other_regex" do
     fab!(:me) { Fabricate(:admin) }
     fab!(:other) { Fabricate(:admin) }
     fab!(:op) { Fabricate(:post) }
@@ -553,12 +553,12 @@ def assigned_to?(assignee)
     end
   end
 
-  context "unassign_on_close" do
+  describe "unassign_on_close" do
     let(:post) { Fabricate(:post) }
     let(:topic) { post.topic }
     let(:moderator) { Fabricate(:moderator) }
 
-    context "topic" do
+    context "with topic" do
       let(:assigner) { described_class.new(topic, moderator) }
 
       before do
@@ -595,7 +595,7 @@ def assigned_to?(assignee)
       end
     end
 
-    context "post" do
+    context "with post" do
       let(:post_2) { Fabricate(:post, topic: topic) }
       let(:assigner) { described_class.new(post_2, moderator) }
       let(:post_3) { Fabricate(:post, topic: topic) }
@@ -650,12 +650,12 @@ def assigned_to?(assignee)
     end
   end
 
-  context "reassign_on_open" do
+  describe "reassign_on_open" do
     let(:post) { Fabricate(:post) }
     let(:topic) { post.topic }
     let(:moderator) { Fabricate(:moderator) }
 
-    context "topic" do
+    describe "topic" do
       let(:assigner) { described_class.new(topic, moderator) }
 
       before do
@@ -677,7 +677,7 @@ def assigned_to?(assignee)
       end
     end
 
-    context "post" do
+    context "with post" do
       let(:post_2) { Fabricate(:post, topic: topic) }
       let(:assigner) { described_class.new(post_2, moderator) }
 
@@ -702,7 +702,7 @@ def assigned_to?(assignee)
     end
   end
 
-  context "assign_emailer" do
+  describe "assign_emailer" do
     let(:post) { Fabricate(:post) }
     let(:topic) { post.topic }
     let(:moderator) { Fabricate(:moderator) }
@@ -719,9 +719,9 @@ def assigned_to?(assignee)
     it "doesn't send an email if assignee is a group" do
       SiteSetting.assign_mailer = AssignMailer.levels[:always]
 
-      expect { described_class.new(topic, moderator).assign(assign_allowed_group) }.to change {
+      expect { described_class.new(topic, moderator).assign(assign_allowed_group) }.not_to change {
         ActionMailer::Base.deliveries.size
-      }.by(0)
+      }
     end
 
     it "doesn't send an email if the assigner and assignee are not different" do
@@ -735,17 +735,17 @@ def assigned_to?(assignee)
     it "doesn't send an email if the assigner and assignee are not different" do
       SiteSetting.assign_mailer = AssignMailer.levels[:different_users]
 
-      expect { described_class.new(topic, moderator).assign(moderator) }.to change {
+      expect { described_class.new(topic, moderator).assign(moderator) }.not_to change {
         ActionMailer::Base.deliveries.size
-      }.by(0)
+      }
     end
 
     it "doesn't send an email" do
       SiteSetting.assign_mailer = AssignMailer.levels[:never]
 
-      expect { described_class.new(topic, moderator).assign(moderator_2) }.to change {
+      expect { described_class.new(topic, moderator).assign(moderator_2) }.not_to change {
         ActionMailer::Base.deliveries.size
-      }.by(0)
+      }
     end
   end
 end

spec/lib/pending_assigns_reminder_spec.rb

@@ -4,7 +4,7 @@
 require_relative "../support/assign_allowed_group"
 
 def assert_reminder_not_created
-  expect { subject.remind(user) }.to change { Post.count }.by(0)
+  expect { subject.remind(user) }.not_to change { Post.count }
 end
 
 RSpec.describe PendingAssignsReminder do
@@ -26,7 +26,7 @@ def assert_reminder_not_created
   describe "when the user has multiple tasks" do
     let(:system) { Discourse.system_user }
 
-    include_context "A group that is allowed to assign"
+    include_context "with group that is allowed to assign"
 
     before do
       add_to_assign_allowed_group(user)