added visibility tests

app/views/teachers/_form.html.erb

@@ -214,8 +214,8 @@ which can result in inexplicable, and sometimes undefined behavior -->
     debugger;
     <%# For TEALS volunteer, we also ask for the name of their host teacher. %>
     if (status_val == 3) {
-      $("#teacher-more-info-reminder").text("Please tell us why you need access, and include the name of your host teacher here.");
       $('#upload_file_field').hide();
+      $("#teacher-more-info-reminder").text("Please tell us why you need access, and include the name of your host teacher here.");
     } else {
       $("#teacher-more-info-reminder").text("Please tell us why you need access.");
       <%# If homeschool, then show upload file field %>

features/form_submission.feature

@@ -230,6 +230,15 @@ Feature: submit a form as a teacher
     Then I should not see "Tags"
     And I should not see "NCES ID"
 
+  Scenario: Upload file field visibility based on status option
+    Given I am on the BJC home page
+    And I set my status as "I am a TEALS volunteer, and am teaching the BJC curriculum."
+    Then the upload file field should be hidden
+    When I set my status as "I am teaching BJC as an AP CS Principles course."
+    Then the upload file field should be hidden
+    When I set my status as "I am teaching homeschool with the BJC curriculum."
+    Then the upload file field should be visible
+
   Scenario: Teacher updates information and two emails are sent
     Given the following schools exist:
       |       name      |     country     |     city     |  state  |            website            |  grade_level  |  school_type  |

features/step_definitions/teacher_steps.rb

@@ -123,3 +123,11 @@
     expect(Teacher.find_by(teacher_params).blank?).to be true
   end
 end
+
+Then("the upload file field should be hidden") do
+  expect(page).to have_selector('#upload_file_field', visible: :hidden)
+end
+
+Then("the upload file field should be visible") do
+  expect(page).to have_selector('#upload_file_field', visible: :visible)
+end

spec/fixtures/test_file.txt

@@ -0,0 +1,2 @@
+This file only exists for the purposes of testing that 
+attaching files works

spec/fixtures/test_file2.txt

@@ -0,0 +1,2 @@
+This file, likewise, only exists for the purposes of testing that
+attaching files works