medivo 0.2.2 → 0.2.3

data/spec/models/lab_test_order_spec.rb

@@ -9,13 +9,13 @@ describe LabTestOrder do
   end
 
   it "#pdf_requisition" do
-    stub_order_requisition(@requisition_id, medivo_fixture_path("lc_order_with_requisition.xml"))
+    stub_order_requisition(@requisition_id, medivo_xml_fixture_path("lc_order_with_requisition.xml"))
     @lab_order.pdf_requisition.read.size.should > 30000
     @lab_order.pdf_requisition.close
   end
 
   it "#pdf_result" do
-    stub_order_result(@requisition_id, medivo_fixture_path("lc_order_with_normal_results.xml"))
+    stub_order_result(@requisition_id, medivo_xml_fixture_path("lc_order_with_normal_results.xml"))
     @lab_order.pdf_result.read.size.should > 30000
     show_me_the_pdf(@lab_order.pdf_result.pdf, '/tmp/test4.pdf')
     @lab_order.pdf_result.close

data/spec/models/orders_spec.rb

@@ -12,7 +12,7 @@ describe Medivo::Order do
   it "requires proper gender" do
     order = Medivo::Order.new({:gender=>"dude"})
     order.valid?
-    order.errors.messages[:gender].should include "is not included in the list"
+    order.errors.messages[:gender].should include "gender field must be Male or Female"
   end
 
   it "requires proper dob" do
@@ -74,6 +74,39 @@ describe Medivo::Order do
     order.valid?.should == true
   end
 
+  it "save failure returning status code 422 sets an error message instead of throwing exception" do
+    stub_request(:post, "http://test:test@test.medivo.com/customers.xml").
+            to_return(:body => '<errors><error>Invalid PSC location</error></errors>',
+                      :status => 422)
+    order = Medivo::Order.new(
+            :first_name=>'dan',
+            :last_name=>'dude',
+            :email => "dude@dude.com",
+            :address=> "my house",
+            :city=> "dudeville",
+            :state=> "CA",
+            :zip=> "90210",
+            :dob=> "19801201",
+            :home_phone=> "4155551212",
+            :psc=> 11, # let's pretend this is invalid psc
+            :test_types=> "2020",
+            :account_number => "423",
+            :gender => "Male",
+            :dob => "19801201")
+
+    order.save.should == false
+    order.valid?.should == false
+    order.errors.messages.keys.should include :base
+    order.errors.messages[:base].should == ["Invalid PSC location"]
+
+    # fixing the problem .. it now saves and is valid
+    stub_request(:post, "http://test:test@test.medivo.com/customers.xml").
+            to_return(:status => 200)
+
+    order.save.should == true
+    order.valid?.should == true
+  end
+
 
   it "creates valid order" do
     order = Medivo::Order.new(
@@ -121,7 +154,7 @@ describe Medivo::Order do
     before(:each) do
       @requisition_id = 170420
       order = Medivo::Order.new(:requisition_number=> @requisition_id)
-      stub_order_requisition(@requisition_id, medivo_fixture_path("lc_order_with_requisition.xml"))
+      stub_order_requisition(@requisition_id, medivo_xml_fixture_path("lc_order_with_requisition.xml"))
     end
 
     let(:order) { Medivo::Order.find_with_requisition(@requisition_id) }
@@ -153,7 +186,7 @@ describe Medivo::Order do
     before(:each) do
       @requisition_id = 170853
       order = Medivo::Order.new(:requisition_number=>@requisition_id)
-      stub_order_result(@requisition_id, medivo_fixture_path("lc_order_with_positive_results.xml"))
+      stub_order_result(@requisition_id, medivo_xml_fixture_path("lc_order_with_positive_results.xml"))
     end
 
     let(:order) { Medivo::Order.find_with_results(@requisition_id) }
@@ -202,6 +235,7 @@ end
 
 
 describe "creating orders" do
+
   it "UHC Commercial insurance order" do
     insurance_info = Medivo::PrimaryInsurance.new(
             :relationship=>1,
@@ -231,7 +265,7 @@ describe "creating orders" do
 
 
     mock.proxy(order).to_xml({}) do |value|
-      value.strip_space.should == File.read("#{ENGINE_PATH}/spec/fixtures/valid_uhc_order.xml").strip_space
+      value.strip_space.should == File.read("#{ENGINE_PATH}/spec/fixtures/xml/valid_uhc_order.xml").strip_space
       value
     end
 

data/spec/support/helper.rb

@@ -39,6 +39,18 @@ module Medivo::Spec
     "#{FIXTURE_PATH}/#{file_name}"
   end
 
+  def medivo_pdf_fixture_path(file_name)
+    "#{FIXTURE_PATH}/pdfs/#{file_name}"
+  end
+
+  def medivo_img_fixture_path(file_name)
+    "#{FIXTURE_PATH}/images/#{file_name}"
+  end
+
+  def medivo_xml_fixture_path(file_name)
+    "#{FIXTURE_PATH}/xml/#{file_name}"
+  end
+
   ##
   # requisition_id  Number  id of the Medivo::Order you are stubbing
   # file_path       String  points to the path to xml file with order info which contains

data/spec/support/sample_data.rb

@@ -35,7 +35,7 @@ class LabTestOrder
       :order_number => requisition_id
     }
 
-    cover_letter_file_path = "#{FIXTURE_PATH}/uhc_requisition_cover_letter.pdf"
+    cover_letter_file_path = "#{FIXTURE_PATH}/pdfs/uhc_requisition_cover_letter.pdf"
 
     @pdf_requisition = Medivo::PdfGroup.create do
       variable_fields(cover_letter_file_path, cover_letter_fields)
@@ -55,8 +55,8 @@ class LabTestOrder
       :order_number => requisition_id
     }
 
