flex_columns 1.0.0

data/spec/flex_columns/system/bulk_system_spec.rb

@@ -0,0 +1,153 @@
+require 'flex_columns'
+require 'flex_columns/helpers/system_helpers'
+
+describe "FlexColumns bulk operations" do
+  include FlexColumns::Helpers::SystemHelpers
+
+  before :each do
+    @dh = FlexColumns::Helpers::DatabaseHelper.new
+    @dh.setup_activerecord!
+
+    create_standard_system_spec_tables!
+  end
+
+  after :each do
+    drop_standard_system_spec_tables!
+  end
+
+  before :each do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes do
+        field :aaa, :string
+        field :bbb, :integer
+      end
+    end
+  end
+
+  it "should return #to_stored_data correctly on a text column, and return the exact same thing for #to_json" do
+    user = ::User.new
+    user.name = "User 1"
+    user.aaa = "aaa#{rand(1_000_000)}"
+    user.bbb = rand(1_000_000)
+
+    json = user.user_attributes.to_stored_data
+    json.class.should be(String)
+
+    parsed = JSON.parse(json)
+    parsed.keys.sort.should == %w{aaa bbb}.sort
+    parsed['aaa'].should == user.aaa
+
+    user.user_attributes.to_json.should == json
+  end
+
+  context "with a binary column" do
+    before :each do
+      migrate do
+        drop_table :flexcols_spec_users rescue nil
+        create_table :flexcols_spec_users do |t|
+          t.string :name, :null => false
+          t.binary :user_attributes
+        end
+      end
+
+      ::User.reset_column_information
+
+      define_model_class(:User, 'flexcols_spec_users') do
+        flex_column :user_attributes do
+          field :aaa, :string
+          field :bbb, :integer
+        end
+      end
+    end
+
+    it "should return #to_stored_data correctly on a binary column, uncompressed, but return JSON separately with #to_json" do
+      user = ::User.new
+      user.name = "User 1"
+      user.aaa = "aaa#{rand(1_000_000)}"
+      user.bbb = rand(1_000_000)
+
+      stored_data = user.user_attributes.to_stored_data
+      stored_data.class.should be(String)
+      stored_data.should match(/^FC:01,0,/)
+
+      stored_data =~ /^FC:01,0,(.*)$/i
+      json = $1
+      parsed = JSON.parse(json)
+      parsed.keys.sort.should == %w{aaa bbb}.sort
+      parsed['aaa'].should == user.aaa
+
+      user.user_attributes.to_json.should == json
+    end
+
+    it "should return #to_stored_data correctly on a binary column, compressed, but return JSON separately with #to_json" do
+      user = ::User.new
+      user.name = "User 1"
+      user.aaa = "aaa#{rand(1_000_000)}" * 1_000
+      user.bbb = rand(1_000_000)
+
+      stored_data = user.user_attributes.to_stored_data
+      stored_data.class.should be(String)
+      stored_data.should match(/^FC:01,1,/)
+
+      compressed = stored_data[8..-1]
+
+      require 'stringio'
+      stream = StringIO.new(compressed, "r")
+      reader = Zlib::GzipReader.new(stream)
+      json = reader.read
+
+      parsed = JSON.parse(json)
+      parsed.keys.sort.should == %w{aaa bbb}.sort
+      parsed['aaa'].should == user.aaa
+
+      user.user_attributes.to_json.should == json
+    end
+  end
+
+  it "should be able to instantiate fields without an ActiveRecord model, and then serialize them again" do
+    users = [ ]
+    10.times do |i|
+      user = ::User.new
+      user.name = "User #{i}"
+      user.aaa = "aaa#{rand(1_000_000)}"
+      user.bbb = rand(1_000_000)
+      user.save!
+
+      users << user
+    end
+
+    json_blobs = [ ]
+    ::User.connection.select_all("SELECT id, user_attributes FROM flexcols_spec_users ORDER BY id ASC").each do |row|
+      json_blobs << row['user_attributes']
+    end
+
+    as_objects = ::User.create_flex_objects_from(:user_attributes, json_blobs)
+    as_objects.each_with_index do |object, i|
+      user = users[i]
+
+      object.aaa.should == user.aaa
+      object.bbb.should == user.bbb
+
+      object.bbb = "cannot-validate"
+      object.valid?.should_not be
+      object.errors.keys.should == [ :bbb ]
+      object.errors[:bbb].length.should == 1
+      object.errors[:bbb][0].should match(/is not a number/i)
+    end
+
+    json_blobs.each_with_index do |json_blob, i|
+      object = ::User.create_flex_object_from(:user_attributes, json_blob)
+
+      user = users[i]
+
+      object.aaa.should == user.aaa
+      object.bbb.should == user.bbb
+
+      object.bbb = "cannot-validate"
+      object.valid?.should_not be
+      object.errors.keys.should == [ :bbb ]
+      object.errors[:bbb].length.should == 1
+      object.errors[:bbb][0].should match(/is not a number/i)
+    end
+  end
+end

