dm-transactions 1.0.0.rc1

data/lib/dm-transactions/adapters/dm-do-adapter.rb

@@ -0,0 +1,96 @@
+module DataMapper
+  class Transaction
+
+    module DataObjectsAdapter
+      extend Chainable
+
+      # Produces a fresh transaction primitive for this Adapter
+      #
+      # Used by Transaction to perform its various tasks.
+      #
+      # @return [Object]
+      #   a new Object that responds to :close, :begin, :commit,
+      #   and :rollback,
+      #
+      # @api private
+      def transaction_primitive
+        DataObjects::Transaction.create_for_uri(normalized_uri)
+      end
+
+      # Pushes the given Transaction onto the per thread Transaction stack so
+      # that everything done by this Adapter is done within the context of said
+      # Transaction.
+      #
+      # @param [Transaction] transaction
+      #   a Transaction to be the 'current' transaction until popped.
+      #
+      # @return [Array(Transaction)]
+      #   the stack of active transactions for the current thread
+      #
+      # @api private
+      def push_transaction(transaction)
+        transactions << transaction
+      end
+
+      # Pop the 'current' Transaction from the per thread Transaction stack so
+      # that everything done by this Adapter is no longer necessarily within the
+      # context of said Transaction.
+      #
+      # @return [Transaction]
+      #   the former 'current' transaction.
+      #
+      # @api private
+      def pop_transaction
+        transactions.pop
+      end
+
+      # Retrieve the current transaction for this Adapter.
+      #
+      # Everything done by this Adapter is done within the context of this
+      # Transaction.
+      #
+      # @return [Transaction]
+      #   the 'current' transaction for this Adapter.
+      #
+      # @api private
+      def current_transaction
+        transactions.last
+      end
+
+      chainable do
+        protected
+
+        # @api semipublic
+        def open_connection
+          current_connection || super
+        end
+
+        # @api semipublic
+        def close_connection(connection)
+          super unless current_connection.equal?(connection)
+        end
+      end
+
+      private
+
+      # @api private
+      def transactions
+        Thread.current[:dm_transactions] ||= []
+      end
+
+      # Retrieve the current connection for this Adapter.
+      #
+      # @return [Transaction]
+      #   the 'current' connection for this Adapter.
+      #
+      # @api private
+      def current_connection
+        if transaction = current_transaction
+          transaction.primitive_for(self).connection
+        end
+      end
+
+    end # module DataObjectsAdapter
+
+  end # class Transaction
+end # module DataMapper

data/lib/dm-transactions/adapters/dm-mysql-adapter.rb

@@ -0,0 +1,11 @@
+require 'dm-transactions/adapters/dm-do-adapter'
+
+module DataMapper
+  class Transaction
+
+    module MysqlAdapter
+      include DataObjectsAdapter
+    end
+
+  end
+end

data/lib/dm-transactions/adapters/dm-oracle-adapter.rb

@@ -0,0 +1,11 @@
+require 'dm-transactions/adapters/dm-do-adapter'
+
+module DataMapper
+  class Transaction
+
+    module OracleAdapter
+      include DataObjectsAdapter
+    end
+
+  end
+end

data/lib/dm-transactions/adapters/dm-postgres-adapter.rb

@@ -0,0 +1,11 @@
+require 'dm-transactions/adapters/dm-do-adapter'
+
+module DataMapper
+  class Transaction
+
+    module PostgresAdapter
+      include DataObjectsAdapter
+    end
+
+  end
+end

data/lib/dm-transactions/adapters/dm-sqlite-adapter.rb

@@ -0,0 +1,11 @@
+require 'dm-transactions/adapters/dm-do-adapter'
+
+module DataMapper
+  class Transaction
+
+    module SqliteAdapter
+      include DataObjectsAdapter
+    end
+
+  end
+end

data/lib/dm-transactions/adapters/dm-sqlserver-adapter.rb

@@ -0,0 +1,11 @@
+require 'dm-transactions/adapters/dm-do-adapter'
+
+module DataMapper
+  class Transaction
+
+    module SqlserverAdapter
+      include DataObjectsAdapter
+    end
+
+  end
+end

data/spec/isolated/require_after_setup_spec.rb

