mongify 1.2.4 → 1.3.0

data/lib/mongify/translation/sync.rb

@@ -63,7 +63,8 @@ module Mongify
           max_updated_at, max_updated_at_id = Time.new(1970), nil
           rows.each do |row|
             row_hash = t.translate(row)
-            updated_at = Time.parse(row['updated_at'])
+            updated_at = row['updated_at']
+            updated_at = Time.parse(updated_at) if updated_at.instance_of?(String)
             if updated_at > max_updated_at
               max_updated_at = updated_at
               max_updated_at_id = row_hash['pre_mongified_id']
@@ -71,6 +72,7 @@ module Mongify
             no_sql_connection.upsert(t.name, row_hash.merge({DRAFT_KEY => true}))
             Mongify::Status.publish('copy_data')
           end
+          raise "Table #{t.sql_name} must have a primary key denoted by :key in the translation file" if t.key_column.nil?
           (self.max_updated_at ||= {})[t.sql_name] = {'max_updated_at_id' => max_updated_at_id, 'key_column' => t.key_column.name}
           Mongify::Status.publish('copy_data', :action => 'finish')
         end

data/lib/mongify/version.rb

@@ -1,4 +1,4 @@
 module Mongify
   # Mongify's Current Version Number
-  VERSION = "1.2.4"
+  VERSION = "1.3.0"
 end

data/mongify.gemspec

@@ -12,24 +12,25 @@ Gem::Specification.new do |s|
   s.description = %q{Mongify allows you to map your sql data into a mongodb document database with a simple DSL.}
   s.required_ruby_version = ">= 1.8.7"
 
-  s.add_dependency('activerecord', "~> 3.2")
-  s.add_dependency('activesupport', "~> 3.2")
-  s.add_dependency('mongo', "~> 1.10.2")
-  s.add_dependency('bson', "~> 1.10.2")
-  s.add_dependency('bson_ext', "~> 1.10.2") unless RUBY_PLATFORM == 'java'
+  s.add_dependency('activerecord', ">= 4.2")
+  s.add_dependency('activesupport', ">= 4.2")
+  s.add_dependency('mongo', ">= 1.10.2")
+  s.add_dependency('bson', ">= 1.10.2")
+  s.add_dependency('bson_ext', ">= 1.10.2") unless RUBY_PLATFORM == 'java'
   s.add_dependency('highline', '>= 1.6.1')
 
 
   s.add_development_dependency('rspec', '~> 2.0')
+  s.add_development_dependency('rspec-collection_matchers', '~> 1.0')
   s.add_development_dependency('cucumber', '>= 0.10')
   s.add_development_dependency('mocha', '>= 0.9.8')
-  s.add_development_dependency('yard', '>= 0.5.3')
+  s.add_development_dependency('yard', '>= 0.8')
   s.add_development_dependency('sqlite3', '>= 1.3')
   s.add_development_dependency('pg', '>= 0.17')
-  s.add_development_dependency('mysql2', '~> 0.3.1')
+  s.add_development_dependency('mysql2', '>= 0.4')
   s.add_development_dependency('watchr', '>= 0.6')
   s.add_development_dependency('rake')
-  s.add_development_dependency('pry-plus')
+  s.add_development_dependency('jazz_fingers')
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

data/spec/mongify/cli/command/help_spec.rb

@@ -3,7 +3,7 @@ describe Mongify::CLI::Command::Help do
   before :each do
     @text = 'Piece of interesting text'
     @cmd = Mongify::CLI::Command::Help.new(@text)
-    @view = mock('view').as_null_object
+    @view = double('view').as_null_object
   end
 
   it 'displays the correct text on the view' do

data/spec/mongify/cli/command/version_spec.rb

@@ -4,7 +4,7 @@ describe Mongify::CLI::Command::Version do
   before :each do
     @text = 'Piece of interesting text'
     @cmd = Mongify::CLI::Command::Version.new(@text)
-    @view = mock('view').as_null_object
+    @view = double('view').as_null_object
   end
 
   it 'displays the text on the view' do

