medivo 0.2.14 → 0.2.15

data/app/models/medivo/appointment.rb

@@ -17,7 +17,7 @@ module Medivo
       #
       # @return Hash      { :times=>['time1', 'time2'] }
       #
-      # @throws exception mainly Restclien::Exception types
+      # @throws exception mainly Restclient::Exception types
       def find(lab_code, date, am_pm='')
         if real_data?
           data = resource.get :params=>{:labcorp_id=>lab_code, :appointment_date=>date}
@@ -72,8 +72,9 @@ module Medivo
 
       def filter_data(hash, am_pm)
         now = Time.now
-        hash['times'].reject! do |time|
-          (!time.match(/#{am_pm}/) or Time.strptime(time, LABCORP_FORMAT) < now)
+        hash['times'].reject! do |time_string|
+          time = Time.strptime(time_string, LABCORP_FORMAT) rescue nil
+          !time or !time_string.match(/#{am_pm}/) or (time < now)
         end
         hash
       end

data/lib/medivo/version.rb

@@ -1,3 +1,3 @@
 module Medivo
-  VERSION = "0.2.14"
+  VERSION = "0.2.15"
 end

data/spec/dummy/log/test.log

@@ -2997,3 +2997,89 @@ Started POST "/medivo/labs/make_appointment" for 127.0.0.1 at 2011-11-15 17:41:5
   Processing by Medivo::LabsController#make_appointment as HTML
   Parameters: {"lab_code"=>"20060", "time"=>"2011-11-01 08:30:00 UTC", "user"=>{"first_name"=>"dude", "last_name"=>nil, "date_of_birth"=>"12/28/1980", "email_address"=>"dudeman@dude.com", "phone_number"=>"4155551212"}}
 Completed 200 OK in 2ms (Views: 0.2ms)
+  Processing by OrdersController#download_requisition as HTML
+Rendered text template (0.0ms)
+Sent data unitedhealthcare_requisition.pdf (76.0ms)
+Completed 200 OK in 989ms (Views: 75.1ms)
+  Processing by OrdersController#download_result as HTML
+Sent data unitedhealthcare_result.pdf (2.4ms)
+Completed 200 OK in 709ms (Views: 1.7ms)
+
+
+Started GET "/medivo/labs?zip_code=90210" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by Medivo::LabsController#index as HTML
+  Parameters: {"zip_code"=>"90210"}
+Completed 200 OK in 0ms (Views: 0.0ms)
+
+
+Started GET "/labs/lab_search?zip_code=90210" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by LabsController#lab_search as HTML
+  Parameters: {"zip_code"=>"90210"}
+Completed 200 OK in 0ms (Views: 0.0ms)
+
+
+Started GET "/assets/application.css" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Compiled application.css  (0ms)  (pid 9135)
+Served asset /application.css - 200 OK (0ms)
+
+
+Started GET "/assets/application.js" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Compiled application.js  (0ms)  (pid 9135)
+Compiled jquery.js  (0ms)  (pid 9135)
+Compiled jquery_ujs.js  (0ms)  (pid 9135)
+Compiled medivo/lab_list/application.js  (0ms)  (pid 9135)
+Compiled medivo/handlebars.js  (0ms)  (pid 9135)
+Compiled medivo/helpers.js  (0ms)  (pid 9135)
+Compiled medivo/lab_list/models.js  (0ms)  (pid 9135)
+Compiled medivo/models.js  (0ms)  (pid 9135)
+Compiled medivo/lab_list/views.js  (0ms)  (pid 9135)
+Compiled medivo/views.js  (0ms)  (pid 9135)
+Compiled medivo/lab_list/handlers.js  (0ms)  (pid 9135)
+Compiled medivo/lab_list/sample/show_labs.js  (0ms)  (pid 9135)
+Compiled medivo/lab_appointment/application.js  (0ms)  (pid 9135)
+Compiled medivo/date.js  (0ms)  (pid 9135)
+Compiled medivo/lab_appointment/models.js  (0ms)  (pid 9135)
+Compiled medivo/lab_appointment/views.js  (0ms)  (pid 9135)
+Compiled medivo/lab_appointment/handlers.js  (0ms)  (pid 9135)
+Compiled medivo/lab_appointment/sample/show_appointments.js  (0ms)  (pid 9135)
+Served asset /application.js - 200 OK (0ms)
+
+
+Started GET "/assets/medivo/ajax_waiting.gif" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Served asset /medivo/ajax_waiting.gif - 200 OK (0ms)
+
+
+Started GET "/assets/medivo/markerA.png" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Served asset /medivo/markerA.png - 200 OK (0ms)
+
+
+Started GET "/assets/medivo/markerB.png" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Served asset /medivo/markerB.png - 200 OK (0ms)
+
+
+Started GET "/assets/medivo/arrow.png" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+Served asset /medivo/arrow.png - 200 OK (0ms)
+
+
+Started GET "/medivo/labs/find_appointment?date=11%2F01%2F2011&lab_code=20060" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by Medivo::LabsController#find_appointment as HTML
+  Parameters: {"date"=>"11/01/2011", "lab_code"=>"20060"}
+Completed 200 OK in 0ms (Views: 0.0ms)
+
+
+Started GET "/medivo/labs/find_appointment?date=11%2F01%2F2011&lab_code=20060" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by Medivo::LabsController#find_appointment as HTML
+  Parameters: {"date"=>"11/01/2011", "lab_code"=>"20060"}
+Completed 200 OK in 0ms (Views: 0.0ms)
+
+
+Started GET "/medivo/labs/find_appointment?date=11%2F01%2F2011&lab_code=20060" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by Medivo::LabsController#find_appointment as HTML
+  Parameters: {"date"=>"11/01/2011", "lab_code"=>"20060"}
+Completed 404 Not Found in 0ms (Views: 0.0ms)
+
+
+Started POST "/medivo/labs/make_appointment" for 127.0.0.1 at 2011-11-04 00:00:00 -0400
+  Processing by Medivo::LabsController#make_appointment as HTML
+  Parameters: {"lab_code"=>"20060", "time"=>"2011-11-01 08:30:00 UTC", "user"=>{"first_name"=>"dude", "last_name"=>nil, "date_of_birth"=>"12/28/1980", "email_address"=>"dudeman@dude.com", "phone_number"=>"4155551212"}}
+Completed 200 OK in 0ms (Views: 0.0ms)

data/spec/models/appointment_spec.rb

@@ -6,8 +6,11 @@ describe Medivo::Appointment do
   describe "finding appointments" do
 
     describe "with valid data" do
-      let(:all_times) { ["11/03/2011|08:30 AM", "11/04/2011|08:30 AM", "11/04/2011|08:30 PM"] }
-      let(:body) { {'times'=> all_times} }
+      before do
+        @all_times = ["11/03/2011|08:30 AM", "11/04/2011|08:30 AM", "11/04/2011|08:30 PM"]
+      end
+
+      let(:body) { {'times'=> @all_times} }
 
       def returned_times(date, am_pm)
         Timecop.freeze(Time.strptime(date, "%m/%d/%Y"))
@@ -18,11 +21,11 @@ describe Medivo::Appointment do
       end
 
       it "returns times" do
-        returned_times('11/01/2011', '').should == all_times
+        returned_times('11/01/2011', '').should == @all_times
       end
 
       it "filters out past date times" do
-        returned_times('11/04/2011', '').should == all_times.slice(1, 2)
+        returned_times('11/04/2011', '').should == @all_times.slice(1, 2)
       end
 
       it "filters out past or pm times" do
@@ -32,6 +35,11 @@ describe Medivo::Appointment do
       it "filters out past or am times" do
         returned_times('11/04/2011', 'PM').should == ["11/04/2011|08:30 PM"]
       end
+
+      it "filters out invalid time strings" do
+        @all_times = ["11/02011|08:30 AM", "11/04/2011|11:30 AM"]
+        returned_times('11/04/2011', '').should == ["11/04/2011|11:30 AM"]
+      end
     end
 
     describe "with invalid request returns error message" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: medivo
 version: !ruby/object:Gem::Version
-  version: 0.2.14
+  version: 0.2.15
   prerelease: 
 platform: ruby
 authors:
@@ -9,12 +9,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-11-15 00:00:00.000000000 -05:00
+date: 2011-11-16 00:00:00.000000000 -05:00
 default_executable: 
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &2172862740 !ruby/object:Gem::Requirement
+  requirement: &2165114340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -22,10 +22,10 @@ dependencies:
         version: 3.1.0
   type: :runtime
   prerelease: false
-  version_requirements: *2172862740
+  version_requirements: *2165114340
 - !ruby/object:Gem::Dependency
   name: jquery-rails
-  requirement: &2172858980 !ruby/object:Gem::Requirement
+  requirement: &2165113900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +33,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172858980
+  version_requirements: *2165113900
 - !ruby/object:Gem::Dependency
   name: coffee-script
-  requirement: &2172858520 !ruby/object:Gem::Requirement
+  requirement: &2165113440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +44,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172858520
+  version_requirements: *2165113440
 - !ruby/object:Gem::Dependency
   name: haml-rails
-  requirement: &2172858100 !ruby/object:Gem::Requirement
+  requirement: &2165113020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,10 +55,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172858100
+  version_requirements: *2165113020
 - !ruby/object:Gem::Dependency
   name: geocoder
-  requirement: &2172857680 !ruby/object:Gem::Requirement
+  requirement: &2165112600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -66,10 +66,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172857680
+  version_requirements: *2165112600
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &2172857260 !ruby/object:Gem::Requirement
+  requirement: &2165112180 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -77,10 +77,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172857260
+  version_requirements: *2165112180
 - !ruby/object:Gem::Dependency
   name: pdfkit
-  requirement: &2172856840 !ruby/object:Gem::Requirement
+  requirement: &2165111740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -88,10 +88,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172856840
+  version_requirements: *2165111740
 - !ruby/object:Gem::Dependency
   name: prawn
-  requirement: &2172856420 !ruby/object:Gem::Requirement
+  requirement: &2165111320 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -99,7 +99,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2172856420
+  version_requirements: *2165111320
 description: Use the medivo platform to find lab locations, make labcorp appointments,
   place lab orders
 email:
@@ -312,7 +312,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -535201164639526646
+      hash: 883431128477834155
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -321,7 +321,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -535201164639526646
+      hash: 883431128477834155
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.6.2