Merge pull request #659 from degica/validation-if-name-exits

throw error if heritage name already exits

app/controllers/heritages_controller.rb

@@ -12,6 +12,10 @@ def show
   end
 
   def create
+    if Heritage.find_by(name: params[:name]).present?
+      raise ExceptionHandler::InternalServerError.new("heritage name is already used ")
+    end
+
     @heritage = BuildHeritage.new(permitted_params, district: @district).execute
     @heritage.save_and_deploy!(without_before_deploy: true, description: "Create")
     render json: @heritage

spec/requests/create_heritage_spec.rb

@@ -99,6 +99,15 @@
     context "when version is 1" do
       let(:version) { 1 }
       it_behaves_like "create"
+
+      it "throw error when heritage name is already used" do
+        api_request(:post, "/v1/districts/#{district.name}/heritages", params)
+
+        # create same name heritage
+        api_request(:post, "/v1/districts/#{district.name}/heritages", params)
+        expect(response.status).to eq 500
+        expect(JSON.parse(response.body)["error"]).to eq "heritage name is already used "
+      end
     end
 
     context "when version is 2" do