massive_record 0.1.0 → 0.1.1

data/CHANGELOG.md

@@ -0,0 +1,25 @@
+# v0.2.0 (git develop)
+
+
+# v0.1.2 (git master)
+
+
+
+
+
+# v0.1.1
+
+- A ORM record now now if it is read only or not.
+- Added a logger to ORM::Base, which is set to Rails.logger if gem is used with Rails.
+- The database cleaner will no longer destroy tables, only delete all of their contents between tests. Speed tests up a lot.
+- known_attribute_names are now available as instance method as well.
+- If you add a created_at attribute it will be maintained by the ORM with the time the object was created.
+- An adapter (wrapper) row now has an updated_at attribute. ORM objects also responds to updated_at
+- Bugfix: Database cleaner no longer tries to remove tables with same name twice.
+
+
+
+# v0.1.0
+
+- Communication with Hbase via Thrift.
+- Basic ORM capabilities.

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    massive_record (0.1.0)
+    massive_record (0.1.1)
       activemodel
       activesupport
       thrift (>= 0.5.0)
@@ -17,22 +17,19 @@ GEM
     builder (2.1.2)
     diff-lcs (1.1.2)
     i18n (0.5.0)
-    rspec (2.2.0)
-      rspec-core (~> 2.2)
-      rspec-expectations (~> 2.2)
-      rspec-mocks (~> 2.2)
-    rspec-core (2.2.1)
-    rspec-expectations (2.2.0)
+    rspec (2.4.0)
+      rspec-core (~> 2.4.0)
+      rspec-expectations (~> 2.4.0)
+      rspec-mocks (~> 2.4.0)
+    rspec-core (2.4.0)
+    rspec-expectations (2.4.0)
       diff-lcs (~> 1.1.2)
-    rspec-mocks (2.2.0)
+    rspec-mocks (2.4.0)
     thrift (0.5.0)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  activemodel
-  activesupport
   massive_record!
   rspec (>= 2.1.0)
-  thrift (>= 0.5.0)

data/README.md

@@ -53,6 +53,7 @@ Both MassiveRecord::ORM::Table and MassiveRecord::ORM::Column do now have some f
     - Information about changes on attributes.
     - Casting of attributes
     - Serialization of array / hashes
+    - Timestamps like created_at and updated_at. Updated at will always be available, created_at must be defined. See example down:
 
 Tables also have:
     - Persistencey method calls like create, save and destroy (but they do not actually save things to hbase)
@@ -74,6 +75,8 @@ Here is an example of usage, both for Table and Column:
         field :points, :integer, :default => 0
         field :date_of_birth, :date
         field :newsletter, :boolean, :default => false
+
+        timestamps # ..or field :created_at, :time
       end
 
       validates_presence_of :name, :email
@@ -145,7 +148,6 @@ You can, if you'd like, work directly against the adapter.
 ## Planned work
 
 - Rename Wrapper to Adapter, and make it easy to switch from Thrift to another way of communicating with Hbase.
-- Automatically handling time stamps like created_at and updated_at.
 - Associations and embedded objects.
 - Implement other Adapters, for instance using jruby and the Java API.
 

data/lib/massive_record.rb

@@ -15,4 +15,8 @@ require 'massive_record/thrift/hbase'
 require 'massive_record/wrapper/base'
 
 # ORM
-require 'massive_record/orm/base'
+require 'massive_record/orm/base'
+
+if defined?(::Rails) && ::Rails::VERSION::MAJOR == 3
+  require 'massive_record/rails/railtie'
+end

data/lib/massive_record/orm/base.rb

@@ -16,6 +16,7 @@ require 'massive_record/orm/attribute_methods/read'
 require 'massive_record/orm/attribute_methods/dirty'
 require 'massive_record/orm/validations'
 require 'massive_record/orm/callbacks'
+require 'massive_record/orm/timestamps'
 require 'massive_record/orm/persistence'
 
 module MassiveRecord
@@ -23,6 +24,9 @@ module MassiveRecord
     class Base
       include ActiveModel::Conversion
       
+      # Accepts a logger conforming to the interface of Log4r or the default Ruby 1.8+ Logger class,
+      cattr_accessor :logger, :instance_writer => false
+
       # Add a prefix or a suffix to the table name
       # example:
       #
