Merge branch 'main' into add-missing-status-to-scaffolded-destroy-method

Author: dhh

.github/workflows/ruby.yml

@@ -6,7 +6,7 @@ jobs:
   test:
     name: Ruby ${{ matrix.ruby }} (${{ matrix.gemfile }})
     runs-on: ubuntu-20.04
-    continue-on-error: ${{ matrix.experimental }}
+    continue-on-error: ${{ matrix.gemfile == 'rails_head' }}
     env:
       BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
       BUNDLE_JOBS: 4
@@ -15,116 +15,30 @@ jobs:
       fail-fast: false
       matrix:
         ruby:
-          - "2.2"
-          - "2.3"
-          - "2.4"
-          - "2.5"
-          - "2.6"
-          - "2.7"
           - "3.0"
           - "3.1"
+          - "3.2"
+          - "3.3"
 
         gemfile:
-          - "rails_5_0"
-          - "rails_5_1"
-          - "rails_5_2"
-          - "rails_6_0"
-          - "rails_6_1"
           - "rails_7_0"
+          - "rails_7_1"
           - "rails_head"
 
-        experimental: [false]
         exclude:
-          - ruby: 2.7
-            gemfile: rails_5_0
           - ruby: '3.0'
-            gemfile: rails_5_0
-          - ruby: '3.1'
-            gemfile: rails_5_0
-          - ruby: head
-            gemfile: rails_5_0
-          - ruby: 2.7
-            gemfile: rails_5_1
-          - ruby: '3.0'
-            gemfile: rails_5_1
-          - ruby: '3.1'
-            gemfile: rails_5_1
-          - ruby: head
-            gemfile: rails_5_1
-          - ruby: 2.2
-            gemfile: rails_5_2
-          - ruby: 2.7
-            gemfile: rails_5_2
-          - ruby: '3.0'
-            gemfile: rails_5_2
-          - ruby: '3.1'
-            gemfile: rails_5_2
-          - ruby: head
-            gemfile: rails_5_2
-          - ruby: 2.2
-            gemfile: rails_6_0
-          - ruby: 2.3
-            gemfile: rails_6_0
-          - ruby: 2.4
-            gemfile: rails_6_0
-          - ruby: '3.0'
-            gemfile: rails_6_0
-          - ruby: '3.1'
-            gemfile: rails_6_0
-          - ruby: head
-            gemfile: rails_6_0
-          - ruby: 2.2
-            gemfile: rails_6_1
-          - ruby: 2.3
-            gemfile: rails_6_1
-          - ruby: 2.4
-            gemfile: rails_6_1
-          - ruby: 2.2
-            gemfile: rails_7_0
-          - ruby: 2.3
-            gemfile: rails_7_0
-          - ruby: 2.4
-            gemfile: rails_7_0
-          - ruby: 2.5
-            gemfile: rails_7_0
-          - ruby: 2.6
-            gemfile: rails_7_0
-          - ruby: 2.2
-            gemfile: rails_head
-          - ruby: 2.3
             gemfile: rails_head
-          - ruby: 2.4
-            gemfile: rails_head
-          - ruby: 2.5
-            gemfile: rails_head
-          - ruby: 2.6
-            gemfile: rails_head
-          - ruby: 2.7
-            gemfile: rails_head
-            experimental: false
-          - ruby: '3.0'
-            gemfile: rails_head
-            experimental: false
-          - ruby: '3.1'
-            gemfile: rails_head
-            experimental: false
 
         include:
-          - ruby: 2.7
-            gemfile: rails_head
-            experimental: true
-          - ruby: '3.0'
-            gemfile: rails_head
-            experimental: true
           - ruby: '3.1'
             gemfile: rails_head
-            experimental: true
+          - ruby: '3.2'
+            gemfile: rails_head
           - ruby: head
             gemfile: rails_head
-            experimental: true
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
 
       - uses: ruby/setup-ruby@v1
         with:

.gitignore

@@ -1,4 +1,5 @@
 tmp
