transaction_isolation_continued 1.0.5 → 1.1.0

data/test/integration/active_record/connection_adapters/any_adapter/isolation_level_test.rb

@@ -1,69 +1,58 @@
-# -*- encoding : utf-8 -*-
+# frozen_string_literal: true
 
 require 'test_helper'
 
 class ActiveRecordTest < Minitest::Test
-
   class ConnectionAdaptersTest < Minitest::Test
-
     class AnyAdapterTest < Minitest::Test
-
       class IsolationLevelTest < Minitest::Test
-
         def test_without_a_block
           original_isolation_level = ActiveRecord::Base.connection.current_isolation_level
 
-          ActiveRecord::Base.connection.isolation_level( :read_uncommitted )
-          assert_equal( :read_uncommitted, ActiveRecord::Base.connection.current_isolation_level )
+          ActiveRecord::Base.connection.isolation_level(:read_uncommitted)
+          assert_equal(:read_uncommitted, ActiveRecord::Base.connection.current_isolation_level)
 
-          ActiveRecord::Base.connection.isolation_level( original_isolation_level )
-          assert_equal( original_isolation_level, ActiveRecord::Base.connection.current_isolation_level )
+          ActiveRecord::Base.connection.isolation_level(original_isolation_level)
+          assert_equal(original_isolation_level, ActiveRecord::Base.connection.current_isolation_level)
         end
 
         def test_with_a_block
           original_isolation_level = ActiveRecord::Base.connection.current_isolation_level
-          refute_equal( :read_uncommitted, original_isolation_level )
+          refute_equal(:read_uncommitted, original_isolation_level)
 
-          ActiveRecord::Base.connection.isolation_level( :read_uncommitted ) do
-            assert_equal( :read_uncommitted, ActiveRecord::Base.connection.current_isolation_level )
+          ActiveRecord::Base.connection.isolation_level(:read_uncommitted) do
+            assert_equal(:read_uncommitted, ActiveRecord::Base.connection.current_isolation_level)
             ActiveRecord::Base.transaction do
-              assert_equal( :read_uncommitted, ActiveRecord::Base.connection.current_isolation_level )
+              assert_equal(:read_uncommitted, ActiveRecord::Base.connection.current_isolation_level)
               QueuedJob.count
               QueuedJob.first
-              assert_equal( :read_uncommitted, ActiveRecord::Base.connection.current_isolation_level )
+              assert_equal(:read_uncommitted, ActiveRecord::Base.connection.current_isolation_level)
             end
-            assert_equal( :read_uncommitted, ActiveRecord::Base.connection.current_isolation_level )
+            assert_equal(:read_uncommitted, ActiveRecord::Base.connection.current_isolation_level)
           end
 
-          assert_equal( original_isolation_level, ActiveRecord::Base.connection.current_isolation_level )
+          assert_equal(original_isolation_level, ActiveRecord::Base.connection.current_isolation_level)
         end
 
         def test_with_all_possible_ansi_levels
-          [:read_uncommitted, :read_committed, :repeatable_read, :serializable].each do |ansi_level|
-
-            QueuedJob.isolation_level( ansi_level ) do
-
+          %i[read_uncommitted read_committed repeatable_read serializable].each do |ansi_level|
+            QueuedJob.isolation_level(ansi_level) do
               # Some typical usage
               QueuedJob.transaction do
-                QueuedJob.create!( :job => 'is fun' )
-                assert_equal( 1, QueuedJob.count )
+                QueuedJob.create!(job: 'is fun')
+                assert_equal(1, QueuedJob.count)
                 raise ActiveRecord::Rollback
               end
-
             end
           end
         end
 
         def test_with_invalid_isolation_level
-          assert_raises( ArgumentError ) do
-            QueuedJob.isolation_level( :dupa )
+          assert_raises(ArgumentError) do
+            QueuedJob.isolation_level(:dupa)
           end
         end
-
       end
-
     end
-
   end
-
 end

data/test/integration/active_record/connection_adapters/any_adapter/supports_isolation_levels_test.rb

