Refactor model to modular concerns and add Rails 8.1 support

Author: sjke

.github/workflows/tests.yml

@@ -1,20 +1,24 @@
 name: Gem Tests
 
-on: 
+on:
   pull_request:
   push:
     branches:
       - master
 
 jobs:
-  run:
-    name: Run Tests for Ruby ${{ matrix.ruby }}
-
+  test:
     runs-on: ubuntu-latest
+    name: Ruby ${{ matrix.ruby }}
+    strategy:
+      fail-fast: false
+      matrix:
+        # Testing against all major supported Ruby versions
+        ruby: ['4.0', '3.5', '3.4', '3.3', '3.2', '3.1', '3.0', '2.7']
 
     services:
       postgres:
-        image: postgres
+        image: postgres:16
         env:
           POSTGRES_PASSWORD: postgres
           POSTGRES_DB: pg_ltree_test
@@ -26,20 +30,6 @@ jobs:
           --health-timeout 5s
           --health-retries 5
 
-    strategy:
-      fail-fast: false
-      matrix:
-        ruby:
-          - 'head'
-          - '3.4'
-          - '3.3'
-          - '3.2'
-          - '3.1'
-          - '3.0'
-          - '2.7'
-
-    continue-on-error: ${{ matrix.ruby == 'head' }}
-
     steps:
       - uses: actions/checkout@v4
 
@@ -49,11 +39,29 @@ jobs:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
 
-      - name: Setup DB
+      - name: Setup DB config
         run: cp spec/database.yml.sample spec/database.yml
-      
-      - name: Install dependencies
+
+      - name: Install Appraisal dependencies
         run: bundle exec appraisal install
 
-      - name: Run tests
+      - name: Run Tests (Appraisal)
         run: bundle exec appraisal rake spec
+        env:
+          PGHOST: localhost
+          PGUSER: postgres
+          PGPASSWORD: postgres
+
+  # Dedicated job for linting to fail fast
+  lint:
+    runs-on: ubuntu-latest
+    name: Linting
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '3.4'
+          bundler-cache: true
+      - name: Run Lint
+        run: bundle exec rake lint

.gitignore

@@ -8,3 +8,11 @@ doc
 coverage
 database.yml
 *.lock
+
+# .graphifyignore
+vendor/
+node_modules/
+dist/
+*.generated.py
+graphify-out/
+.agents/

Appraisals

@@ -17,7 +17,7 @@ def add_appraise_for(activerecord_version:, pg_version:)
   end
 end
 
-SUPPORTED_PG_VERSIONS = ["~> 1.0", "~> 1.1", "~> 1.2", "~> 1.3", "~> 1.4", "~> 1.5"]
+SUPPORTED_PG_VERSIONS = ["~> 1.0", "~> 1.1", "~> 1.2", "~> 1.3", "~> 1.4", "~> 1.5", "~> 1.6"]
 
 if Gem::Version.new(RUBY_VERSION) <= Gem::Version.new("3.0")
   SUPPORTED_PG_VERSIONS.map do |pg_version|
@@ -46,5 +46,6 @@ end
 if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.2")
   SUPPORTED_PG_VERSIONS.map do |pg_version|
     add_appraise_for(activerecord_version: "~> 8.0", pg_version: pg_version)
+    add_appraise_for(activerecord_version: "~> 8.1", pg_version: pg_version)
   end
 end

Rakefile

@@ -3,6 +3,15 @@ require "standard/rake"
 require "rspec/core/rake_task"
 require "appraisal"
 
+desc "Run all tests"
 RSpec::Core::RakeTask.new(:spec)
 
-task default: :spec
+desc "Run linting"
+task lint: :standard
+
+desc "Run all tests across all appraisals"
+task :test_all do
+  sh "bundle exec appraisal rake spec"
+end
+
+task default: [:lint, :spec]

gemfiles/activerecord_60_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 6.0", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"

gemfiles/activerecord_61_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 6.1", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"

gemfiles/activerecord_70_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 7.0", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"

gemfiles/activerecord_71_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 7.1", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"

gemfiles/activerecord_72_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 7.2", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"

gemfiles/activerecord_80_pg_16.gemfile

@@ -0,0 +1,8 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "activerecord", "~> 8.0", require: "active_record"
+gem "pg", "~> 1.6"
+
+gemspec path: "../"