Update tests

test/factories/job_applications_factory.rb

@@ -1,14 +1,15 @@
 FactoryBot.define do
   factory :job_application do
-    date_applied { Date.new(2023, 1, 1) }
+    claimed_for_unemployment { false }
     company_name { "Example Company" }
+    date_applied { Date.new(2023, 1, 1) }
+    email_address { "[email protected]" }
+    location { "remote" }
     method_of_contact { "email" }
-    position_type { "full_time" }
+    point_of_contact { "John Doe" }
     position_title { "Software Engineer" }
-    claimed_for_unemployment { false }
+    position_type { "full_time" }
     status { "interviewing" }
-    email_address { "[email protected]" }
-    point_of_contact { "John Doe" }
 
     trait :with_website do
       method_of_contact { "internet_job_application" }

test/models/job_application_test.rb

@@ -60,6 +60,12 @@ def setup
     assert_not_nil @job_application.errors[:website_link]
   end
 
+  test "job application validates location inclusion" do
+    assert_raises(ArgumentError) do
+      build(:job_application, location: "invalid_location")
+    end
+  end
+
   test "job application validates status inclusion" do
     assert_raises(ArgumentError) do
       build(:job_application, status: "invalid_status")
@@ -84,6 +90,15 @@ def setup
     assert_equal 1, JobApplication.by_method_of_contact("internet_job_application").count
   end
 
+  test "by_location scope finds job applications by location" do
+    JobApplication.destroy_all
+    create(:job_application, location: "remote")
+    create(:job_application, location: "in_office")
+
+    assert_equal 1, JobApplication.by_location("remote").count
+    assert_equal 1, JobApplication.by_location("in_office").count
+  end
+
   test "by_position_type scope finds job applications by position_type" do
     JobApplication.destroy_all
     create(:job_application, position_type: "full_time")