-    cover_letter_file_path = "#{FIXTURE_PATH}/uhc_result_cover_letter.pdf"
-    faq_path =  "#{FIXTURE_PATH}/hepc_result_faq.pdf"
+    cover_letter_file_path = "#{FIXTURE_PATH}/pdfs/uhc_result_cover_letter.pdf"
+    faq_path =  "#{FIXTURE_PATH}/pdfs/hepc_result_faq.pdf"
 
     @pdf_result = Medivo::PdfGroup.create do
       variable_fields(cover_letter_file_path, cover_letter_fields)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: medivo
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
   prerelease: 
 platform: ruby
 authors:
@@ -14,7 +14,7 @@ default_executable:
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
-  requirement: &2161036180 !ruby/object:Gem::Requirement
+  requirement: &2153066860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -22,10 +22,10 @@ dependencies:
         version: 3.1.0
   type: :runtime
   prerelease: false
-  version_requirements: *2161036180
+  version_requirements: *2153066860
 - !ruby/object:Gem::Dependency
   name: jquery-rails
-  requirement: &2161035760 !ruby/object:Gem::Requirement
+  requirement: &2153066420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -33,10 +33,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161035760
+  version_requirements: *2153066420
 - !ruby/object:Gem::Dependency
   name: coffee-script
-  requirement: &2161035300 !ruby/object:Gem::Requirement
+  requirement: &2153065960 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -44,10 +44,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161035300
+  version_requirements: *2153065960
 - !ruby/object:Gem::Dependency
   name: haml-rails
-  requirement: &2161034880 !ruby/object:Gem::Requirement
+  requirement: &2153065520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -55,10 +55,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161034880
+  version_requirements: *2153065520
 - !ruby/object:Gem::Dependency
   name: geocoder
-  requirement: &2161034380 !ruby/object:Gem::Requirement
+  requirement: &2153065100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -66,10 +66,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161034380
+  version_requirements: *2153065100
 - !ruby/object:Gem::Dependency
   name: rest-client
-  requirement: &2161033960 !ruby/object:Gem::Requirement
+  requirement: &2153064660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -77,10 +77,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161033960
+  version_requirements: *2153064660
 - !ruby/object:Gem::Dependency
   name: pdfkit
-  requirement: &2161033540 !ruby/object:Gem::Requirement
+  requirement: &2153064240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -88,10 +88,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161033540
+  version_requirements: *2153064240
 - !ruby/object:Gem::Dependency
   name: prawn
