Allow update to project attributes without specifying components (#104)

# What's Changed?

- Added ability for a project to be updated without specifying its
components in the request

closes #103

app/controllers/api/projects_controller.rb

@@ -56,6 +56,7 @@ def load_projects
 
     def project_params
       params.fetch(:project, {}).permit(
+        :identifier,
         :name,
         :project_type,
         {

lib/concepts/project/operations/update.rb

@@ -26,6 +26,8 @@ def setup_response(project)
       end
 
       def setup_deletions(response, update_hash)
+        return if update_hash[:components].nil?
+
         existing_component_ids = response[:project].components.pluck(:id)
         updated_component_ids = update_hash[:components].pluck(:id)
         response[:component_ids_to_delete] = existing_component_ids - updated_component_ids
@@ -47,18 +49,22 @@ def update_project_attributes(response, update_hash)
       end
 
       def update_component_attributes(response, update_hash)
-        return if response.failure?
+        return if response.failure? || update_hash[:components].nil?
 
         update_hash[:components].each do |component_params|
           if component_params[:id].present?
-            component = response[:project].components.select { |c| c.id == component_params[:id] }.first
-            component.assign_attributes(component_params)
+            overwrite_component_attributes(response, component_params)
           else
             response[:project].components.build(component_params)
           end
         end
       end
 
+      def overwrite_component_attributes(response, component_params)
+        component = response[:project].components.select { |c| c.id == component_params[:id] }.first
+        component.assign_attributes(component_params)
+      end
+
       def persist_changes(response)
         return if response.failure?
 

spec/concepts/project/update_spec.rb

@@ -57,6 +57,26 @@
         expect(created_component).not_to be_nil
       end
     end
+
+    context 'when a component has been removed' do
+      let(:component_hash) { [default_component_hash] }
+
+      it 'deletes a component' do
+        expect { update }.to change(Component, :count).by(-1)
+      end
+    end
+
+    context 'when no components have been specified' do
+      let(:component_hash) { nil }
+
+      it 'keeps the same number of components' do
+        expect { update }.not_to change(Component, :count)
+      end
+
+      it 'updates project properties' do
+        expect { update }.to change { project.reload.name }.to('updated project name')
+      end
+    end
   end
 
   def component_properties_hash(component)

spec/request/projects/update_spec.rb

@@ -49,6 +49,25 @@
       expect(Project::Update).to have_received(:call)
     end
 
+    context 'when no components specified' do
+      let(:params) { { project: { name: 'updated project name' } } }
+
+      it 'returns success response' do
+        put "/api/projects/#{project.identifier}", params: params
+        expect(response).to have_http_status(:ok)
+      end
+
+      it 'returns json with updated project properties' do
+        put "/api/projects/#{project.identifier}", params: params
+        expect(response.body).to include('updated project name')
+      end
+
+      it 'returns json with previous project components' do
+        put "/api/projects/#{project.identifier}", params: params
+        expect(response.body).to include(project.components.first.attributes[:content].to_s)
+      end
+    end
+
     context 'when update is invalid' do
       let(:params) { { project: { components: [] } } }