WIP get patch working

Author: imRohan

lib/intelligent_foods/api_client.rb

@@ -34,8 +34,13 @@ def execute_request(request:, uri:, body: nil,
       end
     end
 
-    def build_request_with_body(uri:, body:)
-      request = Net::HTTP::Post.new(uri)
+    def build_patch_request(uri:, body:)
+      build_request_with_body(uri: uri, body: body,
+                              http_method: Net::HTTP::Patch)
+    end
+
+    def build_request_with_body(uri:, body:, http_method: Net::HTTP::Post)
+      request = http_method.new(uri)
       request.body = body.to_json
       request["content-type"] = "application/json"
       request

lib/intelligent_foods/errors.rb

@@ -8,4 +8,6 @@ class OrderNotCancelledError < ApiError; end
   class OrderNotCreatedError < ApiError; end
 
   class AuthenticationError < ApiError; end
+
+  class OrderNotUpdatedError < ApiError; end
 end

lib/intelligent_foods/resources/order.rb

@@ -45,6 +45,18 @@ def cancel!
       end
     end
 
+    def update!
+      uri = URI("#{IntelligentFoods.base_url}/order/#{id}")
+      request = client.build_patch_request(uri: uri, body: update_request_body)
+      response = client.execute_request(request: request, uri: uri)
+      if response.success?
+        Order::build_from_response(response.data)
+      else
+        mark_as_invalid
+        raise_error(OrderNotUpdatedError, response)
+      end
+    end
+
     def request_body
       @request_body ||= {
         menu_id: menu.id,
@@ -55,6 +67,12 @@ def request_body
       }
     end
 
+    def update_request_body
+      @update_request_body ||= {
+        ship_to: ship_to,
+      }
+    end
+
     def cancelled?
       status.downcase == CANCELLED
     end

spec/intelligent_foods/api_client_spec.rb

@@ -165,6 +165,16 @@
       end
     end
 
+    describe "#build_patch_request" do
+      it "builds a request using the Patch HTTP method" do
+        client = IntelligentFoods::ApiClient.new(id: "id", secret: "secret")
+
+        result = client.build_patch_request(uri: "test.com", body: {})
+
+        expect(result.method).to eq("PATCH")
+      end
+    end
+
     describe "#authenticated?" do
       it "is not authenticated" do
         client = IntelligentFoods::ApiClient.new(id: "id", secret: "secret")

spec/intelligent_foods/resources/order_spec.rb

@@ -142,4 +142,45 @@
       end
     end
   end
+
+  describe "#update!" do
+    it "updates the order" do
+      order = build_stubbed_order
+      body = build_order_response
+      response = build_response(body: body)
+      stub_api_response response: response
+
+      result = order.update!
+
+      expect(result).to be_accepted
+    end
+
+    context "the response code is not 200" do
+      it "raises a OrderNotUpdatedError" do
+        order = build_stubbed_order
+        response = error_response(message: "Order Not Found",
+                                  http_status_code: 404)
+        stub_api_response response: response
+
+        expect {
+          order.update!
+        }.to raise_error(IntelligentFoods::OrderNotUpdatedError)
+      end
+
+      it "marks the order as not valid" do
+        order = build_stubbed_order
+        response = error_response(message: "Order Not Found",
+                                  http_status_code: 404)
+        stub_api_response response: response
+
+        begin
+          order.update!
+        rescue IntelligentFoods::OrderNotUpdatedError
+          # noop
+        end
+
+        expect(order).not_to be_valid
+      end
+    end
+  end
 end

spec/support/helpers/api_helper.rb

@@ -44,6 +44,19 @@ def read_error_response
     parse_json_file(ERROR_API_RESPONSE)
   end
 
+  def build_stubbed_order
+    recipient = build(:recipient)
+    menu = build(:menu, id: "2023-01-01")
+    order_item = build(:order_item)
+    callback_url = "https://api.domain.com/callback"
+    IntelligentFoods::Order.new(menu: menu,
+                                recipient: recipient,
+                                delivery_date: "2023-01-07",
+                                items: [order_item],
+                                external_id: "1337",
+                                callback_url: callback_url)
+  end
+
   def read_menu_api_response
     parse_json_file(MENU_API_RESPONSE)
   end