@@ -63,7 +67,7 @@ module MassiveRecord
         self.attributes_raw = attributes_from_field_definition.merge(attributes)
         self.attributes = attributes
         @new_record = true
-        @destroyed = false
+        @destroyed = @readonly = false
 
         _run_initialize_callbacks
       end
@@ -83,7 +87,7 @@ module MassiveRecord
       #   person.name # => 'Alice'
       def init_with(coder)
         @new_record = false
-        @destroyed = false
+        @destroyed = @readonly = false
 
         self.attributes_raw = coder['attributes']
 
@@ -107,9 +111,8 @@ module MassiveRecord
       end
 
 
-
       def inspect
-        attributes_as_string = self.class.known_attribute_names.collect do |attr_name|
+        attributes_as_string = known_attribute_names_for_inspect.collect do |attr_name|
           "#{attr_name}: #{attribute_for_inspect(attr_name)}"
         end.join(', ')
 
@@ -117,18 +120,6 @@ module MassiveRecord
       end
 
 
-      def attribute_for_inspect(attr_name)
-        value = read_attribute(attr_name)
-
-        if value.is_a?(String) && value.length > 50
-          "#{value[0..50]}...".inspect
-        elsif value.is_a?(Date) || value.is_a?(Time)
-          %("#{value.to_s}")
-        else
-          value.inspect
-        end
-      end
-
       def id
         if read_attribute(:id).blank? && respond_to?(:default_id, true)
           @attributes["id"] = default_id
@@ -139,10 +130,41 @@ module MassiveRecord
 
 
 
+      def readonly?
+        !!@readonly
+      end
+
+      def readonly!
+        @readonly = true
+      end
 
 
       private
 
+      #
+      # A place to hook in if you need to add attributes
+      # not known by the attribute schema in the inspect string.
+      # Remember to include a call to super in your module so you
+      # don't break the chain if you override it.
+      # See Timestamps for an example
+      #
+      def known_attribute_names_for_inspect
+        (self.class.known_attribute_names + (super rescue [])).uniq
+      end
+
+      def attribute_for_inspect(attr_name)
+        value = read_attribute(attr_name)
+
+        if value.is_a?(String) && value.length > 50
+          "#{value[0..50]}...".inspect
+        elsif value.is_a?(Date) || value.is_a?(Time)
+          %("#{value.to_s}")
+        else
+          value.inspect
+        end
+      end
+
+
       def next_id
         IdFactory.next_for(self.class).to_s
       end
@@ -159,6 +181,7 @@ module MassiveRecord
       include AttributeMethods::Dirty
       include Validations
       include Callbacks
+      include Timestamps
 
 
       alias [] read_attribute

data/lib/massive_record/orm/config.rb

@@ -16,7 +16,7 @@ module MassiveRecord
           if @@connection.blank?
             @@connection = if !connection_configuration.empty?
                             MassiveRecord::Wrapper::Connection.new(connection_configuration)
-                          elsif defined? Rails
+                          elsif defined? ::Rails
                             MassiveRecord::Wrapper::Base.connection
                           else
                             raise ConnectionConfigurationMissing

data/lib/massive_record/orm/errors.rb

@@ -43,5 +43,14 @@ module MassiveRecord
     # requiring a number to work on.
     class NotNumericalFieldError < MassiveRecordError
     end
+
+    # Raised if you try to assign a variable which can't be set manually,
+    # for instance time stamps
+    class CantBeManuallyAssigned < MassiveRecordError
+    end
+
+    # Raised if you try to save a record which is read only
+    class ReadOnlyRecord < MassiveRecordError
+    end
   end
 end

data/lib/massive_record/orm/persistence.rb

@@ -111,6 +111,7 @@ module MassiveRecord
 
 
       def create_or_update
+        raise ReadOnlyRecord if readonly?
         !!(new_record? ? create : update)
       end
 

data/lib/massive_record/orm/schema/column_family.rb

@@ -85,6 +85,10 @@ module MassiveRecord
           self << Field.new_with_arguments_from_dsl(*args)
         end
 
+        # Internal DSL method
+        def timestamps
+          field :created_at, :time
+        end
 
         # Internal DSL method
         def autoload_fields

data/lib/massive_record/orm/schema/column_interface.rb

@@ -29,6 +29,11 @@ module MassiveRecord
             fields << Field.new_with_arguments_from_dsl(*args)
           end
 
