summaryrefslogtreecommitdiffstats
path: root/spec
diff options
context:
space:
mode:
authorYamagishi Kazutoshi <ykzts@desire.sh>2018-01-07 23:19:23 +0900
committerEugen Rochko <eugen@zeonfederated.com>2018-01-07 15:19:23 +0100
commit1d92b90be9f494d90a6f79f53be9cca68562867e (patch)
treea4a76a9b01d122d1ffeef2e462f742f2e353fe63 /spec
parentda809f9eec45cdac0fa31ba89df91cd4ce4a5398 (diff)
Fix force_ssl conditional (#6201)
Diffstat (limited to 'spec')
-rw-r--r--spec/controllers/application_controller_spec.rb20
-rw-r--r--spec/rails_helper.rb2
2 files changed, 9 insertions, 13 deletions
diff --git a/spec/controllers/application_controller_spec.rb b/spec/controllers/application_controller_spec.rb
index 6a54a78c566..51c0e3c7004 100644
--- a/spec/controllers/application_controller_spec.rb
+++ b/spec/controllers/application_controller_spec.rb
@@ -47,22 +47,18 @@ describe ApplicationController, type: :controller do
include_examples 'respond_with_error', 422
end
- it "does not force ssl if LOCAL_HTTPS is not 'true'" do
+ it "does not force ssl if Rails.env.production? is not 'true'" do
routes.draw { get 'success' => 'anonymous#success' }
- ClimateControl.modify LOCAL_HTTPS: '' do
- allow(Rails.env).to receive(:production?).and_return(true)
- get 'success'
- expect(response).to have_http_status(:success)
- end
+ allow(Rails.env).to receive(:production?).and_return(false)
+ get 'success'
+ expect(response).to have_http_status(:success)
end
- it "forces ssl if LOCAL_HTTPS is 'true'" do
+ it "forces ssl if Rails.env.production? is 'true'" do
routes.draw { get 'success' => 'anonymous#success' }
- ClimateControl.modify LOCAL_HTTPS: 'true' do
- allow(Rails.env).to receive(:production?).and_return(true)
- get 'success'
- expect(response).to redirect_to('https://test.host/success')
- end
+ allow(Rails.env).to receive(:production?).and_return(true)
+ get 'success'
+ expect(response).to redirect_to('https://test.host/success')
end
describe 'helper_method :current_account' do
diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
index 67c6b920540..4f7399505c4 100644
--- a/spec/rails_helper.rb
+++ b/spec/rails_helper.rb
@@ -46,7 +46,7 @@ RSpec.configure do |config|
config.include ActiveSupport::Testing::TimeHelpers
config.before :each, type: :feature do
- https = Rails.env.production? || ENV['LOCAL_HTTPS'] == 'true'
+ https = ENV['LOCAL_HTTPS'] == 'true'
Capybara.app_host = "http#{https ? 's' : ''}://#{ENV.fetch('LOCAL_DOMAIN')}"
end