data/spec/flex_columns/system/compression_system_spec.rb

@@ -0,0 +1,218 @@
+require 'flex_columns'
+require 'flex_columns/helpers/system_helpers'
+
+describe "FlexColumns compression operations" do
+  include FlexColumns::Helpers::SystemHelpers
+
+  before :each do
+    @dh = FlexColumns::Helpers::DatabaseHelper.new
+    @dh.setup_activerecord!
+
+    migrate do
+      drop_table :flexcols_spec_users rescue nil
+      create_table :flexcols_spec_users do |t|
+        t.string :name, :null => false
+        t.binary :user_attributes
+      end
+    end
+
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes do
+        field :foo
+        field :bar
+      end
+    end
+
+    define_model_class(:UserBackdoor, 'flexcols_spec_users') { }
+  end
+
+  after :each do
+    migrate do
+      drop_table :flexcols_spec_users rescue nil
+    end
+  end
+
+  it "should not add a header, and should not compress data, if passed :header => false" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :header => false do
+        field :foo
+        field :bar
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = "foo1"
+    user.bar = "bar1"
+    user.save!
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    parsed = JSON.parse(data)
+    parsed.keys.sort.should == %w{foo bar}.sort
+
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = "foo" * 10_000
+    user.bar = "bar1"
+    user.save!
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.length.should > 30_000
+    parsed = JSON.parse(data)
+    parsed.keys.sort.should == %w{foo bar}.sort
+  end
+
+  it "should not compress short data" do
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'foo1'
+    user.bar = 'bar1'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'foo1'
+    user_again.bar.should == 'bar1'
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.should match(/foo1/)
+    data.should match(/bar1/)
+  end
+
+  it "should compress long data" do
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'foo' * 1000
+    user.bar = 'bar1'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'foo' * 1000
+    user_again.bar.should == 'bar1'
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.length.should < 1000
+    data.should_not match(/foo/)
+    data.should_not match(/bar/)
+  end
+
+  it "should read compressed data fine, even if told not to compress new data" do
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'foo' * 1000
+    user.bar = 'bar1'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'foo' * 1000
+    user_again.bar.should == 'bar1'
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.length.should < 1000
+    data.should_not match(/foo/)
+    data.should_not match(/bar/)
+
+    define_model_class(:User2, 'flexcols_spec_users') do
+      flex_column :user_attributes, :compress => false do
+        field :foo
+        field :bar
+      end
+    end
+
+    user2 = ::User2.find(user.id)
+    user.foo.should == 'foo' * 1000
+    user.bar.should == 'bar1'
+  end
+
+  it "should not compress long data, if asked not to" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :compress => false do
+        field :foo
+        field :bar
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'foo' * 1000
+    user.bar = 'bar1'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'foo' * 1000
+    user_again.bar.should == 'bar1'
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.length.should >= 3000
+    data.should match(/foofoofoofoo/)
+    data.should match(/bar1/)
+  end
+
+  it "should not compress data if the compressed version is bigger" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :compress => 1 do
+        field :foo
+        field :bar
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'f'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'f'
+    user_again.bar.should be_nil
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.should match(/^FC:01,0,\{/i)
+  end
+
+  it "should not compress data under a certain limit, if asked to" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :compress => 10_000 do
+        field :foo
+        field :bar
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+    user.foo = 'foo' * 1000
+    user.bar = 'bar1'
+    user.save!
+
+    user_again = ::User.find(user.id)
+    user_again.foo.should == 'foo' * 1000
+    user_again.bar.should == 'bar1'
+
+    user_bd = ::UserBackdoor.find(user.id)
+    data = user_bd.user_attributes
+    data.length.should >= 3000
+    data.should match(/foofoofoofoo/)
+    data.should match(/bar1/)
+
+    user2 = ::User.new
+    user2.name = 'User 1'
+    user2.foo = 'foo' * 10_000
+    user2.bar = 'bar1'
+    user2.save!
+
+    user2_again = ::User.find(user2.id)
+    user2_again.foo.should == 'foo' * 10_000
+    user2_again.bar.should == 'bar1'
+
+    user2_bd = ::UserBackdoor.find(user2.id)
+    data = user2_bd.user_attributes
+    data.length.should < 10_000
+    data.should_not match(/foofoofoofoo/)
+    data.should_not match(/bar1/)
+  end
+end

data/spec/flex_columns/system/custom_methods_system_spec.rb

@@ -0,0 +1,120 @@
+require 'flex_columns'
+require 'flex_columns/helpers/system_helpers'
+
+describe "FlexColumns custom-methods operations" do
+  include FlexColumns::Helpers::SystemHelpers
+
+  before :each do
+    @dh = FlexColumns::Helpers::DatabaseHelper.new
+    @dh.setup_activerecord!
+
+    create_standard_system_spec_tables!
+  end
+
+  after :each do
+    drop_standard_system_spec_tables!
+  end
+
+  it "should allow defining methods that are present on both the attributes class and the enclosing class" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes do
+        field :number_of_emails_sent
+
+        def increment_number_of_emails_sent
+          self.number_of_emails_sent += 1
+        end
+
+        def change_number_of_emails_sent(return_value)
+          self.number_of_emails_sent = yield number_of_emails_sent
+          return_value
+        end
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+
+    user.user_attributes.number_of_emails_sent = 15
+    user.user_attributes.increment_number_of_emails_sent
+    user.user_attributes.number_of_emails_sent.should == 16
+    (user.user_attributes.change_number_of_emails_sent('abc') { |x| x - 5 }).should == 'abc'
+    user.user_attributes.number_of_emails_sent.should == 11
+
+    user.number_of_emails_sent.should == 11
+    user.increment_number_of_emails_sent
+    user.number_of_emails_sent.should == 12
+    (user.change_number_of_emails_sent('abc') { |x| x - 5 }).should == 'abc'
+    user.number_of_emails_sent.should == 7
+  end
+
+  it "should not delegate methods if told not to" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :delegate => false do
+        field :number_of_emails_sent
+
+        def increment_number_of_emails_sent
+          self.number_of_emails_sent += 1
+        end
+
+        def change_number_of_emails_sent(return_value)
+          self.number_of_emails_sent = yield number_of_emails_sent
+          return_value
+        end
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+
+    user.respond_to?(:number_of_emails_sent).should_not be
+    user.respond_to?(:increment_number_of_emails_sent).should_not be
+    user.respond_to?(:change_number_of_emails_sent).should_not be
+  end
+
+  it "should delegate methods privately if told to" do
+    define_model_class(:User, 'flexcols_spec_users') do
+      flex_column :user_attributes, :delegate => :private do
+        field :number_of_emails_sent
+
+        def increment_number_of_emails_sent
+          self.number_of_emails_sent += 1
+        end
+
+        def change_number_of_emails_sent(return_value)
+          self.number_of_emails_sent = yield number_of_emails_sent
+          return_value
+        end
+      end
+
+      def nes=(x)
+        self.number_of_emails_sent = x
+      end
+
+      def nes
+        number_of_emails_sent
+      end
+
+      def ies
+        increment_number_of_emails_sent
+      end
+
+      def cnes(rv, &block)
+        change_number_of_emails_sent(rv, &block)
+      end
+    end
+
+    user = ::User.new
+    user.name = 'User 1'
+
+    user.respond_to?(:number_of_emails_sent).should_not be
+    user.respond_to?(:increment_number_of_emails_sent).should_not be
+    user.respond_to?(:change_number_of_emails_sent).should_not be
+
+    user.nes = 10
+    user.nes.should == 10
+    user.ies
+    user.nes.should == 11
+    (user.cnes('abc') { |x| x - 5 }).should == 'abc'
+    user.nes.should == 6
+  end
+end