+
+          def timestamps
+            add_field :created_at, :time
+          end
+
           #
           # If you need to add fields dynamically, use this method.
           # It wraps functionality needed to keep the class in a consistent state.

data/lib/massive_record/orm/schema/common_interface.rb

@@ -42,6 +42,10 @@ module MassiveRecord
         def attributes_schema
           self.class.attributes_schema
         end
+
+        def known_attribute_names
+          self.class.known_attribute_names
+        end
       end
     end
   end

data/lib/massive_record/orm/timestamps.rb

@@ -0,0 +1,69 @@
+module MassiveRecord
+  module ORM
+    module Timestamps
+      extend ActiveSupport::Concern
+
+
+      included do
+        before_create :if => :set_created_at_on_create? do
+          raise "created_at must be of type time" if attributes_schema['created_at'].type != :time
+          @attributes['created_at'] = Time.now
+        end
+      end
+
+
+
+      
+      module ClassMethods
+        private
+
+        def transpose_hbase_columns_to_record_attributes(row)
+          attributes = super
+          attributes['updated_at'] = row.updated_at
+          attributes
+        end
+      end
+
+
+
+
+      def updated_at
+        self['updated_at']
+      end
+
+      def write_attribute(attr_name, value)
+        attr_name = attr_name.to_s
+
+        if attr_name == 'updated_at' || (attr_name == 'created_at' && has_created_at?)
+          raise MassiveRecord::ORM::CantBeManuallyAssigned.new("#{attr_name} can't be manually assigned.")
+        end
+
+        super
+      end
+
+
+
+      private
+
+      def update(*)
+        # Not 100% accurat, as we might should re-read the saved row from
+        # the database to fetch exactly the correct updated at time, but
+        # it should do for now as it takes an extra query to check the time stamp.
+        @attributes['updated_at'] = Time.now if updated = super 
+        updated
+      end
+
+      def set_created_at_on_create?
+        has_created_at?
+      end
+
+      def has_created_at?
+        known_attribute_names.include? 'created_at'
+      end
+
+      def known_attribute_names_for_inspect
+        out = (super rescue []) << 'updated_at'
+      end
+    end
+  end
+end

data/lib/massive_record/rails/railtie.rb

@@ -0,0 +1,9 @@
+module MassiveRecord
+  module Rails
+    class Railtie < ::Rails::Railtie
+      initializer "massive_record.logger" do
+        MassiveRecord::ORM::Base.logger = ::Rails.logger
+      end
+    end
+  end
+end

data/lib/massive_record/spec/support/simple_database_cleaner.rb

@@ -1,7 +1,7 @@
 require 'active_support/secure_random'
 
 #
-# So, this module does a couple of things:
+# This module does a couple of things:
 #   1.  Iterates over all tables and adds a prefix to
 #       them so that the classes will be uniq for
 #       the test run.
@@ -14,39 +14,35 @@ module MassiveRecord
       extend ActiveSupport::Concern
 
       included do
-        before(:all) { add_prefix_to_tables }
+        before(:all) { add_suffix_to_tables }
         after(:each) { delete_all_tables }
       end
 
-
-
-
       private
 
+      def add_suffix_to_tables
+        each_orm_class do |klass|
+          table_name_overriden = klass.table_name_overriden
+          klass.reset_table_name_configuration!
 
-
-
-      def add_prefix_to_tables
-        prefix = ActiveSupport::SecureRandom.hex(3)
-        each_orm_class { |klass| klass.table_name_prefix = ["test", prefix, klass.table_name_prefix].reject(&:blank?).join("_") + "_" }
+          klass.table_name = table_name_overriden
+          klass.table_name_suffix = '_test'
+        end
       end
 
       def delete_all_tables
-        each_orm_class_where_table_exists { |klass| klass.table.destroy }
+        tables = MassiveRecord::ORM::Base.connection.tables
+        each_orm_class do |klass|
+          if tables.include? klass.table.name
+            klass.table.all.each(&:destroy) # Don't want to use ORM, as it triggers callbacks etc..
+            tables.delete(klass.table.name)
+          end
+        end
       end
 
-
-
-
-
-
       def each_orm_class
         MassiveRecord::ORM::Table.descendants.each { |klass| yield klass }
       end