data/spec/mongify/cli/worker_command_spec.rb

@@ -14,8 +14,8 @@ describe Mongify::CLI::Command::Worker do
 
     @translation_file = 'spec/files/translation.rb'
 
-    Mongify::Translation.stub(:load).and_return(stub(:print => 'worked'))
-    @view = mock('view').as_null_object
+    Mongify::Translation.stub(:load).and_return(double(:print => 'worked'))
+    @view = double('view').as_null_object
   end
 
   context "list_commands" do
@@ -62,7 +62,7 @@ describe Mongify::CLI::Command::Worker do
   context "translation command" do
     before(:each) do
       @command = Mongify::CLI::Command::Worker.new('translation', @config)
-      Mongify::Translation.stub(:load).with(@sql_connection).and_return(stub(:print => 'worked'))
+      Mongify::Translation.stub(:load).with(@sql_connection).and_return(double(:print => 'worked'))
     end
 
     it "should require configuration file" do
@@ -75,7 +75,7 @@ describe Mongify::CLI::Command::Worker do
     end
 
     it "should call load on Translation" do
-      Mongify::Translation.should_receive(:load).with(@sql_connection).and_return(stub(:print => 'worked'))
+      Mongify::Translation.should_receive(:load).with(@sql_connection).and_return(double(:print => 'worked'))
       @command.execute(@view)
     end
   end
@@ -83,7 +83,7 @@ describe Mongify::CLI::Command::Worker do
   context "process command" do
     before(:each) do
       @command = Mongify::CLI::Command::Worker.new('process', @config, 'spec/files/translation.rb')
-      Mongify::Translation.stub(:parse).and_return(mock(:process => true))
+      Mongify::Translation.stub(:parse).and_return(double(:process => true))
     end
     it "should report success" do
       @view.should_receive(:report_error).never
@@ -105,7 +105,7 @@ describe Mongify::CLI::Command::Worker do
     end
 
     it "should call process on translation" do
-      Mongify::Translation.should_receive(:parse).and_return(mock(:process => true))
+      Mongify::Translation.should_receive(:parse).and_return(double(:process => true))
       @command.execute(@view)
     end
   end
@@ -113,7 +113,7 @@ describe Mongify::CLI::Command::Worker do
   context "sync command" do
     before(:each) do
       @command = Mongify::CLI::Command::Worker.new('sync', @config, 'spec/files/translation.rb')
-      Mongify::Translation.stub(:parse).and_return(mock(:sync => true))
+      Mongify::Translation.stub(:parse).and_return(double(:sync => true))
     end
     it "should report success" do
       @view.should_receive(:report_error).never
@@ -135,7 +135,7 @@ describe Mongify::CLI::Command::Worker do
     end
 
     it "should call sync on translation" do
-      Mongify::Translation.should_receive(:parse).and_return(mock(:sync => true))
+      Mongify::Translation.should_receive(:parse).and_return(double(:sync => true))
       @command.execute(@view)
     end
   end

data/spec/mongify/database/base_connection_spec.rb

@@ -39,7 +39,7 @@ describe Mongify::Database::BaseConnection do
   end
 
   it "should respond to available settings" do
-    @base_connection.respond_to?(:host).should be_true
+    @base_connection.respond_to?(:host).should be_truthy
   end
 
   it "should force adapter to a string" do

data/spec/mongify/database/column_spec.rb

@@ -41,7 +41,7 @@ describe Mongify::Database::Column do
 
     context "id column" do
       before(:each) do
-        @col = mock(:sql_name => 'id')
+        @col = double(:sql_name => 'id')
       end
       it "should detect column with type :integer as a :key column" do
         @col.stub(:type).and_return(:integer)
@@ -64,7 +64,7 @@ describe Mongify::Database::Column do
     end
     context "references" do
       before(:each) do
-        @col = mock(:sql_name => 'post_id', :type => :integer, :referenced? => false)
+        @col = double(:sql_name => 'post_id', :type => :integer, :referenced? => false)
       end
       it "should detect column references" do
         @col.should_receive('references=').with('posts')