@@ -1,23 +1,15 @@
-# -*- encoding : utf-8 -*-
+# frozen_string_literal: true
 
 require 'test_helper'
 
 class ActiveRecordTest < Minitest::Test
-
   class ConnectionAdaptersTest < Minitest::Test
-
     class AnyAdapterTest < Minitest::Test
-
       class SupportsIsolationLevelsTest < Minitest::Test
-
         def test_returns_true
-          assert( ActiveRecord::Base.connection.supports_isolation_levels? )
+          assert(ActiveRecord::Base.connection.supports_isolation_levels?)
         end
-
       end
-
     end
-
   end
-
 end

data/test/integration/active_record/connection_adapters/any_adapter/translate_exception_test.rb

@@ -1,46 +1,40 @@
-# -*- encoding : utf-8 -*-
+# frozen_string_literal: true
 
 require 'test_helper'
 
 class ActiveRecordTest < Minitest::Test
-
   class ConnectionAdaptersTest < Minitest::Test
-
     class AnyAdapterTest < Minitest::Test
-
       class TranslateExceptionTest < Minitest::Test
-
         def test_does_not_break_existing_translation
-          assert_raises( ActiveRecord::StatementInvalid ) do
-            ActiveRecord::Base.connection.execute( "WE LIVE IN THE MOST EXCITING TIMES EVER" )
+          assert_raises(ActiveRecord::StatementInvalid) do
+            ActiveRecord::Base.connection.execute('WE LIVE IN THE MOST EXCITING TIMES EVER')
           end
         end
 
         def test_translates_low_level_exceptions_to_transaction_isolation_level
-          if defined?( ActiveRecord::ConnectionAdapters::Mysql2Adapter )
-            message = "Deadlock found when trying to get lock"
-            translated_exception = ActiveRecord::Base.connection.send( :translate_exception, Mysql2::Error.new( message ), message: message, sql: nil, binds: nil )
-            assert_equal( ActiveRecord::TransactionIsolationConflict, translated_exception.class )
-          end
-          
-          if defined?( ActiveRecord::ConnectionAdapters::PostgreSQLAdapter )
-            message = "deadlock detected"
-            translated_exception = ActiveRecord::Base.connection.send( :translate_exception, PG::Error.new( message ), message: message, sql: nil, binds: nil )
-            assert_equal( ActiveRecord::TransactionIsolationConflict, translated_exception.class )
+          if defined?(ActiveRecord::ConnectionAdapters::Mysql2Adapter)
+            message = 'Deadlock found when trying to get lock'
+            translated_exception = ActiveRecord::Base.connection.send(:translate_exception,
+                                                                      Mysql2::Error.new(message), message: message, sql: nil, binds: nil)
+            assert_equal(ActiveRecord::TransactionIsolationConflict, translated_exception.class)
           end
-          
-          if defined?( ActiveRecord::ConnectionAdapters::SQLite3Adapter )
-            message = "The database file is locked"
-            translated_exception = ActiveRecord::Base.connection.send( :translate_exception, StandardError.new( message ), message: message, sql: nil, binds: nil )
-            assert_equal( ActiveRecord::TransactionIsolationConflict, translated_exception.class )
+
+          if defined?(ActiveRecord::ConnectionAdapters::PostgreSQLAdapter)
+            message = 'deadlock detected'
+            translated_exception = ActiveRecord::Base.connection.send(:translate_exception, PG::Error.new(message),
+                                                                      message: message, sql: nil, binds: nil)
+            assert_equal(ActiveRecord::TransactionIsolationConflict, translated_exception.class)
           end
 
-        end
+          return unless defined?(ActiveRecord::ConnectionAdapters::SQLite3Adapter)
 
+          message = 'The database file is locked'
+          translated_exception = ActiveRecord::Base.connection.send(:translate_exception,
+                                                                    StandardError.new(message), message: message, sql: nil, binds: nil)
+          assert_equal(ActiveRecord::TransactionIsolationConflict, translated_exception.class)
+        end
       end
