Teach CreateRemix how to save Scratch remixes

Keep the existing non-Scratch remix flow intact and add a Scratch-specific branch that stores scratch_component content for remixed projects.

The unit coverage now exercises the Scratch remix path directly.

Author: abcampo-iry

lib/concepts/project/operations/create_remix.rb

@@ -23,7 +23,11 @@ def validate_params(response, params, user_id, original_project, remix_origin)
       end
 
       def remix_project(response, params, user_id, original_project, remix_origin)
-        response[:project] = create_remix(original_project, params, user_id, remix_origin)
+        response[:project] = if scratch_project?(original_project)
+                               create_scratch_remix(original_project, params, user_id, remix_origin)
+                             else
+                               create_remix(original_project, params, user_id, remix_origin)
+                             end
         response[:project].save!
         response
       end
@@ -50,17 +54,29 @@ def create_remix(original_project, params, user_id, remix_origin)
         remix
       end
 
+      def create_scratch_remix(original_project, params, user_id, remix_origin)
+        remix = format_project(original_project, params, user_id, remix_origin)
+        scratch_component = params.fetch(:scratch_component)
+        remix.build_scratch_component(content: scratch_component[:content] || scratch_component['content'])
+
+        remix
+      end
+
       def format_project(original_project, params, user_id, remix_origin)
         original_project.dup.tap do |proj|
           proj.identifier = PhraseIdentifier.generate
           proj.locale = nil
-          proj.name = params[:name]
+          proj.name = params[:name] || original_project.name
           proj.user_id = user_id
           proj.remixed_from_id = original_project.id
           proj.remix_origin = remix_origin
           proj.lesson_id = nil # Only the original can have a lesson id
         end
       end
+
+      def scratch_project?(project)
+        project.project_type == Project::Types::CODE_EDITOR_SCRATCH
+      end
     end
   end
 end

spec/concepts/project/create_remix_spec.rb

@@ -129,6 +129,49 @@
       end
     end
 
+    context 'when the original project has Scratch content' do
+      let!(:original_project) do
+        create(:project, project_type: Project::Types::CODE_EDITOR_SCRATCH, locale: nil)
+      end
+      let(:original_scratch_project) do
+        {
+          meta: { semver: '3.0.0' },
+          targets: ['original target'],
+          monitors: [],
+          extensions: []
+        }
+      end
+
+      before do
+        create(:scratch_component, project: original_project, content: original_scratch_project)
+      end
+
+      context 'when new Scratch content is provided' do
+        let(:remixed_scratch_project) do
+          {
+            meta: { semver: '3.0.0' },
+            targets: ['remixed target'],
+            monitors: [],
+            extensions: ['pen']
+          }
+        end
+        let(:remix_params) do
+          {
+            name: 'My remixed project',
+            identifier: original_project.identifier,
+            scratch_component: {
+              content: remixed_scratch_project
+            }
+          }
+        end
+
+        it 'uses the supplied Scratch content' do
+          expect(create_remix[:project].scratch_component.content.to_h)
+            .to eq(remixed_scratch_project.deep_stringify_keys)
+        end
+      end
+    end
+
     context 'when user_id is not present' do
       let(:user_id) { nil }
       let(:params) { { project_id: original_project.identifier } }