Fix warnings and prevent Jbuilder::VERSION constant from being obliterated

.github/workflows/ruby.yml

@@ -23,15 +23,22 @@ jobs:
         gemfile:
           - "rails_7_0"
           - "rails_7_1"
+          - "rails_7_2"
           - "rails_head"
 
         exclude:
+          - ruby: '3.0'
+            gemfile: rails_7_2
           - ruby: '3.0'
             gemfile: rails_head
 
         include:
+          - ruby: '3.1'
+            gemfile: rails_7_2
           - ruby: '3.1'
             gemfile: rails_head
+          - ruby: '3.2'
+            gemfile: rails_7_2
           - ruby: '3.2'
             gemfile: rails_head
           - ruby: head

Appraisals

@@ -7,6 +7,10 @@ if RUBY_VERSION >= "2.7.0"
     gem "rails", "~> 7.1.0"
   end
 
+  appraise "rails-7-2" do
+    gem "rails", "~> 7.2.0"
+  end
+
   appraise "rails-head" do
     gem "rails", github: "rails/rails", branch: "main"
   end

bin/test

@@ -1,4 +1,4 @@
-#!/bin/env bash
+#!/usr/bin/env bash
 set -e
 
 bundle install

gemfiles/rails_7_2.gemfile

@@ -0,0 +1,10 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rake"
+gem "mocha", require: false
+gem "appraisal"
+gem "rails", "~> 7.2.0"
+
+gemspec path: "../"

lib/jbuilder.rb

@@ -3,7 +3,6 @@
 require 'jbuilder/blank'
 require 'jbuilder/key_formatter'
 require 'jbuilder/errors'
-require 'jbuilder/version'
 require 'json'
 require 'active_support/core_ext/hash/deep_merge'
 

lib/jbuilder/errors.rb

@@ -1,4 +1,4 @@
-require 'jbuilder/jbuilder'
+require 'jbuilder/version'
 
 class Jbuilder
   class NullError < ::NoMethodError

lib/jbuilder/jbuilder.rb

@@ -1 +1 @@
-Jbuilder = Class.new(BasicObject)
+require 'jbuilder/version'

lib/jbuilder/version.rb

@@ -1,3 +1,3 @@
-class Jbuilder
+class Jbuilder < BasicObject
   VERSION = "2.13.0"
 end

test/scaffold_api_controller_generator_test.rb

@@ -6,7 +6,7 @@
 
   class ScaffoldApiControllerGeneratorTest < Rails::Generators::TestCase
     tests Rails::Generators::ScaffoldControllerGenerator
-    arguments %w(Post title body:text images:attachments --api)
+    arguments %w(Post title body:text images:attachments --api --skip-routes)
     destination File.expand_path('../tmp', __FILE__)
     setup :prepare_destination
 
@@ -57,7 +57,7 @@ class ScaffoldApiControllerGeneratorTest < Rails::Generators::TestCase
     end
 
     test "don't use require and permit if there are no attributes" do
-      run_generator %w(Post --api)
+      run_generator %w(Post --api --skip-routes)
 
       assert_file 'app/controllers/posts_controller.rb' do |content|
         assert_match %r{def post_params}, content
@@ -68,7 +68,7 @@ class ScaffoldApiControllerGeneratorTest < Rails::Generators::TestCase
 
     if Rails::VERSION::MAJOR >= 6
       test 'handles virtual attributes' do
-        run_generator ["Message", "content:rich_text", "video:attachment", "photos:attachments"]
+        run_generator %w(Message content:rich_text video:attachment photos:attachments --skip-routes)
 
         assert_file 'app/controllers/messages_controller.rb' do |content|
           if Rails::VERSION::MAJOR >= 8

test/scaffold_controller_generator_test.rb

@@ -4,7 +4,7 @@
 
 class ScaffoldControllerGeneratorTest < Rails::Generators::TestCase
   tests Rails::Generators::ScaffoldControllerGenerator
-  arguments %w(Post title body:text images:attachments)
+  arguments %w(Post title body:text images:attachments --skip-routes)
   destination File.expand_path('../tmp', __FILE__)
   setup :prepare_destination
 
@@ -70,7 +70,7 @@ class ScaffoldControllerGeneratorTest < Rails::Generators::TestCase
 
   if Rails::VERSION::MAJOR >= 6
     test 'controller with namespace' do
-      run_generator %w(Admin::Post --model-name=Post)
+      run_generator %w(Admin::Post --model-name=Post --skip-routes)
       assert_file 'app/controllers/admin/posts_controller.rb' do |content|
         assert_instance_method :create, content do |m|
           assert_match %r{format\.html \{ redirect_to \[:admin, @post\], notice: "Post was successfully created\." \}}, m
@@ -88,7 +88,7 @@ class ScaffoldControllerGeneratorTest < Rails::Generators::TestCase
   end
 
   test "don't use require and permit if there are no attributes" do
-    run_generator %w(Post)
+    run_generator %w(Post --skip-routes)
 
     assert_file 'app/controllers/posts_controller.rb' do |content|
       assert_match %r{def post_params}, content
@@ -98,7 +98,7 @@ class ScaffoldControllerGeneratorTest < Rails::Generators::TestCase
 
   if Rails::VERSION::MAJOR >= 6
     test 'handles virtual attributes' do
-      run_generator %w(Message content:rich_text video:attachment photos:attachments)
+      run_generator %w(Message content:rich_text video:attachment photos:attachments --skip-routes)
 
       assert_file 'app/controllers/messages_controller.rb' do |content|
         if Rails::VERSION::MAJOR >= 8

test/test_helper.rb

@@ -19,7 +19,7 @@
 ENV["RAILS_ENV"] ||= "test"
 
 class << Rails
-  def cache
+  redefine_method :cache do
     @cache ||= ActiveSupport::Cache::MemoryStore.new
   end
 end