@@ -0,0 +1,23 @@
+require 'spec'
+require 'isolated/require_spec'
+require 'dm-core/spec/setup'
+require 'dm-core/spec/lib/adapter_helpers'
+
+# To really test this behavior, this spec needs to be run in isolation and not
+# as part of the typical rake spec run, which requires dm-transactions upfront
+
+if %w[postgres mysql sqlite oracle sqlserver].include?(ENV['ADAPTER'])
+
+  describe "require 'dm-transactions after calling DataMapper.setup" do
+    extend DataMapper::Spec::Adapters::Helpers
+
+    before(:all) do
+      @adapter = DataMapper::Spec.adapter
+      require 'dm-transactions'
+    end
+
+    it_should_behave_like "require 'dm-transactions'"
+
+  end
+
+end

data/spec/isolated/require_before_setup_spec.rb

@@ -0,0 +1,23 @@
+require 'spec'
+require 'isolated/require_spec'
+require 'dm-core/spec/setup'
+require 'dm-core/spec/lib/adapter_helpers'
+
+# To really test this behavior, this spec needs to be run in isolation and not
+# as part of the typical rake spec run, which requires dm-transactions upfront
+
+if %w[postgres mysql sqlite oracle sqlserver].include?(ENV['ADAPTER'])
+
+  describe "require 'dm-transactions' before calling DataMapper.setup" do
+    extend DataMapper::Spec::Adapters::Helpers
+
+    before(:all) do
+      require 'dm-transactions'
+      @adapter = DataMapper::Spec.adapter
+    end
+
+    it_should_behave_like "require 'dm-transactions'"
+
+  end
+
+end

data/spec/isolated/require_spec.rb

@@ -0,0 +1,15 @@
+shared_examples_for "require 'dm-transactions'" do
+
+  %w[Repository Model Resource].each do |name|
+    it "should include the transaction api in DataMapper::#{name}" do
+      (DataMapper.const_get(name) < DataMapper::Transaction.const_get(name)).should be_true
+    end
+  end
+
+  it "should include the transaction api into the adapter" do
+    @adapter.respond_to?(:push_transaction   ).should be_true
+    @adapter.respond_to?(:pop_transaction    ).should be_true
+    @adapter.respond_to?(:current_transaction).should be_true
+  end
+
+end

data/spec/public/dm-transactions_spec.rb

@@ -0,0 +1,153 @@
+require 'spec_helper'
+
+describe DataMapper::Resource, 'Transactions' do
+  before :all do
+    module ::Blog
+      class User
+        include DataMapper::Resource
+
+        property :name,        String, :key => true
+        property :age,         Integer
+        property :summary,     Text
+        property :description, Text
+        property :admin,       Boolean, :accessor => :private
+
+        belongs_to :parent, self, :required => false
+        has n, :children, self, :inverse => :parent
+
+        belongs_to :referrer, self, :required => false
+        has n, :comments
+
+        # FIXME: figure out a different approach than stubbing things out
+        def comment=(*)
+          # do nothing with comment
+        end
+      end
+
+      class Author < User; end
+
+      class Comment
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :body, Text
+
+        belongs_to :user
+      end
+
+      class Article
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :body, Text
+
+        has n, :paragraphs
+      end
+
+      class Paragraph
+        include DataMapper::Resource
+
+        property :id,   Serial
+        property :text, String
+
+        belongs_to :article
+      end
+    end
+
+    class ::Default
+      include DataMapper::Resource
+
+      property :name, String, :key => true, :default => 'a default value'
+    end
+
+    @user_model      = Blog::User
+    @author_model    = Blog::Author
+    @comment_model   = Blog::Comment
+    @article_model   = Blog::Article
+    @paragraph_model = Blog::Paragraph
+  end
+
+  supported_by :postgres, :mysql, :sqlite, :oracle, :sqlserver do
+    before :all do
+      user = @user_model.create(:name => 'dbussink', :age => 25, :description => 'Test')
+
+      @user = @user_model.get(*user.key)
+    end
+
+    before do
+      # --- Temporary private api use to get around rspec limitations ---
+      @repository.scope do |repository|
+        transaction = DataMapper::Transaction.new(repository)
+        transaction.begin
+        repository.adapter.push_transaction(transaction)
+      end
+    end
+
+    after do
+      while @repository.adapter.current_transaction
+        @repository.adapter.pop_transaction.rollback
+      end
+    end
+
+    it_should_behave_like 'A public Resource'
+    it_should_behave_like 'A Resource supporting Strategic Eager Loading'
+  end
+
+  supported_by :postgres, :mysql, :sqlite3, :oracle, :sqlserver do
+    describe '#transaction' do
+      before do
+        @user_model.all.destroy!
+      end
+
+      it 'should have access to resources presisted before the transaction' do
+        @user_model.create(:name => 'carllerche')
+        @user_model.transaction do
+          @user_model.first.name.should == 'carllerche'
+        end
+      end
+
+      it 'should have access to resources persisted in the transaction' do
+        @user_model.transaction do
+          @user_model.create(:name => 'carllerche')
+          @user_model.first.name.should == 'carllerche'
+        end
+      end
+
+      it 'should not mark saved resources as new records' do
+        @user_model.transaction do
+          @user_model.create(:name => 'carllerche').should_not be_new
+        end
+      end
+
+      it 'should rollback when an error is thrown in a transaction' do
+        @user_model.all.should have(0).entries
+        lambda {
+          @user_model.transaction do
+            @user_model.create(:name => 'carllerche')
+            raise 'I love coffee'
+          end
+        }.should raise_error('I love coffee')
+        @user_model.all.should have(0).entries
+      end
+
+      it 'should close the transaction if return is called within the closure' do
+        @txn = nil
+
+        def doit
+          @user_model.transaction do
+            @txn = Thread.current[:dm_transactions].last
+            return
+          end
+        end
+        doit
+
+        @txn.instance_variable_get(:@state).should == :commit
+        @txn = nil
+      end
+
+      it 'should return the last statement in the transaction block' do
+        @user_model.transaction { 1 }.should == 1
+      end
+    end
+  end
+end