+/log
 gemfiles/.bundle
 gemfiles/*.lock
 Gemfile.lock

Appraisals

@@ -1,30 +1,12 @@
-appraise "rails-5-0" do
-  gem "rails", "~> 5.0.0"
-end
-
-appraise "rails-5-1" do
-  gem "rails", "~> 5.1.0"
-end
-
-appraise "rails-5-2" do
-  gem "rails", "~> 5.2.0"
-end
-
-if RUBY_VERSION >= "2.5.0"
-  appraise "rails-6-0" do
-    gem "rails", "~> 6.0.0"
-  end
-
-  appraise "rails-6-1" do
-    gem "rails", "~> 6.1.0"
-  end
-end
-
 if RUBY_VERSION >= "2.7.0"
   appraise "rails-7-0" do
     gem "rails", "~> 7.0.0"
   end
 
+  appraise "rails-7-1" do
+    gem "rails", "~> 7.1.0"
+  end
+
   appraise "rails-head" do
     gem "rails", github: "rails/rails", branch: "main"
   end

README.md

@@ -338,6 +338,10 @@ environment.rb for example):
 Jbuilder.deep_format_keys true
 ```
 
+## Testing JBuilder Response body with RSpec
+
+To test the response body of your controller spec, enable `render_views` in your RSpec context. This [configuration](https://rspec.info/features/6-0/rspec-rails/controller-specs/render-views) renders the views in a controller test.
+
 ## Contributing to Jbuilder
 
 Jbuilder is the work of many contributors. You're encouraged to submit pull requests, propose

gemfiles/rails_5_1.gemfile

@@ -5,6 +5,6 @@ source "https://rubygems.org"
 gem "rake"
 gem "mocha", require: false
 gem "appraisal"
-gem "rails", "~> 5.0.0"
+gem "rails", "~> 7.1.0"
 
 gemspec path: "../"

gemfiles/rails_5_2.gemfile

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name     = 'jbuilder'
-  s.version  = '2.11.5'
+  s.version  = '2.12.0'
   s.authors  = 'David Heinemeier Hansson'
   s.email    = '[email protected]'
   s.summary  = 'Create JSON structures via a Builder-style DSL'

gemfiles/rails_6_0.gemfile

@@ -30,7 +30,7 @@ def create
 
     respond_to do |format|
       if @<%= orm_instance.save %>
-        format.html { redirect_to <%= show_helper %>, notice: <%= %("#{human_name} was successfully created.") %> }
+        format.html { redirect_to <%= redirect_resource_name %>, notice: <%= %("#{human_name} was successfully created.") %> }
         format.json { render :show, status: :created, location: <%= "@#{singular_table_name}" %> }
       else
         format.html { render :new, status: :unprocessable_entity }
@@ -43,7 +43,7 @@ def create
   def update
     respond_to do |format|
       if @<%= orm_instance.update("#{singular_table_name}_params") %>
-        format.html { redirect_to <%= show_helper %>, notice: <%= %("#{human_name} was successfully updated.") %> }
+        format.html { redirect_to <%= redirect_resource_name %>, notice: <%= %("#{human_name} was successfully updated.") %> }
         format.json { render :show, status: :ok, location: <%= "@#{singular_table_name}" %> }
       else
         format.html { render :edit, status: :unprocessable_entity }
@@ -57,7 +57,7 @@ def destroy
     @<%= orm_instance.destroy %>
 
     respond_to do |format|
-      format.html { redirect_to <%= index_helper %>_url, status: :see_other, notice: <%= %("#{human_name} was successfully destroyed.") %> }
+      format.html { redirect_to <%= index_helper %>_path, status: :see_other, notice: <%= %("#{human_name} was successfully destroyed.") %> }
       format.json { head :no_content }
     end
   end

gemfiles/rails_6_1.gemfile

@@ -4,8 +4,11 @@
 require 'jbuilder/key_formatter'
 require 'jbuilder/errors'
 require 'json'
-require 'ostruct'
 require 'active_support/core_ext/hash/deep_merge'
+begin
+  require 'ostruct'
+rescue LoadError
+end
 
 class Jbuilder
   @@key_formatter = nil
@@ -28,7 +31,7 @@ def self.encode(*args, &block)
   end
 
   BLANK = Blank.new
-  NON_ENUMERABLES = [ ::Struct, ::OpenStruct ].to_set
+  NON_ENUMERABLES = defined?(::OpenStruct) ? [::Struct, ::OpenStruct].to_set : [::Struct].to_set
 
   def set!(key, value = BLANK, *args, &block)
     result = if ::Kernel.block_given?
@@ -292,7 +295,7 @@ def _extract_method_values(object, attributes)
 
   def _merge_block(key)
     current_value = _blank? ? BLANK : @attributes.fetch(_key(key), BLANK)
-    raise NullError.build(key) if current_value.nil?
+    ::Kernel.raise NullError.build(key) if current_value.nil?
     new_value = _scope{ yield self }
     _merge_values(current_value, new_value)
   end
@@ -307,7 +310,7 @@ def _merge_values(current_value, updates)
     elsif ::Hash === current_value && ::Hash === updates
       current_value.deep_merge(updates)
     else
-      raise MergeError.build(current_value, updates)
+      ::Kernel.raise MergeError.build(current_value, updates)
     end
   end
 
@@ -328,8 +331,8 @@ def _format_keys(hash_or_array)
   end
 
   def _set_value(key, value)
-    raise NullError.build(key) if @attributes.nil?
-    raise ArrayError.build(key) if ::Array === @attributes
+    ::Kernel.raise NullError.build(key) if @attributes.nil?
+    ::Kernel.raise ArrayError.build(key) if ::Array === @attributes
     return if @ignore_nil && value.nil? or _blank?(value)
     @attributes = {} if _blank?
     @attributes[_key(key)] = value

gemfiles/rails_5_0.gemfile gemfiles/rails_7_1.gemfile

@@ -1,7 +1 @@
-Jbuilder = Class.new(begin
-  require 'active_support/proxy_object'
-  ActiveSupport::ProxyObject
-rescue LoadError
-  require 'active_support/basic_object'
-  ActiveSupport::BasicObject
-end)
+Jbuilder = Class.new(BasicObject)