massive_record 0.2.1 → 0.2.2.rc1

data/spec/shared/orm/persistence/a_persistence_embedded_operation_class.rb

@@ -0,0 +1,3 @@
+shared_examples_for "a persistence embedded operation class" do
+  it_should_behave_like "a persistence operation class"
+end

data/spec/shared/orm/persistence/a_persistence_operation_class.rb

@@ -0,0 +1,11 @@
+shared_examples_for "a persistence operation class" do
+  describe described_class do
+    subject { described_class }
+
+    its(:included_modules) { should include MassiveRecord::ORM::Persistence::Operations }
+  end
+
+  it "responds to execute" do
+    subject.execute
+  end
+end

data/spec/shared/orm/persistence/a_persistence_table_operation_class.rb

@@ -0,0 +1,11 @@
+shared_examples_for "a persistence table operation class" do
+  it_should_behave_like "a persistence operation class"
+
+  describe described_class do
+    subject { described_class }
+
+    its(:included_modules) {
+      should include MassiveRecord::ORM::Persistence::Operations::TableOperationHelpers
+    }
+  end
+end

data/spec/shared/orm/relations/proxy.rb

@@ -39,10 +39,14 @@ shared_examples_for "relation proxy" do
 
 
   describe "#reset" do
-    it "should not be loaded after reset" do
+    it "alters the loaded state" do
       subject.loaded!
       subject.reset
-      should_not be_loaded
+      if subject.metadata.embedded_in?
+        should be_loaded
+      else
+        should_not be_loaded
+      end
     end
 
     it "should reset the proxy_target" do
@@ -59,6 +63,7 @@ shared_examples_for "relation proxy" do
     end
 
     it "should reset the proxy" do
+      subject.should_receive(:can_find_proxy_target?).and_return false
       subject.should_receive :reset
       subject.reload
     end
@@ -121,6 +126,8 @@ shared_examples_for "relation proxy" do
     let(:new_proxy_target) { subject.represents_a_collection? ? [subject.proxy_target_class.new] : subject.proxy_target_class.new }
 
     before do
+      new_proxy_target.each { |target| target.stub(:valid?).and_return true } if new_proxy_target.is_a? Array
+      old_proxy_target.each { |target| target.stub(:valid?).and_return true } if new_proxy_target.is_a? Array
       subject.proxy_target = old_proxy_target
     end
 

data/spec/spec_helper.rb

@@ -14,3 +14,12 @@ end
 Dir["#{SPEC_DIR}/orm/models/*.rb"].each { |f| require f }
 Dir["#{SPEC_DIR}/support/**/*.rb"].each { |f| require f }
 Dir["#{SPEC_DIR}/shared/**/*.rb"].each { |f| require f }
+
+
+MassiveRecord::ORM::IdentityMap.enabled = false
+if MassiveRecord::ORM::IdentityMap.enabled?
+  Rspec.configure do |c|
+    puts "IdentityMap is enabled!"
+    c.before { MassiveRecord::ORM::IdentityMap.clear }
+  end
+end

data/spec/support/mock_massive_record_connection.rb

@@ -21,7 +21,8 @@ module MockMassiveRecordConnection
       # Setting up a mock connection when asked for new
       mock_connection = mock(MassiveRecord::Wrapper::Connection,
         :open => true,
-        :tables => MassiveRecord::ORM::Table.descendants.collect(&:table_name)
+        :tables => MassiveRecord::ORM::Table.descendants.collect(&:table_name),
+        :atomicIncrement => 1
       )
       MassiveRecord::Wrapper::Connection.stub(:new).and_return(mock_connection)
 

metadata

@@ -1,8 +1,8 @@
 --- !ruby/object:Gem::Specification 
 name: massive_record
 version: !ruby/object:Gem::Version 
