From 2efe6578ce8f876eeb291f34bcdcf801e4f0fbca Mon Sep 17 00:00:00 2001
From: David Taylor <david@taylorhq.com>
Date: Mon, 3 Oct 2022 21:20:41 +0100
Subject: [PATCH] DEV: Bump rubocop-discourse (#71)

* DEV: Bump rubocop-discourse

* Fix rubocop violations
---
 Gemfile.lock        | 22 ++++++++++++----------
 spec/plugin_spec.rb | 10 +++++-----
 2 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/Gemfile.lock b/Gemfile.lock
index 80f9054..2434c95 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -2,30 +2,32 @@ GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.2)
+    json (2.6.2)
     parallel (1.22.1)
-    parser (3.1.2.0)
+    parser (3.1.2.1)
       ast (~> 2.4.1)
     rainbow (3.1.1)
-    regexp_parser (2.5.0)
+    regexp_parser (2.6.0)
     rexml (3.2.5)
-    rubocop (1.30.1)
+    rubocop (1.36.0)
+      json (~> 2.3)
       parallel (~> 1.10)
-      parser (>= 3.1.0.0)
+      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.18.0, < 2.0)
+      rubocop-ast (>= 1.20.1, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 3.0)
-    rubocop-ast (1.18.0)
+    rubocop-ast (1.21.0)
       parser (>= 3.1.1.0)
-    rubocop-discourse (2.5.0)
+    rubocop-discourse (3.0)
       rubocop (>= 1.1.0)
       rubocop-rspec (>= 2.0.0)
-    rubocop-rspec (2.11.1)
-      rubocop (~> 1.19)
+    rubocop-rspec (2.13.2)
+      rubocop (~> 1.33)
     ruby-progressbar (1.11.0)
-    unicode-display_width (2.1.0)
+    unicode-display_width (2.3.0)
 
 PLATFORMS
   arm64-darwin-20
diff --git a/spec/plugin_spec.rb b/spec/plugin_spec.rb
index c5a8a03..f63837f 100644
--- a/spec/plugin_spec.rb
+++ b/spec/plugin_spec.rb
@@ -3,7 +3,7 @@
 require 'rails_helper'
 
 describe OAuth2BasicAuthenticator do
-  context 'after_authenticate' do
+  describe 'after_authenticate' do
     let(:user) { Fabricate(:user) }
     let(:authenticator) { OAuth2BasicAuthenticator.new }
 
@@ -57,7 +57,7 @@ describe OAuth2BasicAuthenticator do
       expect(result.email_valid).to eq(false)
     end
 
-    context "fetch_user_details" do
+    describe "fetch_user_details" do
       before(:each) do
         SiteSetting.oauth2_fetch_user_details = true
         SiteSetting.oauth2_user_json_url = "https://provider.com/user"
@@ -126,7 +126,7 @@ describe OAuth2BasicAuthenticator do
       end
     end
 
-    context 'avatar downloading' do
+    describe 'avatar downloading' do
       before do
         SiteSetting.queue_jobs = true
         SiteSetting.oauth2_fetch_user_details = true
@@ -165,7 +165,7 @@ describe OAuth2BasicAuthenticator do
         auth_result = nil
         expect {
           auth_result = authenticator.after_authenticate(auth)
-        }.to change { job_klass.jobs.count }.by(0)
+        }.not_to change { job_klass.jobs.count }
 
         expect {
           authenticator.after_create_account(user, auth_result)
@@ -258,7 +258,7 @@ describe OAuth2BasicAuthenticator do
     expect(result).to eq 'http://example.com/1.png'
   end
 
-  context 'token_callback' do
+  describe 'token_callback' do
     let(:user) { Fabricate(:user) }
     let(:strategy) { OmniAuth::Strategies::Oauth2Basic.new({}) }
     let(:authenticator) { OAuth2BasicAuthenticator.new }