capture_migration_sql 1.0.3 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e9d0592f093cdbe9554321bc75518d3af78810d3ed24b96c2f4914d326e5c991
-  data.tar.gz: 0ab138c8d223536e39724814947a4ddccee7dac377ec14b9544aed4d759a77c0
+  metadata.gz: '09a2f079879c03ed186a9ee307884faf19826e2f67b9424cd4848322588a3be0'
+  data.tar.gz: cdc45945a17614ff25f89332fd1fba71872549e190cac1f7d61cd2fedff77a84
 SHA512:
-  metadata.gz: 48a1b618e0815a6e7644435e277b935a2e5bf65d0bb6188d7126404a83dd0cad90e17b1c1d9de675c01bae1a7119f5ab68772fb0e662dbd57e63ac8f517070b7
-  data.tar.gz: 9d311ca2aed3e266723741e5b95970fba37a3c9f133a4315b6e45d063991dd4411355a3f508f8cb3eb85527df6e179b0de177a21de9de9220f18f1981f6e6d9c
+  metadata.gz: 8dd9031cfb5a8e72f94892b6626f3ea8918d75729d4997178b32dbeb53813d6969056bb0634b5f6989bf98a78579952e832819a01ef85655259182f726f97313
+  data.tar.gz: 02a0bff93314e8a73dd6f8408a8c8e8f81ec93f5498aae72fdd3ddcb5306bd863c4d05d9da3e68c9af035156d29bf2f8edd4b8cf060b09cd1b3335011be73b5c

data/CHANGELOG.md

@@ -4,22 +4,32 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 1.0.4
+
+### Fixed
+
+- SQL files are now removed when they were created from a failed migration.
+
 ## 1.0.3
 
 ### Changed
-* SQL files will not be changed if they already exist.
+
+- SQL files will not be changed if they already exist.
 
 ## 1.0.2
 
 ### Changed
-* Fix data type for insterting into schema_migrations
+
+- Fix data type for insterting into schema_migrations
 
 ## 1.0.1
 
 ### Changed
-* Fix SQL for insterting into schema_migrations
+
+- Fix SQL for insterting into schema_migrations
 
 ## 1.0.0
 
 ### Added
-* Initial release
+
+- Initial release

data/README.md

@@ -1,7 +1,9 @@
-# CaptureMigrationSql
+# Capture Migration SQL
 
 [![Continuous Integration](https://github.com/bdurand/capture_migration_sql/actions/workflows/continuous_integration.yml/badge.svg)](https://github.com/bdurand/capture_migration_sql/actions/workflows/continuous_integration.yml)
+[![Regression Test](https://github.com/bdurand/capture_migration_sql/actions/workflows/regression_test.yml/badge.svg)](https://github.com/bdurand/capture_migration_sql/actions/workflows/regression_test.yml)
 [![Ruby Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://github.com/testdouble/standard)
+[![Gem Version](https://badge.fury.io/rb/capture_migration_sql.svg)](https://badge.fury.io/rb/capture_migration_sql)
 
 This gem adds the ability to capture and persist the SQL statements executed by migrations. There are a couple of reasons why you may want to do this.
 

data/VERSION

@@ -1 +1 @@
-1.0.3
+1.0.4

data/capture_migration_sql.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |spec|
     gemfiles/
     spec/
   ]
-  spec.files = Dir.chdir(File.expand_path("..", __FILE__)) do
+  spec.files = Dir.chdir(__dir__) do
     `git ls-files -z`.split("\x0").reject { |f| ignore_files.any? { |path| f.start_with?(path) } }
   end
 
@@ -30,4 +30,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency "activerecord", ">= 4.2"
 
   spec.add_development_dependency "bundler"
+
+  spec.required_ruby_version = ">= 2.5"
 end

data/lib/capture_migration_sql/migration_extension.rb

@@ -75,22 +75,33 @@ module CaptureMigrationSql
         else
           Dir.mkdir(migration_sql_dir) unless File.exist?(migration_sql_dir)
           SqlSubscriber.attach_if_necessary
+          retval = nil
+          success = false
           File.open(output_file, "w") do |f|
-            f.write("--\n-- #{name} : #{version}\n--\n\n")
-            save_stream = Thread.current[:capture_migration_sql_stream]
-            begin
-              Thread.current[:capture_migration_sql_stream] = f
-              sql_logging(enabled: true, &block)
-            ensure
-              Thread.current[:capture_migration_sql_stream] = save_stream
-            end
-            f.write("INSERT INTO schema_migrations (version) VALUES ('#{version.to_i}');\n")
+            retval = capture_migration_sql(f, &block)
+            success = true
+          ensure
+            File.unlink(output_file) unless success
           end
+          retval
         end
       else
         File.unlink(output_file) if output_file && File.exist?(output_file)
         yield
       end
     end
+
+    def capture_migration_sql(f, &block)
+      f.write("--\n-- #{name} : #{version}\n--\n\n")
+      save_stream = Thread.current[:capture_migration_sql_stream]
+      begin
+        Thread.current[:capture_migration_sql_stream] = f
+        retval = sql_logging(enabled: true, &block)
+      ensure
+        Thread.current[:capture_migration_sql_stream] = save_stream
+      end
+      f.write("INSERT INTO schema_migrations (version) VALUES ('#{version.to_i}');\n")
+      retval
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: capture_migration_sql
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.4
 platform: ruby
 authors:
 - Brian Durand
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-08-16 00:00:00.000000000 Z
+date: 2024-04-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -38,7 +38,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description:
 email:
 - bbdurand@gmail.com
 executables: []
@@ -58,7 +58,7 @@ homepage: https://github.com/bdurand/capture_migration_sql
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -66,15 +66,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.12
+signing_key:
 specification_version: 4
 summary: Capture the SQL that is executed when running ActiveRecord migrations so
   that it can be run in in other environments that don't support migrations.