skip sb3 component import for non scratch projects

Author: rammodhvadia

lib/project_importer.rb

@@ -52,6 +52,11 @@ def create_components
     return if project.scratch_project?
 
     components.each do |component|
+      # .sb3 files are only ever imported as a ScratchComponent (see
+      # create_scratch_component); they carry an :io/:file_path key that is not a
+      # Component attribute, so skip them here to avoid building invalid rows.
+      next if component[:extension] == 'sb3'
+
       project_component = Component.new(**component)
       project.components << project_component
     end

spec/lib/project_importer_spec.rb

@@ -115,6 +115,31 @@
     end
   end
 
+  context 'when a non-scratch project contains an sb3 file' do
+    let(:project) { Project.find_by(identifier: importer.identifier, user_id: nil, locale: importer.locale) }
+    let(:importer) do
+      described_class.new(
+        name: 'My amazing project',
+        identifier: 'my-amazing-project',
+        type: Project::Types::PYTHON,
+        locale: 'en',
+        components: [
+          { name: 'main', extension: 'py', content: 'print(\'hello\')', default: true },
+          { name: 'stray', extension: 'sb3', io: StringIO.new('ignored') }
+        ]
+      )
+    end
+
+    it 'does not raise when importing' do
+      expect { importer.import! }.not_to raise_error
+    end
+
+    it 'skips the sb3 file and only creates the standard components' do
+      importer.import!
+      expect(project.components.pluck(:extension)).to eq(['py'])
+    end
+  end
+
   context 'when the project has type code_editor_scratch' do
     let(:scratch_project_file) { Tempfile.new(['test_scratch_project', '.sb3']) }
     let(:parser) { instance_double(Sb3Parser, parse: parser_result) }