ribs 0.0.1 → 0.0.2

data/test/artist_spec.rb

@@ -1,25 +1,27 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 
 class Artist
-  Ribs!
+  Ribs! :identity_map => false
+  attr_accessor :id
+  attr_accessor :name
 end
 
 describe Artist do 
   it "should be able to find all artists" do 
-    Artist.find(:all).length.should == 3
+    R(Artist).all.length.should == 3
   end
 
   it "should return correct value and type for id property" do 
-    Artist.find(:all).map { |a| a.id }.sort.should == [1,2,3]
+    R(Artist).all.map { |a| a.id }.sort.should == [1,2,3]
   end
   
   it "should return correct value and type for name property" do 
-    Artist.find(:all).map { |a| a.name }.sort.should == ["David Bowie","New Model Army","Public Image Ltd"]
+    R(Artist).all.map { |a| a.name }.sort.should == ["David Bowie","New Model Army","Public Image Ltd"]
   end
   
   it "should only have the appropriate methods defined" do 
     methods = (Artist.instance_methods - Object.instance_methods).sort
-    methods.should == ['__ribs_meat', 'destroy!', 'id=', 'name', 'name=', 'save']
+    methods.should == ['id=', 'name', 'name=']
   end
   
   it "should be possible to create a new instance by setting properties" do 
@@ -27,8 +29,8 @@ describe Artist do
       artist = Artist.new
       artist.id = 4
       artist.name = "Assemblage 23"
-      artist.save
-      artist2 = Artist.find(4)
+      R(artist).save
+      artist2 = R(Artist).get(4)
       artist2.should_not be_nil
       artist2.id.should == 4
       artist2.name.should == "Assemblage 23"
@@ -39,9 +41,9 @@ describe Artist do
 
   it "should be possible to create a new instance by giving properties to new" do 
     begin
-      artist = Artist.new :id => 4, :name => "Assemblage 23"
-      artist.save
-      artist2 = Artist.find(4)
+      artist = R(Artist).new :id => 4, :name => "Assemblage 23"
+      R(artist).save
+      artist2 = R(Artist).get(4)
       artist2.should_not be_nil
       artist2.id.should == 4
       artist2.name.should == "Assemblage 23"
@@ -52,8 +54,8 @@ describe Artist do
 
   it "should be possible to create a new instance by using create" do 
     begin
-      Artist.create :id => 4, :name => "Assemblage 23"
-      artist = Artist.find(4)
+      R(Artist).create :id => 4, :name => "Assemblage 23"
+      artist = R(Artist).get(4)
       artist.should_not be_nil
       artist.id.should == 4
       artist.name.should == "Assemblage 23"
@@ -64,11 +66,11 @@ describe Artist do
   
   it "should be possible to update name property on existing bean" do 
     begin
-      artist = Artist.find(2)
+      artist = R(Artist).get(2)
       artist.name = "U2"
-      Artist.find(2).name.should == "New Model Army"
-      artist.save
-      Artist.find(2).name.should == "U2"
+      R(Artist).get(2).name.should == "New Model Army"
+      R(artist).save
+      R(Artist).get(2).name.should == "U2"
     ensure
       reset_database!
     end
@@ -76,8 +78,8 @@ describe Artist do
 
   it "should be possible to delete existing bean" do 
     begin
-      Artist.find(2).destroy!
-      Artist.find(2).should be_nil
+      R(R(Artist).get(2)).destroy!
+      R(Artist).get(2).should be_nil
     ensure
       reset_database!
     end
@@ -85,8 +87,8 @@ describe Artist do
 
   it "should be possible to delete existing bean by id" do 
     begin
-      Artist.destroy(2)
-      Artist.find(2).should be_nil
+      R(Artist).destroy(2)
+      R(Artist).get(2).should be_nil
     ensure
       reset_database!
     end

data/test/define_accessors_spec.rb

