From 2aca87b3ae851d355579c80c8c80da2bc8d064c2 Mon Sep 17 00:00:00 2001 From: Bertocq Date: Thu, 15 Feb 2018 23:05:47 +0100 Subject: [PATCH] Fix verifiable support spec line lenght issues --- spec/support/verifiable.rb | 43 +++++++++++++++++++++++++++----------- 1 file changed, 31 insertions(+), 12 deletions(-) diff --git a/spec/support/verifiable.rb b/spec/support/verifiable.rb index c8d2e5701..f63cbdaae 100644 --- a/spec/support/verifiable.rb +++ b/spec/support/verifiable.rb @@ -45,9 +45,12 @@ shared_examples_for "verifiable" do describe "#unverified" do it "returns unverified users" do user1 = create(:user, verified_at: nil, confirmed_phone: nil) - user2 = create(:user, verified_at: nil, residence_verified_at: nil, confirmed_phone: "123456789") - user3 = create(:user, verified_at: nil, residence_verified_at: Time.current, confirmed_phone: nil) - user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, confirmed_phone: "123456789") + user2 = create(:user, verified_at: nil, residence_verified_at: nil, + confirmed_phone: "123456789") + user3 = create(:user, verified_at: nil, residence_verified_at: Time.current, + confirmed_phone: nil) + user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, + confirmed_phone: "123456789") user5 = create(:user, level_two_verified_at: Time.current) expect(model.unverified).to include(user1) @@ -63,7 +66,8 @@ shared_examples_for "verifiable" do user1 = create(:user, verified_at: nil, confirmed_phone: nil) create(:failed_census_call, user: user1) - user2 = create(:user, verified_at: nil, residence_verified_at: Time.current, unconfirmed_phone: nil) + user2 = create(:user, verified_at: nil, residence_verified_at: Time.current, + unconfirmed_phone: nil) user3 = create(:user, verified_at: nil, confirmed_phone: nil) user4 = create(:user, verified_at: Time.current, residence_verified_at: Time.current, unconfirmed_phone: "123456789", confirmed_phone: "123456789") @@ -84,16 +88,20 @@ shared_examples_for "verifiable" do user = create(:user, residence_verified_at: nil) expect(user.residence_verified?).to eq(false) end + end - it "sms_verified? is true only if confirmed_phone" do + describe '#sms_verified?' do + it "is true only if confirmed_phone" do user = create(:user, confirmed_phone: "123456789") expect(user.sms_verified?).to eq(true) user = create(:user, confirmed_phone: nil) expect(user.sms_verified?).to eq(false) end + end - it "level_two_verified? is true if manually set, or if residence_verified_at and confirmed_phone" do + describe '#level_two_verified?' do + it "is true if manually set, or if residence_verified_at and confirmed_phone" do user = create(:user, level_two_verified_at: Time.current) expect(user.level_two_verified?).to eq(true) @@ -106,32 +114,41 @@ shared_examples_for "verifiable" do user = create(:user, confirmed_phone: "123456789", residence_verified_at: nil) expect(user.level_two_verified?).to eq(false) end + end - it "level_three_verified? is true only if verified_at" do + describe '#level_three_verified?' do + it "is true only if verified_at" do user = create(:user, verified_at: Time.current) expect(user.level_three_verified?).to eq(true) user = create(:user, verified_at: nil) expect(user.level_three_verified?).to eq(false) end + end - it "unverified? is true only if not level_three_verified and not level_two_verified" do + describe '#unverified?' do + it "is true only if not level_three_verified and not level_two_verified" do user = create(:user, verified_at: nil, confirmed_phone: nil) expect(user.unverified?).to eq(true) - user = create(:user, verified_at: Time.current, confirmed_phone: "123456789", residence_verified_at: Time.current) + user = create(:user, verified_at: Time.current, confirmed_phone: "123456789", + residence_verified_at: Time.current) expect(user.unverified?).to eq(false) end + end - it "verification_email_sent? is true only if user has email_verification_token" do + describe '#verification_email_sent?' do + it "is true only if user has email_verification_token" do user = create(:user, email_verification_token: "xxxxxxx") expect(user.verification_email_sent?).to eq(true) user = create(:user, email_verification_token: nil) expect(user.verification_email_sent?).to eq(false) end + end - it "verification_sms_sent? is true only if user has unconfirmed_phone and sms_confirmation_code" do + describe '#verification_sms_sent?' do + it "is true if user has unconfirmed_phone & sms_confirmation_code" do user = create(:user, unconfirmed_phone: "666666666", sms_confirmation_code: "666") expect(user.verification_sms_sent?).to eq(true) @@ -144,8 +161,10 @@ shared_examples_for "verifiable" do user = create(:user, unconfirmed_phone: nil, sms_confirmation_code: nil) expect(user.verification_sms_sent?).to eq(false) end + end - it "verification_letter_sent? is true only if user has letter_requested_at and letter_verification_code" do + describe '#verification_letter_sent?' do + it "is true if user has letter_requested_at & letter_verification_code" do user = create(:user, letter_requested_at: Time.current, letter_verification_code: "666") expect(user.verification_letter_sent?).to eq(true)