@@ -242,7 +242,7 @@ describe Mongify::Database::Column do
         @column = Mongify::Database::Column.new('created_at', :datetime)
       end
       it "should return a datetime format" do
-        @column.send(:type_cast, '2011-01-14 21:23:39').should == Time.local(2011, 01, 14, 21, 23,39)
+        @column.send(:type_cast, '2011-01-14 21:23:39').should == Time.utc(2011, 01, 14, 21, 23,39)
       end
       it "should return nil if input is nil" do
         @column.send(:type_cast, nil).should == nil
@@ -326,7 +326,7 @@ describe Mongify::Database::Column do
         @column = Mongify::Database::Column.new('created_at', :timestamp)
       end
       it "should return a datetime format" do
-        @column.send(:type_cast, '2011-01-14 21:23:39').should == Time.local(2011, 01, 14, 21, 23,39)
+        @column.send(:type_cast, '2011-01-14 21:23:39').should == Time.utc(2011, 01, 14, 21, 23,39)
       end
       it "should return nil if input is nil" do
         @column.send(:type_cast, nil).should == nil
@@ -337,7 +337,7 @@ describe Mongify::Database::Column do
         @column = Mongify::Database::Column.new('created_at', :time)
       end
       it "should return a time format" do
-        @column.send(:type_cast, '21:23:39').should == Time.local(2000, 01, 01, 21, 23,39)
+        @column.send(:type_cast, '21:23:39').should == Time.utc(2000, 01, 01, 21, 23,39)
       end
       it "should return nil if input is nil" do
         @column.send(:type_cast, nil).should == nil
@@ -348,7 +348,7 @@ describe Mongify::Database::Column do
         @column = Mongify::Database::Column.new('created_at', :date)
       end
       it "should return a time format" do
-        @column.send(:type_cast, '2011-01-14').should == Time.local(2011, 01, 14)
+        @column.send(:type_cast, '2011-01-14').should == Time.utc(2011, 01, 14)
       end
       it "should return nil if input is nil" do
         @column.send(:type_cast, nil).should == nil

data/spec/mongify/database/data_row_spec.rb

@@ -65,10 +65,10 @@ describe Mongify::Database::DataRow do
 
   context "respond_to" do
     it "should be true for first_name" do
-      @datarow.respond_to?('first_name').should be_true
+      @datarow.respond_to?('first_name').should be_truthy
     end
     it "should be true for first_name=" do
-      @datarow.respond_to?('first_name=').should be_true
+      @datarow.respond_to?('first_name=').should be_truthy
     end
   end
 
@@ -78,7 +78,7 @@ describe Mongify::Database::DataRow do
     end
     it "should read attributes like delete" do
       @datarow.delete=true
-      @datarow.read_attribute('delete').should be_true
+      @datarow.read_attribute('delete').should be_truthy
     end
     it "should read non existing attributes" do
       @datarow.read_attributes('monkey').should be_nil

data/spec/mongify/database/no_sql_connection_spec.rb

@@ -48,7 +48,7 @@ describe Mongify::Database::NoSqlConnection do
 
   context "connection" do
     before(:each) do
-      @mock_connection = mock(:connected? => true)
+      @mock_connection = double(:connected? => true)
       Mongo::Connection.stub(:new).and_return(@mock_connection)
     end
 
@@ -70,8 +70,8 @@ describe Mongify::Database::NoSqlConnection do
 
   context "database action:" do
     before(:each) do
-       @collection = mock
-       @db = mock
+       @collection = double
+       @db = double
        @db.stub(:[]).with('users').and_return(@collection)
        @mongodb_connection.stub(:db).and_return(@db)
     end
@@ -95,7 +95,7 @@ describe Mongify::Database::NoSqlConnection do
 
     context "select_rows" do
       it "should return all records" do
-        @collection.should_receive(:find).with().and_return([])
+        @collection.should_receive(:find).with(no_args).and_return([])
         @mongodb_connection.select_rows('users')
       end
     end