@@ -0,0 +1,27 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class Person;end
+
+R(Person).define_accessors
+
+describe "define_accessors" do 
+  it "should define id accessors" do 
+    Person.instance_methods.should include("id")
+    Person.instance_methods.should include("id=")
+  end
+
+  it "should define given_name accessors" do 
+    Person.instance_methods.should include("given_name")
+    Person.instance_methods.should include("given_name=")
+  end
+  
+  it "should define sur_name accessors" do 
+    Person.instance_methods.should include("sur_name")
+    Person.instance_methods.should include("sur_name=")
+  end
+
+  it "should define age accessors" do 
+    Person.instance_methods.should include("age")
+    Person.instance_methods.should include("age=")
+  end
+end

data/test/define_model_spec.rb

@@ -0,0 +1,41 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+Ribs::define_model :Address do |rib|
+  rib.zip_code :column => :zip
+end
+
+describe "define_model" do 
+  it "should create a class" do 
+    Address.class.should == Class
+  end
+  
+  it "should create all the needed properties" do 
+    Address.instance_methods.should include("id")
+    Address.instance_methods.should include("id=")
+    Address.instance_methods.should include("street")
+    Address.instance_methods.should include("street=")
+    Address.instance_methods.should include("postal")
+    Address.instance_methods.should include("postal=")
+    Address.instance_methods.should include("zip_code")
+    Address.instance_methods.should include("zip_code=")
+    Address.instance_methods.should_not include("zip")
+    Address.instance_methods.should_not include("zip=")
+    Address.instance_methods.should include("country")
+    Address.instance_methods.should include("country=")
+  end
+  
+  it "should be a working Ribs class" do 
+    R(Address).create :id => 1, :street => "foobar 42"
+    all = R(Address).all
+    all.length.should == 1
+    all[0].id.should == 1
+    all[0].street.should == "foobar 42"
+    all[0].zip_code.should be_nil
+    R(Address).get(1).street.should == all[0].street
+    all[0].street = "foobar 43"
+    R(all[0]).save
+    R(Address).get(1).street.should == "foobar 43"
+    R(Address).destroy(1)
+    R(Address).all.should be_empty
+  end
+end

data/test/identity_map_spec.rb

@@ -0,0 +1,63 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class IdentityMapAddress
+  Ribs! :table => :address 
+end
+
+R(IdentityMapAddress).define_accessors
+
+class IdentityMapPerson
+  Ribs! :identity_map => false, :table => :person
+end
+
+R(IdentityMapPerson).define_accessors
+
+describe "Identity map" do 
+  describe "(disabled)" do 
+    before :all do
+      R(IdentityMapPerson).create :id => 1, :sur_name => "DeWhite", :age => 23
+    end
+    
+    after :all do 
+      R(IdentityMapPerson).destroy 1
+    end
+
+    it "should return different objects with get" do 
+      R(IdentityMapPerson).get(1).should_not == R(IdentityMapPerson).get(1)
+    end
+    it "should return different objects with all" do 
+      R(IdentityMapPerson).all[0].should_not == R(IdentityMapPerson).all[0]
+    end
+    it "should return different objects with a combination of get and all" do 
+      R(IdentityMapPerson).all[0].should_not == R(IdentityMapPerson).get(1)
+    end
+  end
+  
+  describe "(enabled)" do 
+    before :each do
+      @@id ||= 0
+      @@id += 1
+      R(IdentityMapAddress).create :id => @@id
+    end
+    
+    after :each do
+      R(IdentityMapAddress).destroy @@id
+    end
+    
+    it "should return same objects with get" do 
+      p [:BLARG, @@id]
+      R(IdentityMapAddress).get(@@id).should == R(IdentityMapAddress).get(@@id)
+    end
+    it "should return same objects with all" do 
+      R(IdentityMapAddress).all[0].should == R(IdentityMapAddress).all[0]
+    end
+    it "should return same objects with a combination of get -> all" do 
+      obj = R(IdentityMapAddress).all[0]
+      obj.should == R(IdentityMapAddress).get(@@id)
+    end
+    it "should return same objects with a combination of all -> get" do 
+      obj = R(IdentityMapAddress).get(@@id)
+      R(IdentityMapAddress).all[0].should == obj
+    end
+  end
+end

