summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJacob Vosmaer <contact@jacobvosmaer.nl>2014-08-06 15:16:45 +0200
committerJacob Vosmaer <contact@jacobvosmaer.nl>2014-08-06 18:03:01 +0200
commit1118a6fd342f2b5cfe438b63a829368782ff1919 (patch)
tree96e7c8d23cbde960a14e70a03ce28ce41ce3b3df
parent82dc40936a233edee59b4be45e0458883446ca9e (diff)
downloadgitlab-ce-1118a6fd342f2b5cfe438b63a829368782ff1919.tar.gz
Add a spec for User#requires_ldap_check?
-rw-r--r--spec/models/user_spec.rb23
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) }