-
     end
-
   end
-
 end

data/test/library_setup.rb

@@ -1,17 +1,19 @@
+# frozen_string_literal: true
+
 # Prepares application to be tested (requires files, connects to db, resets schema and data, applies patches, etc.)
 
 # Initialize database
 require 'db/all'
 
 case ENV['db']
-  when 'mysql2'
-    TransactionIsolation::Test::Db.connect_to_mysql2
-  when 'postgresql'
-    TransactionIsolation::Test::Db.connect_to_postgresql
-  when 'sqlite3'
-    TransactionIsolation::Test::Db.connect_to_sqlite3
-  else
-    TransactionIsolation::Test::Db.connect_to_sqlite3
+when 'mysql2'
+  TransactionIsolation::Test::Db.connect_to_mysql2
+when 'postgresql'
+  TransactionIsolation::Test::Db.connect_to_postgresql
+when 'sqlite3'
+  TransactionIsolation::Test::Db.connect_to_sqlite3
+else
+  TransactionIsolation::Test::Db.connect_to_sqlite3
 end
 
 TransactionIsolation::Test::Migrations.run!
@@ -20,6 +22,6 @@ TransactionIsolation::Test::Migrations.run!
 require 'transaction_isolation'
 
 require 'logger'
-ActiveRecord::Base.logger = Logger.new( File.expand_path( "#{File.dirname( __FILE__ )}/log/test.log" ) )
+ActiveRecord::Base.logger = Logger.new(File.expand_path("#{File.dirname(__FILE__)}/log/test.log"))
 
 TransactionIsolation.apply_activerecord_patch

data/test/test_helper.rb

@@ -1,9 +1,12 @@
-# Load test coverage tool (must be loaded before any code)
-#require 'simplecov'
-#SimpleCov.start do
-#  add_filter '/test/'
-#  add_filter '/config/'
-#end
+# frozen_string_literal: true
+
+require 'simplecov'
+require 'simplecov_json_formatter'
+SimpleCov.formatter = SimpleCov::Formatter::JSONFormatter
+
+SimpleCov.start do
+  add_filter '/test/'
+end
 
 # Load and initialize the application to be tested
 require 'library_setup'

data/test/test_runner.rb

@@ -1,4 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
-# Load all tests
-Dir.glob( "./**/*_test.rb" ).each { |test_file| require test_file }
+Dir.glob('./**/*_test.rb').sort.each { |test_file| require test_file }

data/transaction_isolation_continued.gemspec

@@ -1,25 +1,25 @@
-# -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "transaction_isolation/version"
+# frozen_string_literal: true
+
+$LOAD_PATH.push File.expand_path('lib', __dir__)
+require 'transaction_isolation/version'
 
 Gem::Specification.new do |s|
-  s.name        = "transaction_isolation_continued"
+  s.name        = 'transaction_isolation_continued'
   s.version     = TransactionIsolation::VERSION
-  s.authors     = ["Piotr 'Qertoip' Włodarek"]
-  s.email       = ["qertoip@gmail.com"]
-  s.homepage    = "https://github.com/qertoip/transaction_isolation"
-  s.summary     = %q{Set transaction isolation level in the ActiveRecord in a database agnostic way.}
-  s.description = %q{Set transaction isolation level in the ActiveRecord in a database agnostic way.
+  s.authors     = ['Iago Pimenta']
+  s.homepage    = 'https://github.com/iagopiimenta/transaction_isolation_continued'
+  s.summary     = 'Set transaction isolation level in the ActiveRecord in a database agnostic way.'
+  s.description = 'Set transaction isolation level in the ActiveRecord in a database agnostic way.
 Works with MySQL, PostgreSQL and SQLite as long as you are using new adapters mysql2, pg or sqlite3.
-Supports all ANSI SQL isolation levels: :serializable, :repeatable_read, :read_committed, :read_uncommitted.}
+Supports all ANSI SQL isolation levels: :serializable, :repeatable_read, :read_committed, :read_uncommitted.'
   s.required_ruby_version = '>= 1.9.2'
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  s.require_paths = ["lib"]
+  s.executables   = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  s.require_paths = ['lib']
 