data/test/repository_spec.rb

@@ -0,0 +1,79 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class FakeModel;end
+class FakeModel::FakeSecondModel;end
+
+describe Ribs::Repository do 
+  before :all do 
+    R(FakeModel)
+    R(FakeModel::FakeSecondModel)
+
+    R(FakeModel, :flarg)
+    R(FakeModel::FakeSecondModel, :flurg)
+  end
+  
+  it 'should create a Repository class for a model with default db' do 
+    Ribs::Repository::DB_default.constants.should include("FakeModel")
+    Ribs::Repository::DB_default::FakeModel.class.should == Class
+    Ribs::Repository::DB_default::FakeModel.ancestors.should include(Ribs::Repository)
+    Ribs::Repository::DB_default::FakeModel.ancestors.should include(Ribs::Repository::InstanceMethods)
+    Ribs::Repository::DB_default::FakeModel.ancestors.should include(Ribs::Repository::FakeModel)
+    (class << Ribs::Repository::DB_default::FakeModel; self; end).ancestors.should include(Ribs::Repository)
+    (class << Ribs::Repository::DB_default::FakeModel; self; end).ancestors.should include(Ribs::Repository::ClassMethods)
+    (class << Ribs::Repository::DB_default::FakeModel; self; end).ancestors.should include(Ribs::Repository::FakeModel::ClassMethods)
+  end
+
+  it 'should create a Repository class for a model with another db' do 
+    Ribs::Repository::DB_flarg.constants.should include("FakeModel")
+    Ribs::Repository::DB_flarg::FakeModel.class.should == Class
+    Ribs::Repository::DB_flarg::FakeModel.ancestors.should include(Ribs::Repository)
+    Ribs::Repository::DB_flarg::FakeModel.ancestors.should include(Ribs::Repository::InstanceMethods)
+    Ribs::Repository::DB_flarg::FakeModel.ancestors.should include(Ribs::Repository::FakeModel)
+    (class << Ribs::Repository::DB_flarg::FakeModel; self; end).ancestors.should include(Ribs::Repository)
+    (class << Ribs::Repository::DB_flarg::FakeModel; self; end).ancestors.should include(Ribs::Repository::ClassMethods)
+    (class << Ribs::Repository::DB_flarg::FakeModel; self; end).ancestors.should include(Ribs::Repository::FakeModel::ClassMethods)
+  end
+  
+  it 'should create a DB place the first time asked for it' do 
+    Ribs::Repository::DB_flux
+    Ribs::Repository::DB_flux.class.should == Module
+  end
+
+  it 'should only create a db if prefix with the DB_' do 
+    proc do 
+      Ribs::Repository::Noxic
+    end.should raise_error(NameError)
+  end
+
+  it 'should return a Repository class for a model' do 
+    R(FakeModel).should == Ribs::Repository::DB_default::FakeModel
+    R(FakeModel).model.should == ::FakeModel
+    R(FakeModel).database.should == :default
+  end
+  
+  it 'should return an instance of the Repository class for a model instance' do 
+    ff = FakeModel.new
+    R(ff).should be_kind_of(Ribs::Repository::DB_default::FakeModel)
+    R(ff).model.should == ff
+    R(ff).database.should == :default
+  end
+
+  it 'should return a Repository class for a model with another db' do 
+    R(FakeModel, :flarg).should == Ribs::Repository::DB_flarg::FakeModel
+    R(FakeModel, :flarg).model.should == ::FakeModel
+    R(FakeModel, :flarg).database.should == :flarg
+  end
+  
+  it 'should return an instance of the Repository class for a model instance with another db' do 
+    ff = FakeModel.new
+    R(ff, :flarg).should be_kind_of(Ribs::Repository::DB_flarg::FakeModel)
+    R(ff, :flarg).model.should == ff
+    R(ff, :flarg).database.should == :flarg
+  end
+  
+  it 'should handle submodules correctly' do 
+    R(FakeModel::FakeSecondModel).should == Ribs::Repository::DB_default::FakeModel_FakeSecondModel
+    R(FakeModel::FakeSecondModel).model.should == ::FakeModel::FakeSecondModel
+    R(FakeModel::FakeSecondModel).database.should == :default
+  end
+end

