user_plane 0.0.15

data/spec/features/user_plane/user_plane_signed_in_only_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'shared_contexts/user'
+
+RSpec.feature "SignedInOnly", type: :feature do
+  include_context 'user'
+
+  scenario 'trying to access a signed-in-only' do
+    visit '/signed_in_only'
+    expect(current_path).to eql(polymorphic_path([User::SignIn.new], action: :new))
+        
+    fill_in 'Email', with: a_user.email.address
+    fill_in 'Password', with: a_user.email.password
+    click_button 'Sign In'
+
+    expect(current_path).to eql('/signed_in_only')
+    expect(page).to have_text 'You are successfully signed in.'
+  end
+
+  scenario 'trying to access a non-existing route when sigend in' do
+    visit '/non_existing'
+    expect(current_path).to eql(polymorphic_path([User::SignIn.new], action: :new))
+        
+    fill_in 'Email', with: a_user.email.address
+    fill_in 'Password', with: a_user.email.password
+    click_button 'Sign In'
+
+    expect(current_path).to eql('/non_existing')
+    expect(page.status_code).to eq(404)
+  end
+
+end

data/spec/features/user_plane/user_plane_update_details_spec.rb

@@ -0,0 +1,43 @@
+require 'spec_helper'
+require 'shared_contexts/user'
+require 'shared_contexts/feature_helpers'
+
+RSpec.feature "UpdateDetails", type: :feature do
+  include_context 'user'
+  include_context 'feature_helpers'
+
+  scenario 'changing the email address' do
+    sign_in(a_user)
+    
+    new_email = Faker::Internet.safe_email
+
+    visit polymorphic_path([User::UpdateDetails.new], action: :edit)
+    fill_in 'Email', with: new_email
+    click_button 'Update'
+
+    click_link 'Sign Out'
+
+    open_email(new_email)
+    current_email.click_link 'link'
+
+    
+    visit polymorphic_path([User::SignIn.new], action: :new)
+    fill_in 'Email', with: new_email
+    fill_in 'Password', with: a_user.email.password
+    click_button 'Sign In'
+
+    expect(page).to have_text 'You are successfully signed in.'
+    # expect(current_path).to eql('/signed_in_only')
+  end
+
+  scenario 'changing the user name' do
+    sign_in(a_user)
+    
+    visit polymorphic_path([User::UpdateDetails.new], action: :edit)
+    fill_in 'Name', with: 'marvin'
+    click_button 'Update'
+
+    expect(page).to have_selector('input[value="marvin"]')
+  end
+
+end

data/spec/fixtures/user_plane/invite_mailer/invite

@@ -0,0 +1,3 @@
+InviteMailer#invite
+
+Hi, find me in app/views/user_plane/invite_mailer/invite

data/spec/fixtures/user_plane/verification_mailer/address_verification

@@ -0,0 +1,3 @@
+VerificationMailer#address_verification
+
+Hi, find me in app/views/user_plane/verification_mailer/address_verification

data/spec/fixtures/user_plane/verification_mailer/password_reset

@@ -0,0 +1,3 @@
+VerificationMailer#password_reset
+
+Hi, find me in app/views/user_plane/verification_mailer/password_reset

data/spec/lib/generators/views_generator_spec.rb

@@ -0,0 +1,16 @@
+require "spec_helper"
+require 'generators/user_plane/views_generator'
+
+describe UserPlane::Generators::ViewsGenerator, type: :generator do
+  destination Dir.mktmpdir
+
+  before(:each) do
+    prepare_destination
+    run_generator
+  end
+
+  it "creates a test initializer" do
+    assert_file 'app/views/user/invites/new.html.erb', /form_for\(@send_sign_up_invite\)/
+    assert_file 'app/views/user/invites/edit.html.erb', /form_for\(@sign_up_with_invite\)/
+  end
+end

data/spec/lib/route_concerns_spec.rb

@@ -0,0 +1,54 @@
+require 'spec_helper'
+
+describe UserPlane::RouteConcerns::AbstractConcern do
+
+  let (:mapper) {double()}
+  before {allow(mapper).to receive(:namespace).and_yield}
+  
+  let :init_options do
+    {module: 'baz'}
+  end
+
+  subject :a_resource_concern do
+    class ConcreteConcern < described_class
+      def build
+        mapper.resource options(as: :foo)
+      end
+    end
+
+    ConcreteConcern.new(init_options)
+  end
+
+  it 'combines options' do
+    allow(mapper).to receive(:resource_scope?).and_return(true)
+    expect(mapper).to receive(:resource).with(hash_including(as: :foo))
+    a_resource_concern.call(mapper)
+  end
+
+
+  context 'concern-specific options' do
+    subject :a_two_level_concern do
+      class ConcreteConcern < described_class
+        def build
+          foo_controller = concern_options.delete(:controller)
+          mapper.resource options
+          mapper.resource options(as: :foo, to: "#{foo_controller}#action")
+        end
+      end
+
+      ConcreteConcern.new(init_options)
+    end
+
+    it 'combines options' do
+      allow(mapper).to receive(:resource_scope?).and_return(true)
+
+      allow(mapper).to receive(:resource)
+      expect(mapper).to receive(:resource).with(hash_including(as: :foo, to: 'foo#action'))
+
+      a_two_level_concern.call(mapper, controller: :foo)
+    end
+  end
+
+  it 'removes :on if the route is not a resoucrce scope'
+
+end