-  prerelease: 
-  version: 0.2.1
+  prerelease: 6
+  version: 0.2.2.rc1
 platform: ruby
 authors: 
 - Companybook
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-24 00:00:00 +02:00
+date: 2011-10-07 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -19,9 +19,9 @@ dependencies:
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ">="
+    - - "="
       - !ruby/object:Gem::Version 
-        version: 0.5.0
+        version: 0.6.0
   type: :runtime
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
@@ -30,9 +30,9 @@ dependencies:
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ">="
+    - - ~>
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 3.0.7
   type: :runtime
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
@@ -41,9 +41,9 @@ dependencies:
   requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
-    - - ">="
+    - - ~>
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 3.0.7
   type: :runtime
   version_requirements: *id003
 - !ruby/object:Gem::Dependency 
@@ -65,7 +65,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 2.1.0
+        version: "0"
   type: :development
   version_requirements: *id005
 description: HBase Ruby client API
@@ -99,6 +99,7 @@ files:
 - lib/massive_record/adapters/thrift/scanner.rb
 - lib/massive_record/adapters/thrift/table.rb
 - lib/massive_record/orm/attribute_methods.rb
+- lib/massive_record/orm/attribute_methods/cast_numbers_on_write.rb
 - lib/massive_record/orm/attribute_methods/dirty.rb
 - lib/massive_record/orm/attribute_methods/read.rb
 - lib/massive_record/orm/attribute_methods/time_zone_conversion.rb
@@ -112,26 +113,49 @@ files:
 - lib/massive_record/orm/column.rb
 - lib/massive_record/orm/config.rb
 - lib/massive_record/orm/default_id.rb
+- lib/massive_record/orm/embedded.rb
 - lib/massive_record/orm/errors.rb
 - lib/massive_record/orm/finders.rb
 - lib/massive_record/orm/finders/rescue_missing_table_on_find.rb
 - lib/massive_record/orm/finders/scope.rb
 - lib/massive_record/orm/id_factory.rb
+- lib/massive_record/orm/id_factory/atomic_incrementation.rb
+- lib/massive_record/orm/id_factory/timestamp.rb
+- lib/massive_record/orm/identity_map.rb
 - lib/massive_record/orm/log_subscriber.rb
+- lib/massive_record/orm/observer.rb
 - lib/massive_record/orm/persistence.rb
+- lib/massive_record/orm/persistence/operations.rb
+- lib/massive_record/orm/persistence/operations/atomic_operation.rb
+- lib/massive_record/orm/persistence/operations/destroy.rb
+- lib/massive_record/orm/persistence/operations/embedded/destroy.rb
+- lib/massive_record/orm/persistence/operations/embedded/insert.rb
+- lib/massive_record/orm/persistence/operations/embedded/operation_helpers.rb
+- lib/massive_record/orm/persistence/operations/embedded/reload.rb
+- lib/massive_record/orm/persistence/operations/embedded/update.rb
+- lib/massive_record/orm/persistence/operations/insert.rb
+- lib/massive_record/orm/persistence/operations/reload.rb
+- lib/massive_record/orm/persistence/operations/suppress.rb
+- lib/massive_record/orm/persistence/operations/table_operation_helpers.rb
+- lib/massive_record/orm/persistence/operations/update.rb
 - lib/massive_record/orm/query_instrumentation.rb
+- lib/massive_record/orm/raw_data.rb
 - lib/massive_record/orm/relations.rb
 - lib/massive_record/orm/relations/interface.rb
 - lib/massive_record/orm/relations/metadata.rb
 - lib/massive_record/orm/relations/proxy.rb
+- lib/massive_record/orm/relations/proxy/embedded_in.rb
+- lib/massive_record/orm/relations/proxy/embedded_in_polymorphic.rb
+- lib/massive_record/orm/relations/proxy/embeds_many.rb
 - lib/massive_record/orm/relations/proxy/references_many.rb
 - lib/massive_record/orm/relations/proxy/references_one.rb
 - lib/massive_record/orm/relations/proxy/references_one_polymorphic.rb