data/test/rib_spec.rb

@@ -0,0 +1,73 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+describe Ribs::Rib do 
+  it "should not have any methods defined except for method_missing" do 
+    Ribs::Rib.instance_methods.sort.should == 
+      %w(Ribs! __send__ __id__ rspec_reset 
+         rspec_verify should_receive 
+         should_not_receive R received_message? 
+         stub! __column_data__ method_missing).sort
+  end
+  
+  it "should be possible to define a primary key with a method call" do 
+    r = Ribs::Rib.new
+    r.track_id.primary_key!
+    r.__column_data__.primary_keys.should == %w(track_id)
+  end
+
+  it "should be possible to define a primary key with a hash" do 
+    r = Ribs::Rib.new
+    r.track_id :primary_key => true
+    r.__column_data__.primary_keys.should == %w(track_id)
+  end
+
+  it "should be possible to define a primary key with a simple symbol" do 
+    r = Ribs::Rib.new
+    r.track_id :primary_key
+    r.__column_data__.primary_keys.should == %w(track_id)
+  end
+
+  it "should be possible to define more than one primary key" do 
+    r = Ribs::Rib.new
+    r.track_id.primary_key!
+    r.fox_id.primary_key!
+    r.__column_data__.primary_keys.should == %w(track_id fox_id)
+  end
+
+  it "should handle methods with capital letters" do 
+    r = Ribs::Rib.new
+    r.TRACK_ID.primary_key!
+    r.__column_data__.primary_keys.should == %w(TRACK_ID)
+  end
+
+  it "should be possible to define a simple columns mapping with a hash" do 
+    r = Ribs::Rib.new
+    r.track_id :column => :TRUCK_ID
+    r.__column_data__.columns.should == { 'track_id' => ['TRUCK_ID', {:column => :TRUCK_ID}]}
+  end
+  
+  it "should be possible to define a simple columns mapping with a method call" do 
+    r = Ribs::Rib.new
+    r.track_id.column = :TRUCK_ID
+    r.__column_data__.columns.should == { 'track_id' => ['TRUCK_ID', {}]}
+  end
+  
+  it "should be possible to define something to avoid with a method call" do 
+    r = Ribs::Rib.new
+    r.track_id.avoid!
+    r.TRUCK_ID.avoid!
+    r.__column_data__.to_avoid.should == %w(track_id truck_id)
+  end
+
+  it "should be possible to define something to avoid with a hash" do 
+    r = Ribs::Rib.new
+    r.track_id :avoid => true
+    r.__column_data__.to_avoid.should == %w(track_id)
+  end
+  
+  it "should be possible to define something to avoid with a simple symbol" do 
+    r = Ribs::Rib.new
+    r.track_id :avoid
+    r.__column_data__.to_avoid.should == %w(track_id)
+  end
+end

data/test/simple_select_spec.rb

@@ -2,38 +2,38 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 
 describe "Simple select" do 
   it "should return correct data for ints" do 
-    Ribs.with_session do |s|
-      s.select("SELECT TRACK_ID, volume FROM DB_TRACK").should == [[1, 13], [2, 13]]
+    Ribs.with_handle do |h|
+      h.select("SELECT TRACK_ID, volume FROM DB_TRACK").should == [[1, 13], [2, 13]]
     end
   end
 
   it "should return correct data for strings" do 
-    Ribs.with_session do |s|
-      s.select("SELECT title, filePath FROM DB_TRACK").should == [["foobar", "c:/abc/cde/foo.mp3"], ["flux", "d:/abc/cde/flax.mp3"]]
+    Ribs.with_handle do |h|
+      h.select("SELECT title, filePath FROM DB_TRACK").should == [["foobar", "c:/abc/cde/foo.mp3"], ["flux", "d:/abc/cde/flax.mp3"]]
     end
   end
 
   it "should return correct data for times" do 