-  requirement: &2161033100 !ruby/object:Gem::Requirement
+  requirement: &2153063740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -99,7 +99,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2161033100
+  version_requirements: *2153063740
 description: Use the medivo platform to find lab locations, make labcorp appointments,
   place lab orders
 email:
@@ -263,15 +263,16 @@ files:
 - spec/dummy/tmp/cache/assets/E32/FC0/sprockets%2Fe55fb2eef11567fe04fdaf7e73af5ba4
 - spec/dummy/tmp/cache/assets/E59/D50/sprockets%2Fbe1cc4d9b0efb617e58baa16dd1dee58
 - spec/dummy/tmp/image_path_fb5f3a59200b_1320622886.pdf
-- spec/fixtures/hepc_negative_results.pdf
-- spec/fixtures/hepc_result_faq.pdf
-- spec/fixtures/lc_order_with_normal_results.xml
-- spec/fixtures/lc_order_with_positive_results.xml
-- spec/fixtures/lc_order_with_requisition.xml
-- spec/fixtures/Southeast.jpg
-- spec/fixtures/uhc_requisition_cover_letter.pdf
-- spec/fixtures/uhc_result_cover_letter.pdf
-- spec/fixtures/valid_uhc_order.xml
+- spec/dummy/tmp/pids/server.pid
+- spec/fixtures/images/southeast.jpg
+- spec/fixtures/pdfs/hepc_negative_results.pdf
+- spec/fixtures/pdfs/hepc_result_faq.pdf
+- spec/fixtures/pdfs/uhc_requisition_cover_letter.pdf
+- spec/fixtures/pdfs/uhc_result_cover_letter.pdf
+- spec/fixtures/xml/lc_order_with_normal_results.xml
+- spec/fixtures/xml/lc_order_with_positive_results.xml
+- spec/fixtures/xml/lc_order_with_requisition.xml
+- spec/fixtures/xml/valid_uhc_order.xml
 - spec/lib/fdf_generator_spec.rb
 - spec/lib/pdf_generator_spec.rb
 - spec/lib/pdf_group_spec.rb
@@ -300,7 +301,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2876077184686065658
+      hash: 56929817900109279
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -309,7 +310,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2876077184686065658
+      hash: 56929817900109279
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.6.2
@@ -412,15 +413,16 @@ test_files:
 - spec/dummy/tmp/cache/assets/E32/FC0/sprockets%2Fe55fb2eef11567fe04fdaf7e73af5ba4
 - spec/dummy/tmp/cache/assets/E59/D50/sprockets%2Fbe1cc4d9b0efb617e58baa16dd1dee58
 - spec/dummy/tmp/image_path_fb5f3a59200b_1320622886.pdf
-- spec/fixtures/hepc_negative_results.pdf
-- spec/fixtures/hepc_result_faq.pdf
-- spec/fixtures/lc_order_with_normal_results.xml
-- spec/fixtures/lc_order_with_positive_results.xml
-- spec/fixtures/lc_order_with_requisition.xml
-- spec/fixtures/Southeast.jpg
-- spec/fixtures/uhc_requisition_cover_letter.pdf
-- spec/fixtures/uhc_result_cover_letter.pdf
-- spec/fixtures/valid_uhc_order.xml
+- spec/dummy/tmp/pids/server.pid
+- spec/fixtures/images/southeast.jpg
+- spec/fixtures/pdfs/hepc_negative_results.pdf
+- spec/fixtures/pdfs/hepc_result_faq.pdf
+- spec/fixtures/pdfs/uhc_requisition_cover_letter.pdf
+- spec/fixtures/pdfs/uhc_result_cover_letter.pdf
+- spec/fixtures/xml/lc_order_with_normal_results.xml
+- spec/fixtures/xml/lc_order_with_positive_results.xml
+- spec/fixtures/xml/lc_order_with_requisition.xml
+- spec/fixtures/xml/valid_uhc_order.xml
 - spec/lib/fdf_generator_spec.rb
 - spec/lib/pdf_generator_spec.rb
 - spec/lib/pdf_group_spec.rb