Introduced concept of session. It does not yet support multiple databases, nor transactions, but it's a start!

6 files changed, 100 insertions, 43 deletions

diff --git a/spec/active_relation/integration/scratch_spec.rb b/spec/active_relation/integration/scratch_spec.rb
index 4b02b22a11..d6ec030518 100644
--- a/spec/active_relation/integration/scratch_spec.rb
+++ b/spec/active_relation/integration/scratch_spec.rb
@@ -136,24 +136,6 @@ describe 'ActiveRelation', 'A proposed refactoring to ActiveRecord, introducing
       """)
     end
 
-    describe 'write operations' do
-      it 'generates the query for user.destroy' do
-        @user.delete.to_sql.should be_like("""
-          DELETE
-          FROM `users`
-          WHERE `users`.`id` = 1
-        """)
-      end
-      
-     it 'generates an efficient query for two User.creates -- UnitOfWork is within reach!' do
-        @users.insert(@users[:name] => "humpty").insert(@users[:name] => "dumpty").to_sql.should be_like("""
-          INSERT
-          INTO `users`
-          (`users`.`name`) VALUES ('humpty'), ('dumpty')
-        """)
-      end
-    end
-
     describe 'with_scope' do
       it 'obviates the need for with_scope merging logic since, e.g.,
             `with_scope :conditions => ...` is just a #select operation on the relation' do
diff --git a/spec/active_relation/unit/relations/insertion_spec.rb b/spec/active_relation/unit/relations/insertion_spec.rb
index a0427bc74f..a39c4aeaf3 100644
--- a/spec/active_relation/unit/relations/insertion_spec.rb
+++ b/spec/active_relation/unit/relations/insertion_spec.rb
@@ -14,15 +14,6 @@ module ActiveRelation
           (`users`.`name`) VALUES ('nick')
         """)
       end
-    
-      it 'manufactures sql inserting the data for multiple items' do
-        nested_insertion = Insertion.new(@relation, @relation[:name] => "cobra")
-        Insertion.new(nested_insertion, nested_insertion[:name] => "commander").to_sql.should be_like("""
-          INSERT
-          INTO `users`
-          (`users`.`name`) VALUES ('cobra'), ('commander')
-        """)
-      end
     end
   end
 end
\ No newline at end of file
diff --git a/spec/active_relation/unit/relations/relation_spec.rb b/spec/active_relation/unit/relations/relation_spec.rb
index a80df928c7..d3252c658b 100644
--- a/spec/active_relation/unit/relations/relation_spec.rb
+++ b/spec/active_relation/unit/relations/relation_spec.rb
@@ -36,13 +36,13 @@ module ActiveRelation
       end
     end
     
-    describe '#Expression?' do
+    describe '#aggregation?' do
       it "returns false" do
         @relation.should_not be_aggregation
       end
     end
 
-    describe 'read operations' do
+    describe Relation::Operations do
       describe 'joins' do
         before do
           @predicate = @relation[:id].equals(@relation[:id])
@@ -105,20 +105,35 @@ module ActiveRelation
             should == Aggregation.new(@relation, :expressions => [@expresion, @expression2], :groupings => [@attribute1, @attribute2])
         end
       end
-    end
-  
-    describe 'write operations' do
-      describe '#delete' do
-        it 'manufactures a deletion relation' do
-          @relation.delete.should be_kind_of(Deletion)
+      
+      describe Relation::Operations::Writes do
+        describe '#delete' do
+          it 'manufactures a deletion relation' do
+            mock(Session.instance).delete(Deletion.new(@relation))
+            @relation.delete.should == @relation
+          end
         end
-      end
-    
-      describe '#insert' do
-        it 'manufactures an insertion relation' do
-          @relation.insert(record = {:id => 1}).should be_kind_of(Insertion)
+
+        describe '#insert' do
+          it 'manufactures an insertion relation' do
+            mock(Session.instance).create(Insertion.new(@relation, record = {@relation[:name] => 'carl'}))
+            @relation.insert(record).should == @relation
+          end
+        end
+
+        describe '#update' do
+          it 'manufactures an update relation' do
+            mock(Session.instance).update(Update.new(@relation, assignments = {@relation[:name] => 'bob'}))
+            @relation.update(assignments).should == @relation
+          end
         end
       end
     end
+      
+    describe Relation::Enumerable do
+      it "is enumerable" do
+        pending
+      end
+    end
   end
 end
\ No newline at end of file
diff --git a/spec/active_relation/unit/relations/update_spec.rb b/spec/active_relation/unit/relations/update_spec.rb
index c26e7fb525..642d652057 100644
--- a/spec/active_relation/unit/relations/update_spec.rb
+++ b/spec/active_relation/unit/relations/update_spec.rb
@@ -21,7 +21,6 @@ module ActiveRelation
           WHERE `users`.`id` = 1
         """)
       end
-      
     end
   end
 end
\ No newline at end of file
diff --git a/spec/active_relation/unit/session/session_spec.rb b/spec/active_relation/unit/session/session_spec.rb
new file mode 100644
index 0000000000..ddd748334a
--- /dev/null
+++ b/spec/active_relation/unit/session/session_spec.rb
@@ -0,0 +1,70 @@
+require File.join(File.dirname(__FILE__), '..', '..', '..', 'spec_helper')
+
+module ActiveRelation
+  describe Session do
+    before do
+      @relation = Table.new(:users)
+      @session = Session.instance
+    end
+    
+    describe Singleton do
+      it "is a singleton" do
+        Session.instance.should be_equal(Session.instance)
+        lambda { Session.new }.should raise_error
+      end
+    end
+    
+    describe Session::CRUD do
+      before do
+        @insert = Insertion.new(@relation, @relation[:name] => 'nick')
+        @update = Update.new(@relation, @relation[:name] => 'nick')
+        @delete = Deletion.new(@relation)
+        @select = @relation
+      end
+      
+      describe '#create' do
+        it "should execute an insertion on the connection" do
+          mock(@session.connection).insert(@insert.to_sql)
+          @session.create(@insert)
+        end
+      end
+      
+      describe '#read' do
+        it "should execute an selection on the connection" do
+          mock(@session.connection).select_all(@select.to_sql)
+          @session.read(@select)
+        end
+      end
+      
+      describe '#update' do
+        it "should execute an update on the connection" do
+          mock(@session.connection).update(@update.to_sql)
+          @session.update(@update)
+        end
+      end
+      
+      describe '#delete' do
+        it "should execute a delete on the connection" do
+          mock(@session.connection).delete(@delete.to_sql)
+          @session.delete(@delete)
+        end
+      end
+    end
+    
+    describe Session::Transactions do
+      describe '#begin' do
+      end
+    
+      describe '#end' do
+      end
+    end
+    
+    describe Session::UnitOfWork do
+      describe '#flush' do
+      end
+      
+      describe '#clear' do
+      end
+    end
+  end
+end
\ No newline at end of file
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index e3508aac90..59202dc9f0 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -7,7 +7,7 @@ Dir["#{dir}/matchers/*"].each { |m| require "#{dir}/matchers/#{File.basename(m)}
 require 'active_relation'
 
 ActiveRecord::Base.configurations = {
-  'sql_algebra_test' => {
+  'test' => {
     :adapter  => 'mysql',
     :username => 'root',
     :password => 'password',
@@ -15,7 +15,7 @@ ActiveRecord::Base.configurations = {
     :database => 'sql_algebra_test',
   },
 }
-ActiveRecord::Base.establish_connection 'sql_algebra_test'
+ActiveRecord::Base.establish_connection 'test'
 
 class Hash
   def shift