Rename meta_tables_after_main to group_table_definition

Author: kravcneger

CHANGELOG.md

@@ -7,12 +7,12 @@
 ## 0.5.0    2025-07-09
 
 ### Added
-* Introduced `:meta_tables_after_main` option as a more flexible replacement for `:indexes_after_tables`.  
+* Introduced `:group_table_definition` option as a more flexible replacement for `:indexes_after_tables`.  
   This option places indexes and deferrable constraints after their respective `CREATE TABLE` statements.
 
 ### Deprecated
 * `:indexes_after_tables` is now deprecated and will be removed in version 1.0.0.  
-  Please migrate to the new `:meta_tables_after_main` option.
+  Please migrate to the new `:group_table_definition` option.
 
 ## 0.4.3    2024-09-22
 

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    activerecord-clean-db-structure (0.5.0)
+    activerecord-clean-db-structure (0.4.3)
       activerecord (>= 4.2)
 
 GEM

README.md

@@ -40,13 +40,13 @@ end
 
 ## Other options
 
-### meta_tables_after_main
+### group_table_definition
 
-You can optionally place indexes and constraints (limited to deferrable ones) after the main tables by setting meta_tables_after_main:
+You can optionally place indexes and constraints (limited to deferrable ones) after the main tables by setting group_table_definition:
 
 ```ruby
 Rails.application.configure do
-  config.activerecord_clean_db_structure.meta_tables_after_main = true
+  config.activerecord_clean_db_structure.group_table_definition = true
 end
 ```
 

lib/activerecord-clean-db-structure/clean_dump.rb

@@ -131,10 +131,10 @@ def clean_options
 
       # TODO: Remove support for :indexes_after_tables in version 1.0.0
       if options[:indexes_after_tables]
-        warn "[DEPRECATION] The :indexes_after_tables option is deprecated and will be removed in version 1.0.0. Use :meta_tables_after_main instead."
+        warn "[DEPRECATION] The :indexes_after_tables option is deprecated and will be removed in version 1.0.0. Use :group_table_definition instead."
       end
 
-      if options[:indexes_after_tables] || options[:meta_tables_after_main]
+      if options[:indexes_after_tables] || options[:group_table_definition]
         # Extract indexes
         indexes =
           dump

test/clean_dump_test.rb

@@ -1,37 +1,40 @@
-require "test_helper"
+require 'test_helper'
 
 class CleanDumpTest < Minitest::Test
   def test_basic_case
-    assert_cleans_dump "data/input.sql", "expectations/default_props.sql"
+    assert_cleans_dump 'data/input.sql', 'expectations/default_props.sql'
   end
 
   def test_ignore_ids
-    assert_cleans_dump "data/input.sql", "expectations/ignore_ids.sql", ignore_ids: true
+    assert_cleans_dump 'data/input.sql', 'expectations/ignore_ids.sql', ignore_ids: true
   end
 
   def test_order_column_definitions
-    assert_cleans_dump "data/input.sql", "expectations/order_column_definitions.sql", order_column_definitions: true
+    assert_cleans_dump 'data/input.sql', 'expectations/order_column_definitions.sql', order_column_definitions: true
   end
 
   def test_order_schema_migrations_values
-    assert_cleans_dump "data/input.sql", "expectations/order_schema_migrations_values.sql", order_schema_migrations_values: true
+    assert_cleans_dump 'data/input.sql', 'expectations/order_schema_migrations_values.sql',
+                       order_schema_migrations_values: true
   end
 
-  def test_meta_tables_after_main
-    assert_cleans_dump "data/input.sql", "expectations/meta_tables_after_main.sql", meta_tables_after_main: true
+  def test_group_table_definition
+    assert_cleans_dump 'data/input.sql', 'expectations/group_table_definition.sql', group_table_definition: true
   end
 
   def test_keep_extensions_all
-    assert_cleans_dump "data/input.sql", "expectations/keep_extensions_all.sql", keep_extensions: :all
+    assert_cleans_dump 'data/input.sql', 'expectations/keep_extensions_all.sql', keep_extensions: :all
   end
 
   def test_partitions
-    assert_cleans_dump "data/partitions.sql", "expectations/partitions.sql"
+    assert_cleans_dump 'data/partitions.sql', 'expectations/partitions.sql'
   end
 
   def test_ignored_schemas
-    assert_cleans_dump "data/ignored_schemas.sql", "expectations/ignored_schemas_pganalyze.sql", ignore_schemas: ['pganalyze']
-    assert_cleans_dump "data/ignored_schemas.sql", "expectations/ignored_schemas_myschema.sql", ignore_schemas: ['myschema']
+    assert_cleans_dump 'data/ignored_schemas.sql', 'expectations/ignored_schemas_pganalyze.sql',
+                       ignore_schemas: ['pganalyze']
+    assert_cleans_dump 'data/ignored_schemas.sql', 'expectations/ignored_schemas_myschema.sql',
+                       ignore_schemas: ['myschema']
   end
 
   private