lk | fixes client date saves and improves test readability

app/controllers/clients_controller.rb

@@ -49,8 +49,8 @@ class ClientsController < ApplicationController
     end
 
     def client_params
-      params["client"]["application_date"] = Date.strptime(params["client"]["application_date"], '%m/%d/%Y') unless params["client"]["application_date"].empty?
-      params["client"]["pickup_date"] = Date.strptime(params["client"]["pickup_date"], '%m/%d/%Y') unless params["client"]["pickup_date"].empty?
+      params["client"]["application_date"] = Date.strptime(params["client"]["application_date"], '%m/%d/%Y') if params["client"]["application_date"].present?
+      params["client"]["pickup_date"] = Date.strptime(params["client"]["pickup_date"], '%m/%d/%Y') if params["client"]["pickup_date"].present?
       params.require(:client).permit(
         :first_name,
         :last_name,

spec/controllers/clients_controller_spec.rb

@@ -19,13 +19,16 @@ describe ClientsController do
   end
 
   describe "PUT #update" do
-    xit "updates a client with an application date" do
-      put :update, id: client.id, client: {application_date: "12/21/2015"}
-      expect(client.reload.application_date.strftime('%m/%d/%Y')).to eq("12/21/2015")
+    it "updates a client with an application date" do
+      date = Time.zone.now.beginning_of_day
+      put :update, id: client.id, client: {application_date: date.strftime('%m/%d/%Y')}
+      expect(client.reload.application_date).to eq(date)
     end
-    xit "updates a client with a pickup date" do
-      put :update, id: client.id, client: {pickup_date: "12/21/2015"}
-      expect(client.reload.pickup_date.strftime('%m/%d/%Y')).to eq("12/21/2015")
+
+    it "updates a client with a pickup date" do
+      date = Time.zone.now.beginning_of_day
+      put :update, id: client.id, client: {pickup_date: date.strftime('%m/%d/%Y')}
+      expect(client.reload.pickup_date).to eq(date)
     end
   end
 end