summaryrefslogtreecommitdiff
path: root/lib/mattermost
diff options
context:
space:
mode:
authorZ.J. van de Weg <git@zjvandeweg.nl>2016-12-15 14:32:50 +0100
committerZ.J. van de Weg <git@zjvandeweg.nl>2016-12-16 12:21:09 +0100
commitdd385c7c3d3046da18c6c251bce25afab1129662 (patch)
tree1664b885ec5f684f23c45e7280c2d992daaa937c /lib/mattermost
parent178638c4b2fa6634dc0aaa0ef13fb5a14eb7021f (diff)
downloadgitlab-ce-dd385c7c3d3046da18c6c251bce25afab1129662.tar.gz
Improve session tests
Diffstat (limited to 'lib/mattermost')
-rw-r--r--lib/mattermost/session.rb (renamed from lib/mattermost/mattermost.rb)8
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/mattermost/mattermost.rb b/lib/mattermost/session.rb
index 84d016bb197..81964666757 100644
--- a/lib/mattermost/mattermost.rb
+++ b/lib/mattermost/session.rb
@@ -13,7 +13,7 @@ module Mattermost
# This class however skips the button click, and also the approval phase to
# speed up the process and keep it without manual action and get a session
# going.
- class Mattermost
+ class Session
include Doorkeeper::Helpers::Controller
include HTTParty
@@ -27,8 +27,10 @@ module Mattermost
def with_session
raise NoSessionError unless create
- yield
+ result = yield
destroy
+
+ result
end
# Next methods are needed for Doorkeeper
@@ -85,7 +87,7 @@ module Mattermost
end
def request_token
- @request_token ||= if @token_uri
+ @request_token ||= begin
response = get(@token_uri, follow_redirects: false)
response.headers['token'] if 200 <= response.code && response.code < 400
end