@@ -120,7 +120,7 @@ describe Mongify::Database::NoSqlConnection do
       it "should update the record if its pre_mongified_id exists" do
         attributes = {'pre_mongified_id' => 1, 'post_id' => 123}
         id = 10
-        duplicate = mock
+        duplicate = double
         duplicate.stub(:[]).with(:_id).and_return(id)
         @mongodb_connection.stub(:find_one).with('users', {"pre_mongified_id" => 1}).and_return(duplicate)
         @mongodb_connection.should_receive(:find_one).with('users', {"pre_mongified_id" => 1})
@@ -174,7 +174,7 @@ describe Mongify::Database::NoSqlConnection do
 
   context "force" do
     before(:each) do
-      @mock_connection = mock(:connected? => true, :drop_database => true)
+      @mock_connection = double(:connected? => true, :drop_database => true)
       Mongo::Connection.stub(:new).and_return(@mock_connection)
       @mongodb_connection = Mongify::Database::NoSqlConnection.new(:host => 'localhost', :database => 'blue', :force => true)
       Mongify::UI.stub(:ask).and_return(true)

data/spec/mongify/database/sql_connection_spec.rb

@@ -91,7 +91,7 @@ describe Mongify::Database::SqlConnection do
 
   context "select_all" do
     it "should generate correct select statement" do
-      @mock_conn = mock
+      @mock_conn = double
       @mock_conn.should_receive(:select_all).with('SELECT * FROM users')
       sqlite_connection.stub(:connection).and_return(@mock_conn)
       sqlite_connection.select_rows('users')
@@ -101,7 +101,7 @@ describe Mongify::Database::SqlConnection do
   context "select_by_query" do
     it "should select rows based on a query" do
       query = "SELECT * FROM users WHERE true"
-      @mock_conn = mock
+      @mock_conn = double
       @mock_conn.should_receive(:select_all).with(query)
       sqlite_connection.stub(:connection).and_return(@mock_conn)
       sqlite_connection.select_by_query(query)
@@ -111,7 +111,7 @@ describe Mongify::Database::SqlConnection do
   context "count" do
     it "should get count of all rows in a table" do
       query = "SELECT COUNT(*) FROM users"
-      @mock_conn = mock
+      @mock_conn = double
       @mock_conn.should_receive(:select_value).with(query)
       sqlite_connection.stub(:connection).and_return(@mock_conn)
       sqlite_connection.count('users')
@@ -119,7 +119,7 @@ describe Mongify::Database::SqlConnection do
 
     it "should get count of rows in a table filtered by a query" do
       query = "SELECT COUNT(*) FROM users WHERE true"
-      @mock_conn = mock
+      @mock_conn = double
       @mock_conn.should_receive(:select_value).with(query)
       sqlite_connection.stub(:connection).and_return(@mock_conn)
       sqlite_connection.count('users', 'true')
@@ -129,7 +129,7 @@ describe Mongify::Database::SqlConnection do
   context "execute" do
     it "should execute an arbitrary query" do
       query = "CREATE TABLE x(int y);"
-      @mock_conn = mock
+      @mock_conn = double
       @mock_conn.should_receive(:execute).with(query)
       sqlite_connection.stub(:connection).and_return(@mock_conn)
       sqlite_connection.execute(query)

data/spec/mongify/database/table_spec.rb

@@ -94,14 +94,14 @@ describe Mongify::Database::Table do
 
   context "add_column" do
     it "should require Mongify::Database::Column" do
-      lambda { @table.add_column("Not a column") }.should raise_error(Mongify::DatabaseColumnExpected)
+      expect { @table.add_column("Not a column") }.to raise_error(Mongify::DatabaseColumnExpected)
     end
     it "shold except Mongify::Database::Column as a parameter" do
-      lambda { @table.add_column(Mongify::Database::Column.new('test')) }.should_not raise_error(Mongify::DatabaseColumnExpected)
+      expect { @table.add_column(Mongify::Database::Column.new('test')) }.to_not raise_error
     end
 
     it "should add to the column count" do
