diff options
author | Toon Claes <toon@gitlab.com> | 2017-05-09 20:54:57 +0200 |
---|---|---|
committer | Toon Claes <toon@gitlab.com> | 2017-05-09 20:58:15 +0200 |
commit | ab7c90331f8e77ef6c8c1d77ce817aaf066e7842 (patch) | |
tree | d6b23a984a914b9a4179e7c5277eafe0494e83d3 /spec/requests/openid_connect_spec.rb | |
parent | 561ec48cfae5dd453646cba40f3e5f1f7b1147ea (diff) | |
download | gitlab-ce-ab7c90331f8e77ef6c8c1d77ce817aaf066e7842.tar.gz |
Add :redis keyword to some specs clear state of trackable attributes
The specs that rely on a correct value of the trackable attributes, should
include the `:redis` keyword in the spec to ensure the state is reset between
various specs.
The trackable attributes being:
- sign_in_count : Increased every time a sign in is made (by form, openid, oauth)
- current_sign_in_at : A timestamp updated when the user signs in
- last_sign_in_at : Holds the timestamp of the previous sign in
- current_sign_in_ip : The remote ip updated when the user sign in
- last_sign_in_ip : Holds the remote ip of the previous sign in
The limiting of writing trackable attributes was introduced in
gitlab-org/gitlab-ce!11053.
Diffstat (limited to 'spec/requests/openid_connect_spec.rb')
-rw-r--r-- | spec/requests/openid_connect_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/requests/openid_connect_spec.rb b/spec/requests/openid_connect_spec.rb index 132c983fb0d..eb03884b8dc 100644 --- a/spec/requests/openid_connect_spec.rb +++ b/spec/requests/openid_connect_spec.rb @@ -98,7 +98,7 @@ describe 'OpenID Connect requests' do expect(@payload['sub']).to eq hashed_subject end - it 'includes the time of the last authentication' do + it 'includes the time of the last authentication', :redis do expect(@payload['auth_time']).to eq user.current_sign_in_at.to_i end |