+- lib/massive_record/orm/relations/proxy_collection.rb
 - lib/massive_record/orm/schema.rb
 - lib/massive_record/orm/schema/column_families.rb
 - lib/massive_record/orm/schema/column_family.rb
-- lib/massive_record/orm/schema/column_interface.rb
 - lib/massive_record/orm/schema/common_interface.rb
+- lib/massive_record/orm/schema/embedded_interface.rb
 - lib/massive_record/orm/schema/field.rb
 - lib/massive_record/orm/schema/fields.rb
 - lib/massive_record/orm/schema/table_interface.rb
@@ -139,6 +163,7 @@ files:
 - lib/massive_record/orm/table.rb
 - lib/massive_record/orm/timestamps.rb
 - lib/massive_record/orm/validations.rb
+- lib/massive_record/orm/validations/associated.rb
 - lib/massive_record/rails/controller_runtime.rb
 - lib/massive_record/rails/railtie.rb
 - lib/massive_record/spec/support/simple_database_cleaner.rb
@@ -155,25 +180,32 @@ files:
 - spec/adapter/thrift/connection_spec.rb
 - spec/adapter/thrift/table_find_spec.rb
 - spec/adapter/thrift/table_spec.rb
+- spec/adapter/thrift/utf8_encoding_of_id_spec.rb
 - spec/config.yml.example
 - spec/orm/cases/attribute_methods_spec.rb
 - spec/orm/cases/auto_generate_id_spec.rb
 - spec/orm/cases/base_spec.rb
 - spec/orm/cases/callbacks_spec.rb
-- spec/orm/cases/column_spec.rb
+- spec/orm/cases/change_id_spec.rb
 - spec/orm/cases/config_spec.rb
 - spec/orm/cases/default_id_spec.rb
 - spec/orm/cases/default_values_spec.rb
 - spec/orm/cases/dirty_spec.rb
+- spec/orm/cases/embedded_spec.rb
 - spec/orm/cases/encoding_spec.rb
-- spec/orm/cases/finder_default_scope.rb
+- spec/orm/cases/finder_default_scope_spec.rb
 - spec/orm/cases/finder_scope_spec.rb
 - spec/orm/cases/finders_spec.rb
 - spec/orm/cases/i18n_spec.rb
-- spec/orm/cases/id_factory_spec.rb
+- spec/orm/cases/id_factory/atomic_incrementation_spec.rb
+- spec/orm/cases/id_factory/timestamp_spec.rb
+- spec/orm/cases/identity_map/identity_map_spec.rb
+- spec/orm/cases/identity_map/middleware_spec.rb
 - spec/orm/cases/log_subscriber_spec.rb
 - spec/orm/cases/mass_assignment_security_spec.rb
+- spec/orm/cases/observing_spec.rb
 - spec/orm/cases/persistence_spec.rb
+- spec/orm/cases/raw_data_spec.rb
 - spec/orm/cases/single_table_inheritance_spec.rb
 - spec/orm/cases/table_spec.rb
 - spec/orm/cases/time_zone_awareness_spec.rb
@@ -183,25 +215,48 @@ files:
 - spec/orm/coders/json_spec.rb
 - spec/orm/coders/yaml_spec.rb
 - spec/orm/models/address.rb
+- spec/orm/models/address_with_timestamp.rb
 - spec/orm/models/best_friend.rb
+- spec/orm/models/car.rb
 - spec/orm/models/friend.rb
 - spec/orm/models/model_without_default_id.rb
 - spec/orm/models/person.rb
 - spec/orm/models/person_with_timestamp.rb
 - spec/orm/models/test_class.rb
