From 937f46faddb82ae06fe5d1276e422ca63cffc9af Mon Sep 17 00:00:00 2001
From: jazairi <16103405+jazairi@users.noreply.github.com>
Date: Wed, 25 Oct 2023 08:19:13 -0400
Subject: [PATCH] Address code review feedback

---
 app/views/transfer/_welcome_maintenance.html.erb |  2 +-
 app/views/transfer/new.html.erb                  |  2 +-
 test/controllers/transfer_controller_test.rb     | 11 +++++++++++
 3 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/app/views/transfer/_welcome_maintenance.html.erb b/app/views/transfer/_welcome_maintenance.html.erb
index b0e3571e..cc2ab32a 100644
--- a/app/views/transfer/_welcome_maintenance.html.erb
+++ b/app/views/transfer/_welcome_maintenance.html.erb
@@ -1,6 +1,6 @@
 <div class="well maintenance-message">
   <% if ENV.fetch('MAINTENANCE_MESSAGE_TRANSFER', false) %>
-    <h4 class="hd-5"><%= ENV['MAINTENANCE_MESSAGE_TRANSFER'].to_s %></h4>
+    <h4 class="hd-5"><%= ENV['MAINTENANCE_MESSAGE_TRANSFER'].html_safe %></h4>
   <% else %>
     <h4 class="hd-5">The transfer tool is temporarily unavailable while the application is under maintenance. Please
       contact us at <a href="mailto:mit-theses <mit-theses@mit.edu>" target="_blank">mit-theses@mit.edu</a> with any
diff --git a/app/views/transfer/new.html.erb b/app/views/transfer/new.html.erb
index 39c5527d..c45200bf 100644
--- a/app/views/transfer/new.html.erb
+++ b/app/views/transfer/new.html.erb
@@ -180,6 +180,6 @@
 
   <% if Flipflop.enabled?(:maintenance_mode) %>
     <script type="text/javascript">
-      $(':input').prop('disabled', true);
+      $('#new_transfer').hide().find(':input').prop('disabled', true);
     </script>
   <% end %>
diff --git a/test/controllers/transfer_controller_test.rb b/test/controllers/transfer_controller_test.rb
index 2119f0eb..b462b8fa 100644
--- a/test/controllers/transfer_controller_test.rb
+++ b/test/controllers/transfer_controller_test.rb
@@ -465,6 +465,17 @@ def create_transfer_with_file
     assert_select 'input', disabled: true, count: 5
   end
 
+  test 'form is hidden when app is in maintenance mode' do
+    test_strategy = Flipflop::FeatureSet.current.test!
+    test_strategy.switch!(:maintenance_mode, true)
+
+    sign_in users(:transfer_submitter)
+    get '/transfer/new'
+    assert_select 'form', hidden: true, count: 1
+    assert_select 'textarea', hidden: true, count: 1
+    assert_select 'input', hidden: true, count: 5
+  end
+
   test 'maintenance message renders when app is in maintenance mode' do
     test_strategy = Flipflop::FeatureSet.current.test!
     test_strategy.switch!(:maintenance_mode, true)