From 43496a6cf1bfc650a60e3d6c4dfe3f0256d7cbcc Mon Sep 17 00:00:00 2001
From: Josh <josha5@outlook.com>
Date: Fri, 7 Jul 2017 10:23:19 -0400
Subject: [PATCH] change sessions#new test to allow redirect

---
 app/controllers/sessions_controller.rb       | 2 +-
 test/controllers/sessions_controller_test.rb | 7 ++++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb
index 3c723844..1f649ce2 100644
--- a/app/controllers/sessions_controller.rb
+++ b/app/controllers/sessions_controller.rb
@@ -16,7 +16,7 @@
 
 class SessionsController < ApplicationController
 
-  skip_before_filter :verify_authenticity_token
+  skip_before_action :verify_authenticity_token
 
   def new
     # If LDAP is enabled, just route to it instead.
diff --git a/test/controllers/sessions_controller_test.rb b/test/controllers/sessions_controller_test.rb
index 544c2b77..0bfa08e4 100644
--- a/test/controllers/sessions_controller_test.rb
+++ b/test/controllers/sessions_controller_test.rb
@@ -8,7 +8,12 @@ class SessionsControllerTest < ActionController::TestCase
 
   test "should get new" do
     get :new
-    assert_response :success
+    # We redirect directly to LDAP if configured.
+    if ENV['LDAP_SERVER'].present?
+      assert_response :redirect
+    else
+      assert_response :success
+    end
   end
 
   test "should redirect to home on auth failture" do