+- spec/orm/persistence/operations/atomic_operation_spec.rb
+- spec/orm/persistence/operations/destroy_spec.rb
+- spec/orm/persistence/operations/embedded/destroy_spec.rb
+- spec/orm/persistence/operations/embedded/insert_spec.rb
+- spec/orm/persistence/operations/embedded/operation_helpers_spec.rb
+- spec/orm/persistence/operations/embedded/reload_spec.rb
+- spec/orm/persistence/operations/embedded/update_spec.rb
+- spec/orm/persistence/operations/insert_spec.rb
+- spec/orm/persistence/operations/reload_spec.rb
+- spec/orm/persistence/operations/suppress_spec.rb
+- spec/orm/persistence/operations/table_operation_helpers_spec.rb
+- spec/orm/persistence/operations/update_spec.rb
+- spec/orm/persistence/operations_spec.rb
 - spec/orm/relations/interface_spec.rb
 - spec/orm/relations/metadata_spec.rb
+- spec/orm/relations/proxy/embedded_in_polymorphic_spec.rb
+- spec/orm/relations/proxy/embedded_in_spec.rb
+- spec/orm/relations/proxy/embeds_many_spec.rb
 - spec/orm/relations/proxy/references_many_spec.rb
 - spec/orm/relations/proxy/references_one_polymorphic_spec.rb
 - spec/orm/relations/proxy/references_one_spec.rb
 - spec/orm/relations/proxy_spec.rb
 - spec/orm/schema/column_families_spec.rb
 - spec/orm/schema/column_family_spec.rb
-- spec/orm/schema/column_interface_spec.rb
+- spec/orm/schema/embedded_interface_spec.rb
 - spec/orm/schema/field_spec.rb
 - spec/orm/schema/fields_spec.rb
 - spec/orm/schema/table_interface_spec.rb
 - spec/shared/orm/coders/an_orm_coder.rb
+- spec/shared/orm/id_factories.rb
+- spec/shared/orm/model_with_timestamps.rb
+- spec/shared/orm/persistence/a_persistence_embedded_operation_class.rb
+- spec/shared/orm/persistence/a_persistence_operation_class.rb
+- spec/shared/orm/persistence/a_persistence_table_operation_class.rb
 - spec/shared/orm/relations/proxy.rb
 - spec/shared/orm/relations/singular_proxy.rb
 - spec/spec_helper.rb
@@ -227,9 +282,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version 
-      version: "0"
+      version: 1.3.1
 requirements: []
 
 rubyforge_project: massive_record
@@ -243,25 +298,32 @@ test_files:
 - spec/adapter/thrift/connection_spec.rb
 - spec/adapter/thrift/table_find_spec.rb
 - spec/adapter/thrift/table_spec.rb
+- spec/adapter/thrift/utf8_encoding_of_id_spec.rb
 - spec/config.yml.example
 - spec/orm/cases/attribute_methods_spec.rb
 - spec/orm/cases/auto_generate_id_spec.rb
 - spec/orm/cases/base_spec.rb
 - spec/orm/cases/callbacks_spec.rb
-- spec/orm/cases/column_spec.rb
+- spec/orm/cases/change_id_spec.rb
 - spec/orm/cases/config_spec.rb
 - spec/orm/cases/default_id_spec.rb
 - spec/orm/cases/default_values_spec.rb
 - spec/orm/cases/dirty_spec.rb
+- spec/orm/cases/embedded_spec.rb
 - spec/orm/cases/encoding_spec.rb
-- spec/orm/cases/finder_default_scope.rb
+- spec/orm/cases/finder_default_scope_spec.rb
 - spec/orm/cases/finder_scope_spec.rb
 - spec/orm/cases/finders_spec.rb
 - spec/orm/cases/i18n_spec.rb
-- spec/orm/cases/id_factory_spec.rb
+- spec/orm/cases/id_factory/atomic_incrementation_spec.rb
+- spec/orm/cases/id_factory/timestamp_spec.rb
+- spec/orm/cases/identity_map/identity_map_spec.rb
+- spec/orm/cases/identity_map/middleware_spec.rb
 - spec/orm/cases/log_subscriber_spec.rb
 - spec/orm/cases/mass_assignment_security_spec.rb