-  s.add_development_dependency 'rake', '~> 13.0'
   s.add_development_dependency 'minitest', '5.3.4'
-  s.add_runtime_dependency "activerecord", ">= 5.2"
+  s.add_development_dependency 'rake', '~> 13.0'
+  s.add_runtime_dependency 'activerecord', '>= 5.2'
 end

metadata

@@ -1,43 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: transaction_isolation_continued
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.1.0
 platform: ruby
 authors:
-- Piotr 'Qertoip' Włodarek
+- Iago Pimenta
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-05-14 00:00:00.000000000 Z
+date: 2024-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: 5.3.4
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - '='
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: 5.3.4
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.3.4
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.3.4
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -57,19 +57,27 @@ description: |-
   Works with MySQL, PostgreSQL and SQLite as long as you are using new adapters mysql2, pg or sqlite3.
   Supports all ANSI SQL isolation levels: :serializable, :repeatable_read, :read_committed, :read_uncommitted.
 email:
-- qertoip@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".dockerignore"
 - ".github/workflows/main.yml"
 - ".gitignore"
 - Gemfile
-- Gemfile.local
 - LICENSE
 - README.md
 - Rakefile
-- d
+- docker-compose.yml
+- docker/ruby-2.5/Dockerfile
+- docker/ruby-2.7/Dockerfile
+- docker/ruby-3.0/Dockerfile
+- docker/ruby-3.1/Dockerfile
+- docker/test-ruby-2.5.sh
+- docker/test-ruby-2.7.sh
+- docker/test-ruby-3.0.sh
+- docker/test-ruby-3.1.sh
+- docker/test-ruby.sh
 - gemfiles/Gemfile.base
 - gemfiles/activerecord-5.2/Gemfile.base
 - gemfiles/activerecord-5.2/Gemfile.mysql2
@@ -108,12 +116,11 @@ files:
 - test/integration/active_record/connection_adapters/any_adapter/translate_exception_test.rb
 - test/library_setup.rb
 - test/log/.gitkeep
-- test/test_console.rb
 - test/test_helper.rb
 - test/test_runner.rb
 - tests
 - transaction_isolation_continued.gemspec
-homepage: https://github.com/qertoip/transaction_isolation
+homepage: https://github.com/iagopiimenta/transaction_isolation_continued
 licenses: []
 metadata: {}
 post_install_message:
@@ -149,6 +156,5 @@ test_files:
 - test/integration/active_record/connection_adapters/any_adapter/translate_exception_test.rb
 - test/library_setup.rb
 - test/log/.gitkeep
-- test/test_console.rb
 - test/test_helper.rb
 - test/test_runner.rb

data/Gemfile.local

@@ -1,18 +0,0 @@
-source "http://rubygems.org"
-
-gemspec
-
-gem "rake"
-
-platform :ruby do
-  # https://github.com/rails/rails/pull/51592
-  gem "sqlite3", "~> 1.4"
-  gem "mysql2"
-  gem "pg"
-end
-
-platform :jruby do
-  gem 'activerecord-jdbcpostgresql-adapter'
-  gem 'activerecord-jdbcmysql-adapter'
-  gem 'activerecord-jdbcsqlite3-adapter', '>=1.3.0.beta2'
-end

data/d

@@ -1 +0,0 @@
-bundle exec ruby test/test_console.rb

data/test/test_console.rb

@@ -1,11 +0,0 @@
-# Ensure that LOAD_PATH is the same as when running "rake test"; normally rake takes care of that
-$LOAD_PATH << File.expand_path( ".", File.dirname( __FILE__ ) )
-$LOAD_PATH << File.expand_path( "./lib", File.dirname( __FILE__ ) )
-$LOAD_PATH << File.expand_path( "./test", File.dirname( __FILE__ ) )
-
-# Boot the app
-require_relative 'library_setup'
-
-# Fire the console
-require 'pry'
-binding.pry