-      lambda { @table.add_column(Mongify::Database::Column.new('test')) }.should change{@table.columns.count}.by(1)
+      expect { @table.add_column(Mongify::Database::Column.new('test')) }.to change{@table.columns.count}.by(1)
     end
 
     it "should be indexed" do
@@ -286,8 +286,8 @@ describe Mongify::Database::Table do
 
   context "translate" do
     before(:each) do
-      @column1 = mock(:translate => {'first_name' => 'Timmy'}, :name => 'first_name')
-      @column2 = mock(:translate => {'last_name' => 'Zuza'}, :name => 'last_name')
+      @column1 = double(:translate => {'first_name' => 'Timmy'}, :name => 'first_name')
+      @column2 = double(:translate => {'last_name' => 'Zuza'}, :name => 'last_name')
       @table.stub(:find_column).with(anything).and_return(nil)
       @table.stub(:find_column).with('first_name').and_return(@column1)
       @table.stub(:find_column).with('last_name').and_return(@column2)

data/spec/mongify/status_spec.rb

@@ -7,7 +7,7 @@ describe Mongify::Status do
   end
   context "publish" do
     before(:each) do
-      @bar = stub(:title= => '')
+      @bar = double(:title= => '')
       Mongify::Status.bars.stub(:[]).and_return(@bar)
     end
     it "should create a new progress bar" do

data/spec/mongify/translation/process_spec.rb

@@ -46,7 +46,7 @@ describe Mongify::Translation::Process do
     end
 
     it "should add pre_mongified_id index to database" do
-      tables = [stub(:name => 'users')]
+      tables = [double(:name => 'users')]
       @translation.stub(:copy_tables).and_return(tables)
       @no_sql_connection.should_receive(:create_pre_mongified_id_index).with('users')
       @translation.process(@sql_connection, @no_sql_connection)
@@ -61,20 +61,20 @@ describe Mongify::Translation::Process do
 
   context "fetch_reference_ids" do
     it "should get correct information" do
-      @no_sql_connection = mock()
+      @no_sql_connection = double()
       @translation.stub(:no_sql_connection).and_return(@no_sql_connection)
-      @table = mock(:translate => {}, :name => 'users', :embedded? => false, :find_column => nil)
-      @column = mock(:name => 'user_id', :references => 'users')
+      @table = double(:translate => {}, :name => 'users', :embedded? => false, :find_column => nil)
+      @column = double(:name => 'user_id', :references => 'users')
       @table.stub(:reference_columns).and_return([@column])
       @no_sql_connection.should_receive(:get_id_using_pre_mongified_id).with('users', 1).once.and_return(500)
       @translation.send(:fetch_reference_ids, @table, {'user_id' => 1}).should == {'user_id' => 500}
     end
 
     it "should get correct information for arrays" do
-      @no_sql_connection = mock()
+      @no_sql_connection = double()
       @translation.stub(:no_sql_connection).and_return(@no_sql_connection)
-      @table = mock(:translate => {}, :name => 'users', :embedded? => false)
-      @column = mock(:name => 'user_ids', :references => 'users')
+      @table = double(:translate => {}, :name => 'users', :embedded? => false)
+      @column = double(:name => 'user_ids', :references => 'users')
       @table.stub(:reference_columns).and_return([@column])
       @no_sql_connection.should_receive(:get_id_using_pre_mongified_id).with('users', 1).once.and_return(500)
       @no_sql_connection.should_receive(:get_id_using_pre_mongified_id).with('users', 2).once.and_return(501)
@@ -87,10 +87,10 @@ describe Mongify::Translation::Process do
       @sql_connection.stub(:select_paged_rows).and_return([{'first_name'=> 'Timmy', 'last_name' => 'Zuza', 'preference_id' => 1}])
       @translation.stub(:sql_connection).and_return(@sql_connection)
 
-      @no_sql_connection = mock()
+      @no_sql_connection = double()
       @translation.stub(:no_sql_connection).and_return(@no_sql_connection)
 