-
-      def each_orm_class_where_table_exists
-        MassiveRecord::ORM::Table.descendants.select { |klass| klass.connection.tables.include? klass.table_name }.each { |klass| yield klass }
-      end
     end
   end
 end

data/lib/massive_record/version.rb

@@ -1,3 +1,3 @@
 module MassiveRecord
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/lib/massive_record/wrapper/base.rb

@@ -13,7 +13,7 @@ module MassiveRecord
     class Base
       
       def self.config
-        config = YAML.load_file(Rails.root.join('config', 'hbase.yml'))[Rails.env]
+        config = YAML.load_file(::Rails.root.join('config', 'hbase.yml'))[::Rails.env]
         { :host => config['host'], :port => config['port'] }        
       end
 
@@ -25,4 +25,4 @@ module MassiveRecord
       
     end  
   end
-end
+end

data/lib/massive_record/wrapper/row.rb

@@ -41,7 +41,7 @@ module MassiveRecord
       def values
         @columns.inject({"id" => id}) {|h, (column_name, cell)| h[column_name] = cell.deserialize_value; h}
       end
-    
+
       def values=(data)
         @values = {}
         update_columns(data)
@@ -165,6 +165,9 @@ module MassiveRecord
         self
       end
     
+      def updated_at
+        columns.values.collect(&:created_at).max
+      end
     end  
   end
 end

data/lib/massive_record/wrapper/scanner.rb

@@ -3,7 +3,7 @@ module MassiveRecord
     class Scanner
       
       attr_accessor :connection, :table_name, :column_family_names, :opened_scanner
-      attr_accessor :start_key, :created_at, :limit
+      attr_accessor :start_key, :offset_key, :created_at, :limit
       attr_accessor :formatted_column_family_names, :column_family_names
       
       def initialize(connection, table_name, column_family_names, opts = {})
@@ -13,15 +13,20 @@ module MassiveRecord
         @column_family_names = opts[:columns] unless opts[:columns].nil?
         @formatted_column_family_names = column_family_names.collect{|n| "#{n.split(":").first}:"}
         @start_key = opts[:start_key].to_s
+        @offset_key = opts[:offset_key].to_s
         @created_at = opts[:created_at].to_s
         @limit = opts[:limit] || 10
       end
       
+      def key
+        start_key.empty? ? offset_key : start_key
+      end
+      
       def open
         if created_at.empty?
-          self.opened_scanner = connection.scannerOpen(table_name, start_key, formatted_column_family_names)
+          self.opened_scanner = connection.scannerOpen(table_name, key, formatted_column_family_names)
         else
-          self.opened_scanner = connection.scannerOpenTs(table_name, start_key, formatted_column_family_names, created_at)
+          self.opened_scanner = connection.scannerOpenTs(table_name, key, formatted_column_family_names, created_at)
         end
       end
       
@@ -38,7 +43,13 @@ module MassiveRecord
       end
       
       def populate_rows(results)