data/spec/mailers/previews/user_plane/invite_mailer_preview.rb

@@ -0,0 +1,11 @@
+module UserPlane
+  # Preview all emails at http://localhost:3000/rails/mailers/user_plane/invite_mailer
+  class InviteMailerPreview < ActionMailer::Preview
+
+    # Preview this email at http://localhost:3000/rails/mailers/user_plane/invite_mailer/invite
+    def invite
+      InviteMailer.invite
+    end
+
+  end
+end

data/spec/mailers/previews/user_plane/verification_mailer_preview.rb

@@ -0,0 +1,16 @@
+module UserPlane
+  # Preview all emails at http://localhost:3000/rails/mailers/user_plane/verification_mailer
+  class VerificationMailerPreview < ActionMailer::Preview
+
+    # Preview this email at http://localhost:3000/rails/mailers/user_plane/verification_mailer/address_verification
+    def address_verification
+      VerificationMailer.address_verification
+    end
+
+    # Preview this email at http://localhost:3000/rails/mailers/user_plane/verification_mailer/password_reset
+    def password_reset
+      VerificationMailer.password_reset
+    end
+
+  end
+end

data/spec/mailers/user_plane/invite_mailer_spec.rb

@@ -0,0 +1,25 @@
+require "spec_helper"
+require 'shared_contexts/user'
+
+
+module UserPlane
+  describe InviteMailer, type: :mailer do
+    include_context 'user'
+
+    describe "invite" do
+
+      subject(:mail) { InviteMailer.invite a_sign_up_invite }
+
+      it "renders the headers" do
+        expect(mail.subject).to eq("Invite")
+        expect(mail.to).to eq([a_sign_up_invite.recipient])
+        expect(mail.from).to eq(["accounts@example.com"])
+      end
+
+      it "links to the sign up page" do
+        expect(mail.body.encoded).to match("http://example.com/invites/#{a_sign_up_invite.code}/redeem")
+      end
+    end
+
+  end
+end

data/spec/mailers/user_plane/verification_mailer_spec.rb

@@ -0,0 +1,52 @@
+require "spec_helper"
+require 'shared_contexts/user'
+
+
+module UserPlane
+  describe VerificationMailer, type: :mailer do
+    include_context 'user'
+
+    describe "address_verification" do
+
+      let :address_verification do
+        a_user.email.unverified_address = Faker::Internet.safe_email
+        a_user.email.save!
+        User::ConfirmEmailAddress.new(verification: a_user.email.address_change_verification)
+      end 
+
+      subject(:mail) { VerificationMailer.address_verification address_verification}
+
+      it "renders the headers" do
+        expect(mail.subject).to eq("Address verification")
+        expect(mail.to).to eq([address_verification.verification.recipient])
+        expect(mail.from).to eq(["accounts@example.com"])
+      end
+
+      it "renders the body" do
+        confirm_link = "http://example.com/account/confirm_email/#{address_verification.code}"
+        expect(mail.body.encoded).to match(confirm_link)
+      end
+    end
+
+    describe "password_reset" do
+
+      let :reset_password do
+        User::ResetPassword.new(verification: a_user.email.reset_password!)
+      end
+
+      subject(:mail) { VerificationMailer.password_reset reset_password}
+
+      it "renders the headers" do
+        expect(mail.subject).to eq("Password reset")
+        expect(mail.to).to eq([reset_password.verification.recipient])
+        expect(mail.from).to eq(["accounts@example.com"])
+      end
+
+      it "renders the body" do
+        reset_link = "http://example.com/account/reset_passwords/#{reset_password.code}/edit"
+        expect(mail.body.encoded).to match(reset_link)
+      end
+    end
+
+  end
+end

