Oct 3, 2022 · Oct 3, 2022 · Oct 3, 2022
diff --git a/Gemfile.lock b/Gemfile.lock
  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
 require 'rails_helper'

 describe OAuth2BasicAuthenticator do
 context 'after_authenticate' do
 describe 'after_authenticate' do
    let(:user) { Fabricate(:user) }
    let(:authenticator) { OAuth2BasicAuthenticator.new }

      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"
      end
    end

 context 'avatar downloading' do
 describe 'avatar downloading' do
      before do
        SiteSetting.queue_jobs = true
        SiteSetting.oauth2_fetch_user_details = true
        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)
    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 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -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
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -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 }

Expand DownExpand Up		@@ -57,7 +57,7 @@
		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"
Expand DownExpand Up		@@ -126,7 +126,7 @@
		end
		end

		context 'avatar downloading' do
		describe 'avatar downloading' do
		before do
		SiteSetting.queue_jobs = true
		SiteSetting.oauth2_fetch_user_details = true
Expand DownExpand Up		@@ -165,7 +165,7 @@
		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)
Expand DownExpand Up		@@ -258,7 +258,7 @@
		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 }
Expand Down