data/spec/rcov.opts

@@ -0,0 +1,6 @@
+--exclude "spec"
+--sort coverage
+--callsites
+--xrefs
+--profile
+--text-summary

data/spec/spec.opts

@@ -0,0 +1 @@
+--color

data/spec/spec_helper.rb

@@ -0,0 +1,20 @@
+require 'dm-transactions'
+
+require 'dm-core/spec/setup'
+require 'dm-core/spec/lib/spec_helper'
+require 'dm-core/spec/lib/pending_helpers'
+require 'dm-core/spec/lib/adapter_helpers'
+require 'dm-core/spec/lib/counter_adapter'
+require 'dm-core/spec/shared/resource_spec'
+require 'dm-core/spec/shared/sel_spec'
+
+Spec::Runner.configure do |config|
+
+  config.extend(DataMapper::Spec::Adapters::Helpers)
+  config.include(DataMapper::Spec::PendingHelpers)
+
+  config.after :all do
+    DataMapper::Spec.cleanup_models
+  end
+
+end

data/tasks/ci.rake

@@ -0,0 +1 @@
+task :ci => [ :verify_measurements, 'metrics:all' ]

data/tasks/local_gemfile.rake

@@ -0,0 +1,18 @@
+desc "Support bundling from local source code (allows BUNDLE_GEMFILE=Gemfile.local bundle foo)"
+task :local_gemfile do |t|
+
+  root              = Pathname(__FILE__).dirname.parent
+  datamapper        = root.parent
+
+  source_regex     = /DATAMAPPER = 'git:\/\/github.com\/datamapper'/
+  gem_source_regex = /:git => \"#\{DATAMAPPER\}\/(.+?)(?:\.git)?\"/
+
+  root.join('Gemfile.local').open('w') do |f|
+    root.join('Gemfile').open.each do |line|
+      line.sub!(source_regex,     "DATAMAPPER = '#{datamapper}'")
+      line.sub!(gem_source_regex, ':path => "#{DATAMAPPER}/\1"')
+      f.puts line
+    end
+  end
+
+end

data/tasks/metrics.rake

@@ -0,0 +1,36 @@
+begin
+  require 'metric_fu'
+rescue LoadError
+  namespace :metrics do
+    task :all do
+      abort 'metric_fu is not available. In order to run metrics:all, you must: gem install metric_fu'
+    end
+  end
+end
+
+begin
+  require 'reek/adapters/rake_task'
+
+  Reek::RakeTask.new do |t|
+    t.fail_on_error = true
+    t.verbose       = false
+    t.source_files  = 'lib/**/*.rb'
+  end
+rescue LoadError
+  task :reek do
+    abort 'Reek is not available. In order to run reek, you must: gem install reek'
+  end
+end
+
+begin
+  require 'roodi'
+  require 'roodi_task'
+
+  RoodiTask.new do |t|
+    t.verbose = false
+  end
+rescue LoadError
+  task :roodi do
+    abort 'Roodi is not available. In order to run roodi, you must: gem install roodi'
+  end
+end