+- spec/orm/cases/observing_spec.rb
 - spec/orm/cases/persistence_spec.rb
+- spec/orm/cases/raw_data_spec.rb
 - spec/orm/cases/single_table_inheritance_spec.rb
 - spec/orm/cases/table_spec.rb
 - spec/orm/cases/time_zone_awareness_spec.rb
@@ -271,25 +333,48 @@ test_files:
 - spec/orm/coders/json_spec.rb
 - spec/orm/coders/yaml_spec.rb
 - spec/orm/models/address.rb
+- spec/orm/models/address_with_timestamp.rb
 - spec/orm/models/best_friend.rb
+- spec/orm/models/car.rb
 - spec/orm/models/friend.rb
 - spec/orm/models/model_without_default_id.rb
 - spec/orm/models/person.rb
 - spec/orm/models/person_with_timestamp.rb
 - spec/orm/models/test_class.rb
+- spec/orm/persistence/operations/atomic_operation_spec.rb
+- spec/orm/persistence/operations/destroy_spec.rb
+- spec/orm/persistence/operations/embedded/destroy_spec.rb
+- spec/orm/persistence/operations/embedded/insert_spec.rb
+- spec/orm/persistence/operations/embedded/operation_helpers_spec.rb
+- spec/orm/persistence/operations/embedded/reload_spec.rb
+- spec/orm/persistence/operations/embedded/update_spec.rb
+- spec/orm/persistence/operations/insert_spec.rb
+- spec/orm/persistence/operations/reload_spec.rb
+- spec/orm/persistence/operations/suppress_spec.rb
+- spec/orm/persistence/operations/table_operation_helpers_spec.rb
+- spec/orm/persistence/operations/update_spec.rb
+- spec/orm/persistence/operations_spec.rb
 - spec/orm/relations/interface_spec.rb
 - spec/orm/relations/metadata_spec.rb
+- spec/orm/relations/proxy/embedded_in_polymorphic_spec.rb
+- spec/orm/relations/proxy/embedded_in_spec.rb
+- spec/orm/relations/proxy/embeds_many_spec.rb
 - spec/orm/relations/proxy/references_many_spec.rb
 - spec/orm/relations/proxy/references_one_polymorphic_spec.rb
 - spec/orm/relations/proxy/references_one_spec.rb
 - spec/orm/relations/proxy_spec.rb
 - spec/orm/schema/column_families_spec.rb
 - spec/orm/schema/column_family_spec.rb
-- spec/orm/schema/column_interface_spec.rb
+- spec/orm/schema/embedded_interface_spec.rb
 - spec/orm/schema/field_spec.rb
 - spec/orm/schema/fields_spec.rb
 - spec/orm/schema/table_interface_spec.rb
 - spec/shared/orm/coders/an_orm_coder.rb
+- spec/shared/orm/id_factories.rb
+- spec/shared/orm/model_with_timestamps.rb
+- spec/shared/orm/persistence/a_persistence_embedded_operation_class.rb
+- spec/shared/orm/persistence/a_persistence_operation_class.rb
+- spec/shared/orm/persistence/a_persistence_table_operation_class.rb
 - spec/shared/orm/relations/proxy.rb
 - spec/shared/orm/relations/singular_proxy.rb
 - spec/spec_helper.rb

data/spec/orm/cases/column_spec.rb

