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

@@ -17,8 +17,12 @@ def test_order_schema_migrations_values
     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_indexes_after_tables
+    assert_cleans_dump "data/input.sql", "expectations/indexes_after_tables.sql", indexes_after_tables: true
+  end
+
+  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

test/expectations/meta_tables_after_main.sql renamed to test/expectations/group_table_definition.sql

@@ -0,0 +1,73 @@
+SET statement_timeout = 0;
+SET lock_timeout = 0;
+SET client_encoding = 'UTF8';
+SET standard_conforming_strings = on;
+SELECT pg_catalog.set_config('search_path', '', false);
+SET check_function_bodies = false;
+SET client_min_messages = warning;
+
+SET default_tablespace = '';
+
+-- Name: ar_internal_metadata; Type: TABLE
+
+CREATE TABLE public.ar_internal_metadata (
+    key character varying NOT NULL,
+    value character varying,
+    created_at timestamp(6) without time zone NOT NULL,
+    updated_at timestamp(6) without time zone NOT NULL
+);
+
+-- Name: delayed_jobs; Type: TABLE
+
+CREATE TABLE public.delayed_jobs (
+    id BIGSERIAL PRIMARY KEY,
+    priority integer DEFAULT 0,
+    attempts integer DEFAULT 0,
+    handler text,
+    last_error text,
+    run_at timestamp without time zone,
+    locked_at timestamp without time zone,
+    failed_at timestamp without time zone,
+    locked_by character varying(255),
+    queue character varying(255),
+    created_at timestamp without time zone NOT NULL,
+    updated_at timestamp without time zone NOT NULL,
+    metadata jsonb DEFAULT '{}'::jsonb,
+    whodunnit text
+)
+WITH (fillfactor='85');
+
+CREATE INDEX index_delayed_jobs_on_locked_by ON public.delayed_jobs USING btree (locked_by);
+CREATE INDEX index_delayed_jobs_on_queue ON public.delayed_jobs USING btree (queue);
+CREATE INDEX "index_delayed_jobs_on_failed_at_IS_NULL" ON public.delayed_jobs USING btree (((failed_at IS NULL)));
+CREATE INDEX index_delayed_jobs_on_run_at ON public.delayed_jobs USING btree (run_at) WHERE (locked_at IS NULL);
+ALTER TABLE ONLY public.delayed_jobs
+    ADD CONSTRAINT unique_locked_by_locked_at UNIQUE (locked_by, locked_at) DEFERRABLE INITIALLY DEFERRED;
+
+-- Name: schema_migrations; Type: TABLE
+
+CREATE TABLE public.schema_migrations (
+    version character varying NOT NULL
+);
+
+ALTER TABLE ONLY public.ar_internal_metadata
+    ADD CONSTRAINT ar_internal_metadata_pkey PRIMARY KEY (key);
+
+ALTER TABLE ONLY public.schema_migrations
+    ADD CONSTRAINT schema_migrations_pkey PRIMARY KEY (version);
+
+-- PostgreSQL database dump complete
+
+SET search_path TO "$user", public;
+
+INSERT INTO "schema_migrations" (version) VALUES
+('20240822225012'),
+('20240822224954'),
+('20240725043656'),
+('20240621041110'),
+('20240621020038'),
+('20220802204003'),
+('20211125055031'),
+('20211012054749'),
+('20210923052631'),
+('20210903003251');