-    Ribs.with_session do |s|
-      s.select("SELECT playTime FROM DB_TRACK").should == [[Time.time_at(14,50,0)], [Time.time_at(16,23,0)]]
+    Ribs.with_handle do |h|
+      h.select("SELECT playTime FROM DB_TRACK").should == [[Time.time_at(14,50,0)], [Time.time_at(16,23,0)]]
     end
   end
 
   it "should return correct data for dates" do 
-    Ribs.with_session do |s|
-      s.select("SELECT added FROM DB_TRACK").should == [[Time.local(1984, 12, 13, 0, 0, 0)], [Time.local(1983, 12, 13, 0, 0, 0)]]
+    Ribs.with_handle do |h|
+      h.select("SELECT added FROM DB_TRACK").should == [[Time.local(1984, 12, 13, 0, 0, 0)], [Time.local(1983, 12, 13, 0, 0, 0)]]
     end
   end
 
   it "should return correct data for timestamp" do 
-    Ribs.with_session do |s|
-      s.select("SELECT lastPlayed FROM DB_TRACK").should == [[Time.local(1984, 12, 14, 12,3,11)], [Time.local(1982, 5, 3, 13,3,7)]]
+    Ribs.with_handle do |h|
+      h.select("SELECT lastPlayed FROM DB_TRACK").should == [[Time.local(1984, 12, 14, 12,3,11)], [Time.local(1982, 5, 3, 13,3,7)]]
     end
   end
   
   it "should return correct data for floats" do 
-    result = Ribs.with_session do |s|
-      s.select("SELECT fraction FROM DB_TRACK")
+    result = Ribs.with_handle do |h|
+      h.select("SELECT fraction FROM DB_TRACK")
     end
     result[0].length.should == 1
     result[0][0].should be_close(3.4, 0.00001)
@@ -43,8 +43,8 @@ describe "Simple select" do
   end
 
   it "should return correct data for doubles" do 
-    result = Ribs.with_session do |s|
-      s.select("SELECT otherFraction FROM DB_TRACK")
+    result = Ribs.with_handle do |h|
+      h.select("SELECT otherFraction FROM DB_TRACK")
     end
     result[0].length.should == 1
     result[0][0].should be_close(5.7, 0.00001)
@@ -53,27 +53,27 @@ describe "Simple select" do
   end
 
   it "should return correct data for blobs" do 
-    Ribs.with_session do |s|
-      s.select("SELECT data FROM DB_TRACK").should == [["abc"], ["mumsi"]]
+    Ribs.with_handle do |h|
+      h.select("SELECT data FROM DB_TRACK").should == [["abc"], ["mumsi"]]
     end
   end
   
   it "should return correct data for clobs" do 
-    Ribs.with_session do |s|
-      s.select("SELECT description FROM DB_TRACK").should == [["foobar"], ["maxi"]]
+    Ribs.with_handle do |h|
+      h.select("SELECT description FROM DB_TRACK").should == [["foobar"], ["maxi"]]
     end
   end
   
   it "should return correct data for booleans" do 
-    Ribs.with_session do |s|
+    Ribs.with_handle do |h|
       # Not strictly correct - an artifact of a lack in Derby
-      s.select("SELECT good FROM DB_TRACK").should == [[1], [0]]
+      h.select("SELECT good FROM DB_TRACK").should == [[1], [0]]
     end
   end
 
   it "should return correct data for decimal" do 
-    Ribs.with_session do |s|
-      s.select("SELECT price FROM DB_TRACK").should == [[BigDecimal.new("13134.11")], [BigDecimal.new("55454.33")]]
+    Ribs.with_handle do |h|
+      h.select("SELECT price FROM DB_TRACK").should == [[BigDecimal.new("13134.11")], [BigDecimal.new("55454.33")]]
     end
   end
 end