Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DEV: Remove autospec hacks for social auth providers 

This was causing erratic test failures. Autospec continues to work after
removing, so this 5-year-old code is no longer needed.
This commit is contained in:
David Taylor 2018-10-09 12:42:57 +01:00
parent 1d2b48557c
commit ac89a728f8
3 changed files with 0 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
spec/components/auth/facebook_authenticator_spec.rb
View File

@ -1,11 +1,5 @@
require 'rails_helper' require 'rails_helper'
# In the ghetto ... getting the spec to run in autospec
# thing is we need to load up all auth really early pre-fork
# it means that the require is not going to get a new copy
Auth.send(:remove_const, :FacebookAuthenticator)
load 'auth/facebook_authenticator.rb'
describe Auth::FacebookAuthenticator do describe Auth::FacebookAuthenticator do
context 'after_authenticate' do context 'after_authenticate' do

4
spec/components/auth/google_oauth2_authenticator_spec.rb
View File

@ -1,9 +1,5 @@
require 'rails_helper' require 'rails_helper'
# For autospec:
Auth.send(:remove_const, :GoogleOAuth2Authenticator)
load 'auth/google_oauth2_authenticator.rb'
describe Auth::GoogleOAuth2Authenticator do describe Auth::GoogleOAuth2Authenticator do
it 'does not look up user unless email is verified' do it 'does not look up user unless email is verified' do

6
spec/components/auth/open_id_authenticator_spec.rb
View File

@ -1,11 +1,5 @@
require 'rails_helper' require 'rails_helper'
# In the ghetto ... getting the spec to run in autospec
# thing is we need to load up all auth really early pre-fork
# it means that the require is not going to get a new copy
Auth.send(:remove_const, :OpenIdAuthenticator)
load 'auth/open_id_authenticator.rb'
describe Auth::OpenIdAuthenticator do describe Auth::OpenIdAuthenticator do
it "can lookup pre-existing user if trusted" do it "can lookup pre-existing user if trusted" do