-        results.collect{|result| populate_row(result)}
+        results.collect do |result|
+          if offset_key.empty?
+            populate_row(result) unless result.row.match(/^#{start_key}/).nil?
+          else
+            populate_row(result)            
+          end
+        end.select{|r| !r.nil?}
       end
       
       def populate_row(result)

data/lib/massive_record/wrapper/table.rb

@@ -86,7 +86,8 @@ module MassiveRecord
       
       def format_options_for_scanner(opts = {})
         {
-          :start_key  => opts[:start], 
+          :start_key  => opts[:start],
+          :offset_key => opts[:offset],
           :created_at => opts[:created_at],
           :columns    => opts[:select], # list of column families to fetch from hbase
           :limit      => opts[:limit] || opts[:batch_size]

data/spec/orm/cases/base_spec.rb

@@ -173,4 +173,27 @@ describe MassiveRecord::ORM::Base do
       @test_object.foo.should == "new_value"
     end
   end
+
+
+  describe "logger" do
+    it "should respond to logger" do
+      MassiveRecord::ORM::Base.should respond_to :logger
+    end
+
+    it "should respond to logger=" do
+      MassiveRecord::ORM::Base.should respond_to :logger=
+    end
+  end
+
+  describe "read only" do
+    it "should not be read only by default" do
+      TestClass.new.should_not be_readonly
+    end
+
+    it "should be read only if asked to" do
+      test = TestClass.new
+      test.readonly!
+      test.should be_readonly
+    end
+  end
 end

data/spec/orm/cases/config_spec.rb

@@ -51,7 +51,7 @@ describe "configuration" do
     describe "under Rails" do
       before do
         TestClass.connection_configuration = {}
-        module Rails; end
+        module ::Rails; end
         MassiveRecord::Wrapper::Base.stub!(:connection).and_return(@mock_connection)
       end
       

data/spec/orm/cases/persistence_spec.rb

@@ -476,4 +476,22 @@ describe "persistence" do
       end
     end
   end
+
+  describe "read only objects" do
+    include MockMassiveRecordConnection
+
+    it "should raise an error if new record is read only and you try to save it" do
+      person = Person.new :id => "id1", :name => "Thorbjorn", :age => 29
+      person.readonly!
+      lambda { person.save }.should raise_error MassiveRecord::ORM::ReadOnlyRecord
+    end
+
+    it "should raise an error if record is read only and you try to save it" do
+      person = Person.create :id => "id1", :name => "Thorbjorn", :age => 29
+      person.should be_persisted
+
+      person.readonly!
+      lambda { person.save }.should raise_error MassiveRecord::ORM::ReadOnlyRecord
+    end
+  end
 end

data/spec/orm/cases/timestamps_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+require 'orm/models/person'
+require 'orm/models/person_with_timestamps'
+
+describe "timestamps" do
+  include CreatePersonBeforeEach
+
+  before do
+    @person = Person.first
+  end
+
+
+  describe "#updated_at" do
+    it "should have updated at equal to nil on new records" do
+      Person.new.updated_at.should be_nil
+    end
+
+    it "should not be possible to set updated at" do
+      lambda { @person.updated_at = Time.now }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+      lambda { @person['updated_at'] = Time.now }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+      lambda { @person.write_attribute(:updated_at, Time.now) }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+    end
+
+    it "should have updated at on a persisted record" do
+      @person.updated_at.should be_a_kind_of Time
+    end
+
+    it "should be included in the list of known_attribute_names_for_inspect" do
+      @person.send(:known_attribute_names_for_inspect).should include 'updated_at'
+    end
+
+    it "should include updated_at in inspect" do
+      @person.inspect.should include(%q{updated_at:})
+    end
+
+    it "should be updated after a save" do
+      sleep(1)
+
+      updated_at_was = @person.updated_at
+      @person.update_attribute :name, "Should Give Me New Updated At"
+
+      @person.updated_at.should_not == updated_at_was
+    end
+
+    it "should not be updated after a save which failed" do
+      sleep(1)
+
+      updated_at_was = @person.updated_at
+      @person.name = nil
+
+      @person.should_not be_valid
+
+      @person.updated_at.should == updated_at_was
+    end
+  end
+
+  describe "#created_at" do
+    before do
+      @person_with_timestamps = PersonWithTimestamps.create! :name => "John Doe", :email => "john@base.com", :age => "20"
+    end
+
+    it "should have created at" do
+      @person_with_timestamps.should be_set_created_at_on_create
+    end
+
+    it "should not have created at on create if model does not have created at" do
+      @person.should_not be_set_created_at_on_create
+    end
+
+    it "should have updated at equal to nil on new records" do
+      PersonWithTimestamps.new.created_at.should be_nil
+    end
+
+    it "should not be possible to set updated at" do
+      lambda { @person_with_timestamps.created_at = Time.now }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+      lambda { @person_with_timestamps['created_at'] = Time.now }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+      lambda { @person_with_timestamps.write_attribute(:created_at, Time.now) }.should raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+    end
+
+    it "should not raise cant-set-error if object has no timestamps" do
+      lambda { @person.created_at = Time.now }.should_not raise_error MassiveRecord::ORM::CantBeManuallyAssigned
+    end
+
+    it "should have created_at on a persisted record" do
+      @person_with_timestamps.created_at.should be_a_kind_of Time
+    end
+
+    it "should not alter created at on update" do
+      created_at_was = @person_with_timestamps.created_at
+
+      sleep(1)
+
+      @person_with_timestamps.update_attribute :name, @person_with_timestamps.name + "NEW"
+      @person_with_timestamps.created_at.should == created_at_was
+    end
+
+    it "should be included in the list of known_attribute_names_for_inspect" do
+      @person_with_timestamps.send(:known_attribute_names_for_inspect).should include 'created_at'
+    end
+
+    it "should include created_at in inspect" do
+      @person_with_timestamps.inspect.should include(%q{created_at:})
+    end
+
+    it "should not include created_at if object does not have it" do
+      @person.send(:known_attribute_names_for_inspect).should_not include 'created_at'
+    end
+
+    it "should raise error if created_at is not time" do
+      PersonWithTimestamps.attributes_schema['created_at'].type = :string
+
+      lambda { PersonWithTimestamps.create! }.should raise_error "created_at must be of type time"
+
+      PersonWithTimestamps.attributes_schema['created_at'].type = :time
+    end
+  end
+end

data/spec/orm/models/person_with_timestamps.rb

@@ -0,0 +1,19 @@
+class PersonWithTimestamps < MassiveRecord::ORM::Table
+  column_family :info do
+    field :name
+    field :email
+    field :age, :integer
+    field :points, :integer, :default => 1, :column => :pts
+    field :date_of_birth, :date
+    field :status, :boolean, :default => false
+    field :addresses, :hash, :default => {}
+
+    field :created_at, :time
+  end
+
+  private
+
+  def default_id
+    next_id
+  end
+end

data/spec/orm/schema/column_interface_spec.rb

@@ -64,6 +64,15 @@ describe MassiveRecord::ORM::Schema::TableInterface do
     TestColumnInterface.known_attribute_names.should include("name", "age")
   end
 
+  it "should make known_attribute_names readable for instances" do
+    class TestColumnInterface
+      field :name, :string
+    end
+
+    TestColumnInterface.new.known_attribute_names.should include('name')
+  end
+
+
   it "should give us default attributes from schema" do
     class TestColumnInterface
       field :name
@@ -75,6 +84,18 @@ describe MassiveRecord::ORM::Schema::TableInterface do
     defaults["age"].should == 1
   end
 
+  describe "timestamps" do
+    before do
+      class TestColumnInterface
+        timestamps
+      end
+    end
+
+    it "should have a created_at time field" do
+      TestColumnInterface.attributes_schema['created_at'].type.should == :time
+    end
+  end
+
 
   describe "dynamically adding a field" do
     it "should be possible to dynamically add a field" do

data/spec/orm/schema/table_interface_spec.rb

@@ -87,6 +87,16 @@ describe MassiveRecord::ORM::Schema::TableInterface do
     TestInterface.new.attributes_schema["name"].type.should == :string
   end
 
+  it "should make known_attribute_names readable for instances" do
+    class TestInterface
+      column_family :info do
+        field :name
+      end
+    end
+
+    TestInterface.new.known_attribute_names.should include('name')
+  end
+
   it "should not be shared amonb subclasses" do
     class TestInterface
       column_family :info do
@@ -98,6 +108,20 @@ describe MassiveRecord::ORM::Schema::TableInterface do
     TestInterfaceSubClass.column_families.should be_nil
   end
 
+  describe "timestamps" do
+    before do
+      class TestInterface
+        column_family :info do
+          timestamps
+        end
+      end
+    end
+
+    it "should have a created_at time field" do
+      TestInterface.attributes_schema['created_at'].type.should == :time
+    end
+  end
+
 
   describe "dynamically adding a field" do
     it "should be possible to dynamically add a field" do

data/spec/wrapper/cases/table_spec.rb

@@ -83,7 +83,7 @@ describe MassiveRecord::Wrapper::Table do
         @table.all(:limit => 1, :select => ["info"]).first.column_families.should == ["info"]
         @table.find("ID1", :select => ["info"]).column_families.should == ["info"]
       end
-      
+
       it "should update row values" do
         row = @table.first
         row.values["info:first_name"].should eql("John")
@@ -100,6 +100,46 @@ describe MassiveRecord::Wrapper::Table do
         row.update_columns({ :info => { :first_name => "Bob" } })
         row.save.should be_true
       end
+
+      it "should have a updated_at for a row" do
+        row = @table.first
+        row.updated_at.should be_a_kind_of Time
+      end
+
+      it "should have an updated_at for the row which is taken from the last updated attribute" do
+        sleep(1)
+        row = @table.first
+        row.update_columns({ :info =>  { :first_name => "New Bob" } })
+        row.save
+
+        sleep(1)
+
+        row = @table.first
+
+        row.columns["info:first_name"].created_at.should == row.updated_at
+      end
+
+      it "should have a new updated at for a row" do
+        row = @table.first
+        updated_at_was = row.updated_at
+
+        sleep(1)
+
+        row.update_columns({ :info =>  { :first_name => "Bob" } })
+        row.save
+
+        row = @table.first
+        
+        updated_at_was.should_not == row.updated_at
+      end
+
+      it "should return nil if no cells has been created" do
+        row = MassiveRecord::Wrapper::Row.new
+        row.updated_at.should be_nil
+      end
+
+      
+      
       
       it "should merge data" do
         row = @table.first
@@ -185,7 +225,7 @@ describe MassiveRecord::Wrapper::Table do
         
         @table.all.size.should == 5
       end
-            
+      
       it "should find rows" do
         ids_list = [["ID1"], ["ID1", "ID2", "ID3"]]
         ids_list.each do |ids|
@@ -194,26 +234,39 @@ describe MassiveRecord::Wrapper::Table do
           end
         end
       end
-      
+    
       it "should collect 5 IDs" do
         @table.all.collect(&:id).should eql(1.upto(5).collect{|i| "ID#{i}"})
       end
-      
+    
       it "should iterate through a collection of rows" do
         @table.all.each do |row|
           row.id.should_not be_nil
         end
       end
-      
+    
       it "should iterate through a collection of rows using a batch process" do
         group_number = 0
-        @table.find_in_batches(:batch_size => 2, :start => "ID2", :select => ["info"]) do |group|
+        @table.find_in_batches(:batch_size => 2, :select => ["info"]) do |group|
           group_number += 1
           group.each do |row|
             row.id.should_not be_nil
           end
         end        
-        group_number.should == 2
+        group_number.should == 3
+      end
+    
+      it "should find 1 row using the :start option" do
+        @table.all(:start => "ID1").size.should == 1
+      end
+    
+      it "should find 5 rows using the :start option" do
+        @table.all(:start => "ID").size.should == 5
+      
+      end
+    
+      it "should find 4 rows using the :offset option" do
+        @table.all(:offset => "ID2").size.should == 4
       end
       
       it "should exists in the database" do

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 0
-  version: 0.1.0
+  - 1
+  version: 0.1.1
 platform: ruby
 authors: 
 - Companybook
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-20 00:00:00 +01:00
+date: 2011-02-04 00:00:00 +01:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -85,6 +85,7 @@ files:
 - .autotest
 - .gitignore
 - .rspec
+- CHANGELOG.md
 - Gemfile
 - Gemfile.lock
 - Manifest
@@ -115,7 +116,9 @@ files:
 - lib/massive_record/orm/schema/fields.rb
 - lib/massive_record/orm/schema/table_interface.rb
 - lib/massive_record/orm/table.rb
+- lib/massive_record/orm/timestamps.rb
 - lib/massive_record/orm/validations.rb
+- lib/massive_record/rails/railtie.rb
 - lib/massive_record/spec/support/simple_database_cleaner.rb
 - lib/massive_record/thrift/hbase.rb
 - lib/massive_record/thrift/hbase_constants.rb
@@ -146,9 +149,11 @@ files:
 - spec/orm/cases/id_factory_spec.rb
 - spec/orm/cases/persistence_spec.rb
 - spec/orm/cases/table_spec.rb
+- spec/orm/cases/timestamps_spec.rb
 - spec/orm/cases/validation_spec.rb
 - spec/orm/models/address.rb
 - spec/orm/models/person.rb
+- spec/orm/models/person_with_timestamps.rb
 - spec/orm/models/test_class.rb
 - spec/orm/schema/column_families_spec.rb
 - spec/orm/schema/column_family_spec.rb
@@ -210,9 +215,11 @@ test_files:
 - spec/orm/cases/id_factory_spec.rb
 - spec/orm/cases/persistence_spec.rb
 - spec/orm/cases/table_spec.rb
+- spec/orm/cases/timestamps_spec.rb
 - spec/orm/cases/validation_spec.rb
 - spec/orm/models/address.rb
 - spec/orm/models/person.rb
+- spec/orm/models/person_with_timestamps.rb
 - spec/orm/models/test_class.rb
 - spec/orm/schema/column_families_spec.rb
 - spec/orm/schema/column_family_spec.rb