Merge pull request #6250 from tchak/fix-validation-json

Handle GeoJSON validation errors

app/controllers/champs/carte_controller.rb

@@ -15,18 +15,26 @@ class Champs::CarteController < ApplicationController
 
     if geo_area.nil?
       geo_area = champ.geo_areas.build(source: params_source, properties: {})
-      save_feature!(geo_area, create_params_feature)
-    end
 
-    render json: { feature: geo_area.to_feature }, status: :created
+      if save_feature(geo_area, create_params_feature)
+        render json: { feature: geo_area.to_feature }, status: :created
+      else
+        render json: { errors: geo_area.errors.full_messages }, status: :unprocessable_entity
+      end
+    else
+      render json: { feature: geo_area.to_feature }, status: :ok
+    end
   end
 
   def update
     champ = policy_scope(Champ).find(params[:champ_id])
     geo_area = champ.geo_areas.find(params[:id])
-    save_feature!(geo_area, update_params_feature)
 
-    head :no_content
+    if save_feature(geo_area, update_params_feature)
+      head :no_content
+    else
+      render json: { errors: geo_area.errors.full_messages }, status: :unprocessable_entity
+    end
   end
 
   def destroy
@@ -66,13 +74,13 @@ class Champs::CarteController < ApplicationController
     end
   end
 
-  def save_feature!(geo_area, feature)
+  def save_feature(geo_area, feature)
     if feature[:geometry]
       geo_area.geometry = feature[:geometry]
     end
     if feature[:properties]
       geo_area.properties.merge!(feature[:properties])
     end
-    geo_area.save!
+    geo_area.save
   end
 end

spec/controllers/champs/carte_controller_spec.rb

@@ -41,23 +41,38 @@ describe Champs::CarteController, type: :controller do
         post :create, params: params
       end
 
-      it { expect(response.status).to eq 201 }
+      context 'success' do
+        it { expect(response.status).to eq 201 }
+      end
+
+      context 'error' do
+        let(:feature) { attributes_for(:geo_area, :invalid_right_hand_rule_polygon) }
+        let(:params) do
+          {
+            champ_id: champ.id,
+            feature: feature,
+            source: GeoArea.sources.fetch(:selection_utilisateur)
+          }
+        end
+
+        it { expect(response.status).to eq 422 }
+      end
     end
 
     describe 'PATCH #update' do
+      let(:params) do
+        {
+          champ_id: champ.id,
+          id: geo_area.id,
+          feature: feature
+        }
+      end
+
       before do
         patch :update, params: params
       end
 
       context 'update geometry' do
-        let(:params) do
-          {
-            champ_id: champ.id,
-            id: geo_area.id,
-            feature: feature
-          }
-        end
-
         it { expect(response.status).to eq 204 }
       end
 
@@ -69,19 +84,18 @@ describe Champs::CarteController, type: :controller do
             }
           }
         end
-        let(:params) do
-          {
-            champ_id: champ.id,
-            id: geo_area.id,
-            feature: feature
-          }
-        end
 
         it {
           expect(response.status).to eq 204
           expect(geo_area.reload.description).to eq('un point')
         }
       end
+
+      context 'error' do
+        let(:feature) { attributes_for(:geo_area, :invalid_right_hand_rule_polygon) }
+
+        it { expect(response.status).to eq 422 }
+      end
     end
 
     describe 'DELETE #destroy' do