-      @table = mock(:translate => {'first_name'=> 'Timmy', 'last_name' => 'Zuza', 'preference_id' => 1},
+      @table = double(:translate => {'first_name'=> 'Timmy', 'last_name' => 'Zuza', 'preference_id' => 1},
                     :name => 'users',
                     :embedded? => false,
                     :sql_name => 'users')
@@ -113,8 +113,8 @@ describe Mongify::Translation::Process do
     context "copy_embed_tables" do
       before(:each) do
         @sql_connection.stub(:select_paged_rows).and_return([{'first_name'=> 'Timmy', 'last_name' => 'Zuza', 'preference_id' => 1}])
-        @target_table = mock(:name => 'posts', :embedded? => false, :sql_name => 'posts')
-        @embed_table = mock(:translate => {}, :name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :sql_name => 'comments', :find_column => nil)
+        @target_table = double(:name => 'posts', :embedded? => false, :sql_name => 'posts')
+        @embed_table = double(:translate => {}, :name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :sql_name => 'comments', :find_column => nil)
         @no_sql_connection.stub(:find_one).and_return({'_id' => 500})
         @translation.stub(:tables).and_return([@target_table, @embed_table])
         @translation.stub(:fetch_reference_ids).and_return({})
@@ -126,25 +126,25 @@ describe Mongify::Translation::Process do
         @translation.send(:copy_embedded_tables)
       end
       it "should remove the pre_mongified_id before embedding" do
-        @embed_table = mock(:translate => {'first_name' => 'bob', 'pre_mongified_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
+        @embed_table = double(:translate => {'first_name' => 'bob', 'pre_mongified_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
         @translation.stub(:tables).and_return([@target_table, @embed_table])
         @no_sql_connection.should_receive(:update).with("posts", 500, {"$addToSet"=>{"comments"=>{'first_name' => 'bob'}}})
         @translation.send(:copy_embedded_tables)
       end
       it "should remove the parent_id from the embedding row" do
-        @embed_table = mock(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
+        @embed_table = double(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
         @translation.stub(:tables).and_return([@target_table, @embed_table])
         @no_sql_connection.should_receive(:update).with("posts", 500, {"$addToSet"=>{"comments"=>{'first_name' => 'bob'}}})
         @translation.send(:copy_embedded_tables)
       end
       it "should call $addToSet on update of an embed_as_object table" do
-        @embed_table = mock(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
+        @embed_table = double(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
         @translation.stub(:tables).and_return([@target_table, @embed_table])
         @no_sql_connection.should_receive(:update).with("posts", 500, {"$set"=>{"comments"=>{'first_name' => 'bob'}}})
         @translation.send(:copy_embedded_tables)
       end
       it "should allow rename of table" do
-        @embed_table = mock(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'notes', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
+        @embed_table = double(:translate => {'first_name' => 'bob', 'post_id' => 1}, :name => 'notes', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
         @translation.stub(:tables).and_return([@target_table, @embed_table])
         @no_sql_connection.should_receive(:update).with("posts", 500, {"$set"=>{"notes"=>{'first_name' => 'bob'}}})
         @translation.send(:copy_embedded_tables)
@@ -152,25 +152,25 @@ describe Mongify::Translation::Process do
 
       context "parent modification" do
         it 'should unset fields deleted in the parent row' do
-          @embed_table = mock(:translate => [{}, {'email' => 'true'}, {'field_1' => '1'}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
+          @embed_table = double(:translate => [{}, {'email' => 'true'}, {'field_1' => '1'}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
           @translation.stub(:tables).and_return([@target_table, @embed_table])
           @no_sql_connection.should_receive(:update).with("posts", 500, {"$set"=>{"preferences"=>{}, "email"=>"true"}, "$unset"=>{'field_1' => '1'}})
           @translation.send(:copy_embedded_tables)
         end
         it "should work with embedded objects" do
-          @embed_table = mock(:translate => [{}, {'email' => 'true'}, {}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
+          @embed_table = double(:translate => [{}, {'email' => 'true'}, {}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => true, :find_column => nil)
           @translation.stub(:tables).and_return([@target_table, @embed_table])
           @no_sql_connection.should_receive(:update).with("posts", 500, {"$set"=>{"preferences"=>{}, "email"=>"true"}})
           @translation.send(:copy_embedded_tables)
         end
         it "should work with embedded arrays" do
-          @embed_table = mock(:translate => [{}, {'email' => 'true'}, {}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
+          @embed_table = double(:translate => [{}, {'email' => 'true'}, {}], :name => 'preferences', :sql_name => 'preferences', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
           @translation.stub(:tables).and_return([@target_table, @embed_table])
           @no_sql_connection.should_receive(:update).with("posts", 500, {"$addToSet"=>{"preferences"=>{}}, "$set" => {"email"=>"true"}})
           @translation.send(:copy_embedded_tables)
         end
         it "should not set embedded attribute in parent" do
-          @embed_table = mock(:translate => [{'first_name' => 'joe'}, {'email' => 'true', 'comments' => [{'first_name' => 'bob'}]}, {}], :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
+          @embed_table = double(:translate => [{'first_name' => 'joe'}, {'email' => 'true', 'comments' => [{'first_name' => 'bob'}]}, {}], :name => 'comments', :sql_name => 'comments', :embedded? => true, :embed_on => 'post_id', :embed_in => 'posts', :embedded_as_object? => false, :find_column => nil)
           @translation.stub(:tables).and_return([@target_table, @embed_table])
           @no_sql_connection.should_receive(:update).with("posts", 500, {"$addToSet" => {"comments" => {"first_name" => "joe"}}, "$set" => {"email" => "true"}})
           @translation.send(:copy_embedded_tables)
@@ -210,14 +210,14 @@ describe Mongify::Translation::Process do
       it "should work correctly" do
         @no_sql_connection.should_receive(:select_rows).and_return([{'_id' => 100, 'user_id' => 1}, {'_id'=> 101, 'user_id' => 2}])
         @no_sql_connection.stub(:get_id_using_pre_mongified_id).twice.and_return(500)
-        @table.should_receive(:reference_columns).twice.and_return([mock(:name => 'user_id', :references=>'users')])
+        @table.should_receive(:reference_columns).twice.and_return([double(:name => 'user_id', :references=>'users')])
         @no_sql_connection.should_receive(:update).twice
         @translation.send(:update_reference_ids)
       end
       it "should only update when new_id is present" do
         @no_sql_connection.should_receive(:select_rows).and_return([{'_id' => 100, 'user_id' => 1}, {'_id'=> 101, 'user_id' => 2}])
         @no_sql_connection.stub(:get_id_using_pre_mongified_id).twice.and_return(nil)
-        @table.should_receive(:reference_columns).twice.and_return([mock(:name => 'user_id', :references=>'users')])
+        @table.should_receive(:reference_columns).twice.and_return([double(:name => 'user_id', :references=>'users')])
         @no_sql_connection.should_receive(:update).never
         @translation.send(:update_reference_ids)
       end
@@ -225,7 +225,7 @@ describe Mongify::Translation::Process do
 
     context "copy_polymorphic_tables" do
       before(:each) do
-        @ref_table = mock(:name => 'user_accounts',
+        @ref_table = double(:name => 'user_accounts',
                           :embedded? => false,
                           :ignored? => false,
                           :sql_name => 'user_accounts')
@@ -236,7 +236,7 @@ describe Mongify::Translation::Process do
       end
       context "embedded" do
         it "should work correctly" do
-           @table = mock(:translate => {'data' => 123},
+           @table = double(:translate => {'data' => 123},
                           :name => 'comments',
                           :embedded? => true,
                           :polymorphic_as => 'commentable',
@@ -255,7 +255,7 @@ describe Mongify::Translation::Process do
       end
       context "not embedded" do
         before(:each) do
-          @table = mock(:translate => {'data' => 123, 'commentable_type' => 'UserAccount', 'commentable_id' => 1},
+          @table = double(:translate => {'data' => 123, 'commentable_type' => 'UserAccount', 'commentable_id' => 1},
                           :name => 'comments',
                           :embedded? => false,
                           :polymorphic_as => 'commentable',