diff options
author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-08-06 15:16:45 +0200 |
---|---|---|
committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2014-08-06 18:03:01 +0200 |
commit | 1118a6fd342f2b5cfe438b63a829368782ff1919 (patch) | |
tree | 96e7c8d23cbde960a14e70a03ce28ce41ce3b3df | |
parent | 82dc40936a233edee59b4be45e0458883446ca9e (diff) | |
download | gitlab-ce-1118a6fd342f2b5cfe438b63a829368782ff1919.tar.gz |
Add a spec for User#requires_ldap_check?
-rw-r--r-- | spec/models/user_spec.rb | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index a36b57a95de..c4bd80cc96a 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -312,6 +312,29 @@ describe User do end end + describe :requires_ldap_check? do + let(:user) { User.new } + + it 'is false for non-LDAP users' do + user.stub(ldap_user?: false) + expect(user.requires_ldap_check?).to be_false + end + + context 'when the user is an LDAP user' do + before { user.stub(ldap_user?: true) } + + it 'is true when the user has never had an LDAP check before' do + user.last_credential_check_at = nil + expect(user.requires_ldap_check?).to be_true + end + + it 'is true when the last LDAP check happened over 1 hour ago' do + user.last_credential_check_at = 2.hours.ago + expect(user.requires_ldap_check?).to be_true + end + end + end + describe '#full_website_url' do let(:user) { create(:user) } |