Skip to content

Commit

Permalink
Merge remote-tracking branch 'origin/master' into pr/15
Browse files Browse the repository at this point in the history
  • Loading branch information
jsmestad committed Sep 22, 2015
2 parents 2446d06 + 92b6ac6 commit 4888f26
Show file tree
Hide file tree
Showing 6 changed files with 26 additions and 23 deletions.
4 changes: 3 additions & 1 deletion Gemfile
@@ -1,3 +1,5 @@
source :rubygems
source 'https://rubygems.org'


gem 'rake'
gemspec
6 changes: 3 additions & 3 deletions Gemfile.lock
Expand Up @@ -6,7 +6,7 @@ PATH
warden (~> 1.2.1)

GEM
remote: http://rubygems.org/
remote: https://rubygems.org/
specs:
addressable (2.2.8)
bcrypt-ruby (3.0.1)
Expand All @@ -33,7 +33,7 @@ GEM
rack
rack-test (0.6.2)
rack (>= 1.0)
rake (0.9.2.2)
rake (10.4.2)
rspec (1.2.9)
simplecov (0.6.4)
multi_json (~> 1.0)
Expand All @@ -46,7 +46,7 @@ GEM
tilt (1.3.3)
warden (1.2.1)
rack (>= 1.0)
yard (0.8.2.1)
yard (0.8.7.6)

PLATFORMS
ruby
Expand Down
31 changes: 17 additions & 14 deletions Rakefile
@@ -1,21 +1,24 @@
require 'bundler'
Bundler::GemHelper.install_tasks
Bundler.setup :development
require "bundler/gem_tasks"

require 'spec/rake/spectask'
Spec::Rake::SpecTask.new(:spec) do |spec|
spec.libs << 'lib' << 'spec'
spec.spec_files = FileList['spec/**/*_spec.rb']
end
begin
# require 'rspec/core/rake_task'
require 'spec/rake/spectask'

Spec::Rake::SpecTask.new('coverage') do |spec|
spec.libs << 'lib' << 'spec'
spec.pattern = 'spec/**/*_spec.rb'
ENV['COVERAGE'] = "true"
end
Spec::Rake::SpecTask.new('coverage') do |spec|
spec.libs << 'lib' << 'spec'
spec.pattern = 'spec/**/*_spec.rb'
ENV['COVERAGE'] = "true"
end

task :default => :spec
Spec::Rake::SpecTask.new(:spec) do |spec|
spec.libs << 'lib' << 'spec'
spec.spec_files = FileList['spec/**/*_spec.rb']
end

task default: :spec
rescue LoadError
# no rspec available
end
begin
require 'yard'
YARD::Rake::YardocTask.new
Expand Down
3 changes: 1 addition & 2 deletions lib/sinatra_warden/sinatra.rb
Expand Up @@ -118,8 +118,7 @@ def self.registered(app)
app.post '/login/?' do
authenticate
env['x-rack.flash'][:success] = settings.auth_success_message if defined?(Rack::Flash)
redirect settings.auth_use_referrer && session[:return_to] ? session.delete(:return_to) :
settings.auth_success_path
redirect settings.auth_use_referrer && session[:return_to] ? session.delete(:return_to) : settings.auth_success_path
end

app.get '/logout/?' do
Expand Down
1 change: 0 additions & 1 deletion sinatra_warden.gemspec
Expand Up @@ -25,7 +25,6 @@ Gem::Specification.new do |s|
s.add_development_dependency 'yard', '>= 0.5.4'
s.add_development_dependency 'rack-test', '~> 0.6.2'
s.add_development_dependency 'simplecov'
s.add_development_dependency 'rake'

s.add_development_dependency 'do_sqlite3', '~> 0.10.0'
s.add_development_dependency 'dm-core', '~> 1.0'
Expand Down
4 changes: 2 additions & 2 deletions spec/spec_helper.rb
Expand Up @@ -39,11 +39,11 @@ def app_with_referrer
@app ||= define_app TestingLoginWithReferrer
end

private
private

# :which should be a sinatra app
def define_app(which)
Rack::Builder.app do
Rack::Builder.new do
use Rack::Session::Cookie
use Warden::Manager do |manager|
manager.default_strategies :password
Expand Down

0 comments on commit 4888f26

Please sign in to comment.