data/spec/models/session_manager_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe SessionManager do
+  describe 'logging in' do
+    let(:account) { Fabricate(:user_with_email_identity) }
+    let(:session_manager) { SessionManager.new({}) }
+
+    it 'stores and retrieves a user account through an identity' do
+      session_manager.identity = account.identities.first
+      session_hash = session_manager.instance_variable_get '@session'
+      new_session = SessionManager.new(session_hash)
+      expect(new_session.identity).to eql(account.identities.first)
+      expect(new_session.account).to eql(account)
+    end
+
+
+    it 'goes stale after a few minutes' do
+      session_hash = session_manager.instance_variable_get '@session'
+      new_session_manager = SessionManager.new(session_hash)
+
+      Timecop.travel(90.minutes.from_now) do
+        expect(new_session_manager.identity).to be_nil
+        expect(new_session_manager.account).to be_a(User::Guest)
+      end
+    end
+
+  end
+end

data/spec/models/user/account_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require 'shared_contexts/user'
+
+describe User::Account do
+  include_context 'user'
+
+  it 'must have an identity' do
+    new_user = described_class.new(name: Faker::Internet.user_name)
+    expect(new_user).not_to be_valid
+  end
+
+  it 'has a uuid'
+
+  describe 'an existing user' do
+
+    it 'can forget an identity'
+
+    it 'can be disabled' do
+      login_token = a_user.email.serialize
+
+      a_user.suspensions.create(message: "Dodgy account")
+
+      expect { User::Identity.deserialize!(login_token) }.to raise_error
+    end
+  end
+end

data/spec/models/user/confirm_email_address_spec.rb

@@ -0,0 +1,101 @@
+require 'spec_helper'
+require 'shared_contexts/user'
+
+describe User::ConfirmEmailAddress do
+  include_context 'user'
+  let     (:new_address) { Faker::Internet.safe_email }
+  let!    (:old_address) { a_user.email.address }
+
+  context 'with a valid confirmation token' do
+    let (:address_change_verification_token) do
+      address_verification = a_user.email.verifications.address_change_verification.create(recipient: new_address)
+      address_verification.token
+    end
+
+
+    subject :confirm_email_address do
+      described_class.new(code: address_change_verification_token)
+    end
+
+    it {is_expected.to be_valid}
+
+    it 'confirms the first email address' do
+      expect(a_user.email).to be_verified
+    end
+
+    it 'changes the email address' do
+      confirm_email_address.perform!
+      a_user.reload
+      expect(a_user.email.address).to eql(new_address)
+    end
+
+    context 'once performed' do
+      before do
+        confirm_email_address.perform!
+        a_user.reload
+      end
+
+      it 'spends the verification token' do
+        confirm_email_address.verification.reload
+        expect(confirm_email_address.verification).to be_spent
+      end
+      
+      context 'cannot be redeemed anymore' do        
+        subject :verifying_again do
+          verifying_again = described_class.new(code: address_change_verification_token)
+          verifying_again.valid?
+          verifying_again
+        end
+
+        it {is_expected.not_to be_valid}
+        it {expect(verifying_again.errors.messages[:code].size).to eq(1)}
+      end
+    end
+  end
+
+  context 'without a valid verification token' do
+    subject :confirmation_with_bad_token do
+      email_address_confirmation = described_class.new(code: 'not a valid token')
+      email_address_confirmation.valid?
+      email_address_confirmation
+    end
+
+    it {is_expected.not_to be_valid}
+    it {expect(confirmation_with_bad_token.errors.messages[:code].size).to eq(1)}
+  end
+
+  context 'with an expired verification' do
+    let (:stale_verification_token) do
+      Timecop.travel (2.weeks + 1.day).ago do
+        address_verification = a_user.email.verifications.address_change_verification.create(recipient: new_address)
+        address_verification.token        
+      end
+    end
+
+    subject :confirmation_with_stale_token do
+      email_address_confirmation = described_class.new(code: stale_verification_token)
+      email_address_confirmation.valid?
+      email_address_confirmation
+    end
+
+    it {is_expected.not_to be_valid}
+    it {expect(confirmation_with_stale_token.errors.messages[:code].size).to eq(1)}
+  end
+
+  context 'when the address being verified is taken' do
+    let (:address_change_verification_token) do
+      address_verification = a_user.email.verifications.address_change_verification.create(recipient: new_address)
+      address_verification.token
+    end
+
+    subject :confirm_email_address do
+      conflicting_signup = Fabricate(:user_sign_up, email: new_address)
+      conflicting_signup.perform!
+
+      described_class.new(code: address_change_verification_token)
+    end
+
+    it {is_expected.not_to be_valid}
+  end
+
+end

data/spec/models/user/guest_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+RSpec.describe User::Guest, :type => :model do
+  pending "add some examples to (or delete) #{__FILE__}"
+end

data/spec/models/user/identities/email_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe User::Identities::Email do
+  pending "add some examples to (or delete) #{__FILE__}"
+end