act_as_importable 0.0.8 → 0.0.10

data/.gitignore

@@ -16,3 +16,4 @@ test/tmp
 test/version_tmp
 tmp
 act_as_importable.db
+.idea

data/README.md

@@ -56,6 +56,17 @@ User.import_csv_file('/path/to/file.csv', :uid => :email)
 This will find an existing record with a matching email address and update their name.
 If no record exists it will create a new one.
 
+## Other options
+
+* uid (default: 'uid')
+* except (default: nil)
+* only (default: nil)
+* default_values (default: nil)
+* delete_missing_records (default: false)
+* existing_record_scope (default: Model.all)
+
+
+
 
 ## Test
 

data/lib/act_as_importable/importer.rb

@@ -15,6 +15,24 @@ module ActAsImportable
       data.map do |row|
         import_record(row)
       end
+      delete_missing_records if options[:delete_missing_records]
+    end
+
+    def existing_record_scope
+      options[:existing_record_scope] || model_class.all
+    end
+
+    def existing_record_ids
+      existing_record_scope.map(&:id)
+    end
+
+    def record_ids_to_delete
+      imported_ids = successful_imports.map(&:id)
+      existing_record_ids.reject { |id| imported_ids.include? id }
+    end
+
+    def delete_missing_records
+      model_class.delete(record_ids_to_delete)
     end
 
     def import_record(row)

data/lib/act_as_importable/version.rb

@@ -1,3 +1,3 @@
 module ActAsImportable
-  VERSION = "0.0.8"
+  VERSION = "0.0.10"
 end

data/spec/act_as_importable/importer_spec.rb

@@ -1,3 +1,5 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
 describe ActAsImportable::Importer do
 
   before(:each) do
@@ -35,4 +37,65 @@ describe ActAsImportable::Importer do
     end
   end
 
+  describe "delete_missing_records option" do
+    let(:importer) { ActAsImportable::Importer.new(:uid => :name, :model_class => Item, :delete_missing_records => true) }
+    let(:beer_category) { Category.create!(:name => 'Beer') }
+    let(:wine_category) { Category.create!(:name => 'Wine') }
+
+    before :each do
+      @beer1 = Item.create!(:name => 'Beer 1', :price => 3.0, :category => beer_category)
+      @beer2 = Item.create!(:name => 'Beer 2', :price => 3.5, :category => beer_category)
+      @wine1 = Item.create!(:name => 'Wine 1', :price => 16.5, :category => wine_category)
+    end
+
+    it "should call delete_missing_records when the option is true" do
+      importer.options[:delete_missing_records] = true
+      importer.should_receive(:delete_missing_records).once
+      importer.import_data([{:name => 'Beer 1', :price => 3.0}])
+    end
+
+    it "should not call delete_missing_records when the option is false" do
+      importer.options[:delete_missing_records] = false
+      importer.should_not_receive(:delete_missing_records)
+      importer.import_data([{:name => 'Beer 1', :price => 3.0}])
+    end
+
+    it "should default the delete_missing_records option to false" do
+      importer.options.delete(:delete_missing_records)
+      importer.should_not_receive(:delete_missing_records)
+      importer.import_data([{:name => 'Beer 1', :price => 3.0}])
+    end
+
+    context "when no existing_record_scope option is provided" do
+
+      it "should delete all existing records that were not included in the import" do
+        expect {
+          importer.import_data([{:name => 'Beer 1', :price => 3.0}])
+        }.to change { Item.count }.from(3).to(1)
+      end
+
+    end
+
+    context "when an existing_record_scope option is provided" do
+
+      before :each do
+        importer.options[:existing_record_scope] = Item.for_category(beer_category)
+      end
+
+      it "should not delete any records when all objects in the existing_records_scope are updated" do
+        expect {
+          importer.import_data([{:name => 'Beer 1', :price => 3.0}, {:name => 'Beer 2', :price => 3.0}])
+        }.to change { Item.count }.by(0)
+      end
+
+      it "should delete existing records, in the scope provided, that were not included in the import" do
+        expect {
+          importer.import_data([{:name => 'Beer 1', :price => 3.0}])
+        }.to change { Item.count }.from(3).to(2)
+      end
+
+    end
+
+  end
+
 end

data/spec/spec_helper.rb

@@ -44,4 +44,8 @@ class Item < ActiveRecord::Base
   act_as_importable :uid => 'name'
 
   belongs_to :category
+
+  def self.for_category(category)
+    where(:category_id => category.id)
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: act_as_importable
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.10
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-07 00:00:00.000000000 Z
+date: 2013-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -122,7 +122,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Helps import records from CSV files.