Add specs for shipments

Author: JustShah

api/spec/requests/spree/api/shipments_spec.rb

@@ -92,6 +92,23 @@ module Spree::Api
         expect(json_response['stock_location_name']).to eq(stock_location.name)
       end
 
+      it "can update a shipment and it's metadata" do
+        params = {
+          shipment: {
+            stock_location_id: stock_location.to_param,
+            admin_metadata: { 'delivery_type' => 'express' },
+            customer_metadata: { 'timing' => 'standard' }
+          }
+        }
+
+        put(spree.api_shipment_path(shipment), params:)
+        
+        expect(response.status).to eq(200)
+        expect(json_response['stock_location_name']).to eq(stock_location.name)
+        expect(json_response["admin_metadata"]).to eq({ 'delivery_type' => 'express' })
+        expect(json_response["customer_metadata"]).to eq({ 'timing' => 'standard' })
+      end
+
       it "can make a shipment ready" do
         allow_any_instance_of(Spree::Order).to receive_messages(paid?: true, complete?: true)
         put spree.ready_api_shipment_path(shipment)

core/spec/models/spree/shipment_spec.rb

@@ -872,4 +872,24 @@
 
     it { is_expected.to include carton }
   end
+
+  describe "metadata fields" do
+    it "responds to customer_metadata" do
+      expect(shipment).to respond_to(:customer_metadata)
+    end
+
+    it "responds to admin_metadata" do
+      expect(shipment).to respond_to(:admin_metadata)
+    end
+
+    it "can store data in customer_metadata" do
+      shipment.customer_metadata = { "tracking_info" => "UPS123456789" }
+      expect(shipment.customer_metadata["tracking_info"]).to eq("UPS123456789")
+    end
+
+    it "can store data in admin_metadata" do
+      shipment.admin_metadata = { "internal_note" => "Handle with care" }
+      expect(shipment.admin_metadata["internal_note"]).to eq("Handle with care")
+    end
+  end
 end