sbf-dm-transactions 1.3.0.beta

data/spec/public/dm-transactions_spec.rb

@@ -0,0 +1,222 @@
+require_relative '../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_behaves_like 'A public Resource'
+    it_behaves_like 'A Resource supporting Strategic Eager Loading'
+  end
+
+  describe '#transaction' do
+    before :all do
+      class ::Author
+        include DataMapper::Resource
+
+        property :name, String, :key => true
+      end
+    end
+
+    supported_by :postgres, :mysql do
+      before do
+        @user_model.destroy!
+      end
+
+      it 'supports nested transactions' do
+        # can't assume dm-aggregates
+        begin
+          expect(@user_model.all.length).to eq 0
+
+          @user_model.transaction do # txn 1
+            @user_model.create(:name => 'jpr5')
+
+            expect(@user_model.all.length).to eq 1
+
+            begin
+              @user_model.transaction do # savepoint 1
+                @user_model.create(:name => 'dkubb')
+                expect(@user_model.all.length).to eq 2
+
+                raise
+              end
+            rescue => e
+              expect(@user_model.all.length).to eq 1
+            end
+
+            raise
+          end
+        rescue => e
+          expect(@user_model.all.length).to eq 0
+        end
+      end
+    end
+
+    supported_by :postgres, :mysql, :sqlite, :oracle, :sqlserver do
+      before do
+        @user_model.destroy!
+      end
+
+      it 'has access to resources presisted before the transaction' do
+        @user_model.create(:name => 'carllerche')
+        @user_model.transaction do
+          expect(@user_model.first.name).to eq 'carllerche'
+        end
+      end
+
+      it 'has access to resources persisted in the transaction' do
+        @user_model.transaction do
+          @user_model.create(:name => 'carllerche')
+          expect(@user_model.first.name).to eq 'carllerche'
+        end
+      end
+
+      it 'does not mark saved resources as new records' do
+        @user_model.transaction do
+          expect(@user_model.create(:name => 'carllerche')).not_to be_new
+        end
+      end
+
+      it 'rollback when an error is raised in a transaction' do
+        expect(@user_model.all.size).to eq 0
+        expect {
+          @user_model.transaction do
+            @user_model.create(:name => 'carllerche')
+            raise 'I love coffee'
+          end
+        }.to raise_error('I love coffee')
+        expect(@user_model.all.size).to eq 0
+      end
+
+      it 'closes the transaction if return is called within the closure' do
+        @txn = nil
+
+        def doit
+          @user_model.transaction do |transaction|
+            @txn = transaction
+            return
+          end
+        end
+        doit
+
+        expect(@txn.instance_variable_get(:@state)).to eq :commit
+        @txn = nil
+      end
+
+      it 'returns the last statement in the transaction block' do
+        expect(@user_model.transaction { 1 }).to eq 1
+      end
+
+      with_alternate_adapter do
+        before :all do
+          class ::Article
+            include DataMapper::Resource
+
+            def self.default_repository_name
+              :alternate
+            end
+
+            property :title, String, :key => true
+          end
+
+          DataMapper.auto_migrate!(:alternate)
+        end
+
+        it 'works with other repositories' do
+          expect {
+            DataMapper.repository.transaction.commit do
+              Author.create(:name => 'Dan Kubb')
+
+              # save a resource to another repository
+              Article.create(:title => 'DataMapper Rocks!')
+            end
+          }.not_to raise_error
+        end
+      end
+    end
+  end
+end

data/spec/rcov.opts

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

data/spec/spec_helper.rb

@@ -0,0 +1,18 @@
+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'
+
+RSpec.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/spec.rake

@@ -0,0 +1,21 @@
+require 'rspec/core/rake_task'
+
+begin
+  task(:default).clear
+  task(:spec).clear
+
+  RSpec::Core::RakeTask.new(:spec) do |spec|
+    spec.pattern = 'spec/**/*_spec.rb'
+
+    require 'simplecov'
+    SimpleCov.start do
+      minimum_coverage 100
+    end
+  end
+rescue LoadError
+  task :spec do
+    abort 'rspec is not available. In order to run spec, you must: gem install rspec'
+  end
+end
+
+task default: :spec

data/tasks/yard.rake

@@ -0,0 +1,9 @@
+begin
+  require 'yard'
+
+  YARD::Rake::YardocTask.new
+rescue LoadError
+  task :yard do
+    abort 'YARD is not available. In order to run yard, you must: gem install yard'
+  end
+end

data/tasks/yardstick.rake

@@ -0,0 +1,19 @@
+begin
+  require 'pathname'
+  require 'yardstick/rake/measurement'
+  require 'yardstick/rake/verify'
+
+  # yardstick_measure task
+  Yardstick::Rake::Measurement.new
+
+  # verify_measurements task
+  Yardstick::Rake::Verify.new do |verify|
+    verify.threshold = 100
+  end
+rescue LoadError
+  %w[ yardstick_measure verify_measurements ].each do |name|
+    task name.to_s do
+      abort "Yardstick is not available. In order to run #{name}, you must: gem install yardstick"
+    end
+  end
+end

metadata

@@ -0,0 +1,86 @@
+--- !ruby/object:Gem::Specification
+name: sbf-dm-transactions
+version: !ruby/object:Gem::Version
+  version: 1.3.0.beta
+platform: ruby
+authors:
+- Dirkjan Bussink (dbussink)
+- Dan Kubb (dkubb)
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2024-10-04 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: sbf-dm-core
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.3.0.beta
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.3.0.beta
+description: Makes transaction support available for adapters that support them
+email:
+- gamsnjaga@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files:
+- LICENSE
+- README.rdoc
+files:
+- ".document"
+- ".gitignore"
+- ".rspec"
+- ".rubocop.yml"
+- Gemfile
+- LICENSE
+- README.rdoc
+- Rakefile
+- dm-transactions.gemspec
+- lib/dm-transactions.rb
+- lib/dm-transactions/adapters/dm-do-adapter.rb
+- lib/dm-transactions/adapters/dm-mysql-adapter.rb
+- lib/dm-transactions/adapters/dm-oracle-adapter.rb
+- lib/dm-transactions/adapters/dm-postgres-adapter.rb
+- lib/dm-transactions/adapters/dm-sqlite-adapter.rb
+- lib/dm-transactions/adapters/dm-sqlserver-adapter.rb
+- lib/dm-transactions/version.rb
+- spec/isolated/require_after_setup_spec.rb
+- spec/isolated/require_before_setup_spec.rb
+- spec/isolated/require_spec.rb
+- spec/public/dm-transactions_spec.rb
+- spec/rcov.opts
+- spec/spec_helper.rb
+- tasks/spec.rake
+- tasks/yard.rake
+- tasks/yardstick.rake
+homepage: https://datamapper.org
+licenses:
+- Nonstandard
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">"
+    - !ruby/object:Gem::Version
+      version: 1.3.1
+requirements: []
+rubygems_version: 3.4.10
+signing_key: 
+specification_version: 4
+summary: Adds transaction support to DataMapper
+test_files: []