Merge pull request #3340 from projectblacklight/rails8

Test on Rails 8

Author: jcoyne

.github/workflows/ruby.yml

@@ -43,6 +43,9 @@ jobs:
         additional_engine_cart_rails_options: [""]
         additional_name: [""]
         include:
+          - ruby: "3.3"
+            rails_version: "8.0.0.beta1"
+            additional_engine_cart_rails_options: --css=bootstrap
           - ruby: "3.2"
             rails_version: "7.0.8.4"
           - ruby: "3.2"
@@ -66,7 +69,7 @@ jobs:
       VIEW_COMPONENT_VERSION: ${{ matrix.view_component_version }}
       BOOTSTRAP_VERSION: ${{ matrix.bootstrap_version }}
       BLACKLIGHT_API_TEST: ${{ matrix.api }}
-      ENGINE_CART_RAILS_OPTIONS: "--skip-git --skip-listen --skip-spring --skip-keeps --skip-coffee --skip-test ${{ matrix.additional_engine_cart_rails_options }}"
+      ENGINE_CART_RAILS_OPTIONS: "--skip-git --skip-listen --skip-spring --skip-keeps --skip-kamal --skip-solid --skip-coffee --skip-test ${{ matrix.additional_engine_cart_rails_options }}"
     steps:
       - uses: actions/checkout@v4
       - name: Set up Ruby

blacklight.gemspec

@@ -25,7 +25,7 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = '>= 2.7'
 
-  s.add_dependency "rails", '>= 7.0', '< 8'
+  s.add_dependency "rails", '>= 7.0', '< 9'
   s.add_dependency "globalid"
   s.add_dependency "jbuilder", '~> 2.7'
   s.add_dependency "kaminari", ">= 0.15" # the pagination (page 1,2,3, etc..) of our search results

lib/generators/blacklight/assets/importmap_generator.rb

@@ -19,6 +19,8 @@ def import_javascript_assets
           CONTENT
         end
 
+        return unless defined?(Sprockets)
+
         append_to_file 'app/assets/config/manifest.js' do
           <<~CONTENT
             //= link blacklight/manifest.js
@@ -27,23 +29,43 @@ def import_javascript_assets
       end
 
       def append_blacklight_javascript
+        # This may already be present if rails new was invoked with `--css bootstrap'
+        append_to_file 'app/javascript/application.js' do
+          <<~CONTENT
+            import * as bootstrap from "bootstrap"
+          CONTENT
+        end
+
         append_to_file 'app/javascript/application.js' do
           <<~CONTENT
-            import bootstrap from "bootstrap"
             import githubAutoCompleteElement from "@github/auto-complete-element"
             import Blacklight from "blacklight"
           CONTENT
         end
       end
 
       def add_stylesheet
-        gem "sassc-rails", "~> 2.1"
+        if File.exist? 'app/assets/stylesheets/application.bootstrap.scss'
+          if ENV['CI']
+            run "yarn add file:#{Blacklight::Engine.root}"
+          else
+            run "yarn add blacklight-frontend@#{Blacklight::VERSION}"
+          end
 
-        create_file 'app/assets/stylesheets/blacklight.scss' do
-          <<~CONTENT
-            @import 'bootstrap';
-            @import 'blacklight/blacklight';
-          CONTENT
+          append_to_file 'app/assets/stylesheets/application.bootstrap.scss' do
+            <<~CONTENT
+              @import "blacklight-frontend/app/assets/stylesheets/blacklight/blacklight";
+            CONTENT
+          end
+        else
+          gem "sassc-rails", "~> 2.1"
+
+          create_file 'app/assets/stylesheets/blacklight.scss' do
+            <<~CONTENT
+              @import 'bootstrap';
+              @import 'blacklight/blacklight';
+            CONTENT
+          end
         end
       end
     end

lib/generators/blacklight/assets_generator.rb

@@ -7,10 +7,10 @@ class AssetsGenerator < Rails::Generators::Base
     def run_asset_pipeline_specific_generator
       generated_options = "--bootstrap-version='#{options[:'bootstrap-version']}'" if options[:'bootstrap-version']
 
-      generator = if defined?(Propshaft)
-                    'blacklight:assets:propshaft'
-                  elsif defined?(Importmap)
+      generator = if defined?(Importmap)
                     'blacklight:assets:importmap'
+                  elsif defined?(Propshaft)
+                    'blacklight:assets:propshaft'
                   elsif defined?(Sprockets)
                     'blacklight:assets:sprockets'
                   end

spec/test_app_templates/lib/generators/test_app_generator.rb

@@ -29,12 +29,6 @@ def run_test_support_generator
     generate(:'blacklight:test_support')
   end
 
-  def add_local_assets_for_propshaft
-    return unless defined?(Propshaft)
-
-    run "yarn add #{Blacklight::Engine.root}"
-  end
-
   def add_component_template_override
     src_template = File.join(Blacklight::Engine.root, 'app', 'components', 'blacklight', 'top_navbar_component.html.erb')
     target_template = File.join('app', 'components', 'blacklight', 'top_navbar_component.html.erb')