Fixes invalid mapping in shared/social partial #215
Conversation
| @@ -0,0 +1,3 @@ | |||
| RSpec.configure do |config| | |||
There was a problem hiding this comment.
Missing magic comment # frozen_string_literal: true.
| api_secret: 'fake', | ||
| environment: Rails.env, | ||
| active: true) | ||
| create(:authentication_method, provider: 'twitter') |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| api_secret: 'fake', | ||
| environment: Rails.env, | ||
| active: true) | ||
| create(:authentication_method, provider: 'facebook') |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| end | ||
|
|
||
| it 'shows method as connected' do | ||
| expect(page).to have_text('You Have Signed In With These Services') |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| expect(page).not_to have_link(title: Spree.t(:sign_in_with, provider: authentication_method.provider)) | ||
| end | ||
|
|
||
| it 'shows method as connected' do |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| visit '/account' | ||
| end | ||
|
|
||
| it 'shows possible methods to connect' do |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| let!(:authentication_method) { create(:authentication_method) } | ||
|
|
||
| before do | ||
| visit '/account' |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| login_as(user, scope: :spree_user) | ||
| end | ||
|
|
||
| context 'with existing active authentication methods' do |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| @@ -0,0 +1,35 @@ | |||
| RSpec.feature 'account page visit', :js do | |||
There was a problem hiding this comment.
Block has too many lines. [26/25]
Missing magic comment # frozen_string_literal: true.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
| FactoryBot.define do | ||
| factory :user_authentication, class: Spree::UserAuthentication do | ||
| provider 'facebook' | ||
| uid 'fake' |
There was a problem hiding this comment.
Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.
Fixes #202
Added tests for account page