@@ -1,49 +0,0 @@
-require 'spec_helper'
-require 'orm/models/address'
-
-describe "column classes" do
-  before do
-    @address = Address.new(:street => "Asker", :number => 5)
-  end
-
-  it "should have known_attribute_names" do
-    Address.should have(3).known_attribute_names
-    Address.known_attribute_names.should include("street", "number", "nice_place")
-  end
-
-  it "should have default_attributes_from_schema" do
-    Address.default_attributes_from_schema["street"].should be_nil
-    Address.default_attributes_from_schema["number"].should be_nil
-    Address.default_attributes_from_schema["nice_place"].should be_true
-  end
-
-  it "should have attributes_schema" do
-    Address.attributes_schema["street"].should be_instance_of MassiveRecord::ORM::Schema::Field
-  end
-
-  it "should have a default value set" do
-    @address.nice_place.should be_true
-  end
-
-
-  # TODO  We might want to remove this when we have implemented
-  #       associations correctly. Since Columns are contained within
-  #       tables, calling save should do something on it's proxy_owner object.
-  describe "not be possible to persist (at least for now...)" do
-    %w(first last all exists? destroy_all).each do |method|
-      it "should not respond to class method #{method}" do
-        Address.should_not respond_to method
-      end
-    end
-
-    %w(
-      create! create reload save save!
-      update_attribute update_attributes update_attributes! touch destroy
-      delete increment! atomic_increment! decrement!
-    ).each do |method|
-      it "should not respond to instance method #{method}" do
-        @address.should_not respond_to method
-      end
-    end
-  end
-end

data/spec/orm/cases/id_factory_spec.rb

@@ -1,92 +0,0 @@
-require 'spec_helper'
-require 'orm/models/person'
-
-describe "id factory" do
-  subject { MassiveRecord::ORM::IdFactory.instance }
-
-  it "should be a singleton" do
-    MassiveRecord::ORM::IdFactory.included_modules.should include(Singleton)
-  end
-
-  describe "#next_for" do
-    describe "dry" do
-      include MockMassiveRecordConnection
-
-      it "should respond to next_for" do
-        subject.should respond_to :next_for
-      end
-
-      it "should use incomming table name if it's a string" do
-        subject.should_receive(:next_id).with(hash_including(:table => "test_table"))
-        subject.next_for "test_table"
-      end
-
-      it "should use incomming table name if it's a symbol" do
-        subject.should_receive(:next_id).with(hash_including(:table => "test_table"))
-        subject.next_for :test_table
-      end
-
-      it "should ask object for it's table name if it responds to that" do
-        Person.should_receive(:table_name).and_return("people")
-        subject.should_receive(:next_id).with(hash_including(:table => "people"))
-        subject.next_for(Person)
-      end
-
-      it "should have class method next_for and delegate it to it's instance" do
-        subject.should_receive(:next_for).with("cars")
-        MassiveRecord::ORM::IdFactory.next_for("cars")
-      end
-    end
-
-
-
-    describe "database" do
-      include SetUpHbaseConnectionBeforeAll
-      include SetTableNamesToTestTable
-
-      after do
-        MassiveRecord::ORM::IdFactory.destroy_all
-      end
-
-      it "should increment start a new sequence on 1" do
-        subject.next_for(Person).should == 1
-      end
-
-      it "should increment value one by one" do
-        5.times do |index|
-          expected_id = index + 1
-          subject.next_for(Person).should == expected_id
-        end
-      end
-
-      it "should maintain ids separate for each table" do
-        3.times { subject.next_for(Person) }
-        subject.next_for("cars").should == 1
-      end
-
-
-      describe "old string representation of integers" do
-        it "increments correctly when value is '1'" do
-          old_ensure = MassiveRecord::ORM::Base.backward_compatibility_integers_might_be_persisted_as_strings
-          MassiveRecord::ORM::Base.backward_compatibility_integers_might_be_persisted_as_strings = true
-
-          subject.next_for(Person)
-
-          # Enter incompatible data, number as string.
-          MassiveRecord::ORM::IdFactory.table.first.tap do |row|
-            row.update_column(
-              MassiveRecord::ORM::IdFactory::COLUMN_FAMILY_FOR_TABLES,
-              Person.table_name,
-              MassiveRecord::ORM::Base.coder.dump(1)
-            )
-            row.save
-          end
-
-          subject.next_for(Person).should eq 2
-
-          MassiveRecord::ORM::Base.backward_compatibility_integers_might_be_persisted_as_strings = old_ensure
-        end
-      end
-    end
-  end
-end