diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml index a6613693b5b..f8759bdb474 100644 --- a/.github/workflows/tests.yml +++ b/.github/workflows/tests.yml @@ -159,6 +159,22 @@ jobs: path: tmp/turbo_rspec_runtime.log key: rspec-runtime-backend-core + - name: Run Zeitwerk check + if: matrix.build_type == 'backend' + env: + LOAD_PLUGINS: ${{ (matrix.target == 'plugins') && '1' || '0' }} + run: | + if ! bin/rails zeitwerk:check --trace; then + echo + echo "---------------------------------------------" + echo + echo "::error::'bin/rails zeitwerk:check' failed - the app will fail to boot with 'eager_load=true' (e.g. in production)." + echo "To reproduce locally, run 'bin/rails zeitwerk:check'." + echo "Alternatively, you can run your local server/tests with the 'DISCOURSE_ZEITWERK_EAGER_LOAD=1' environment variable." + echo + exit 1 + fi + - name: Core RSpec if: matrix.build_type == 'backend' && matrix.target == 'core' run: bin/turbo_rspec --verbose diff --git a/config/environments/development.rb b/config/environments/development.rb index cf50de16a28..1ea67794833 100644 --- a/config/environments/development.rb +++ b/config/environments/development.rb @@ -9,7 +9,7 @@ Discourse::Application.configure do config.cache_classes = false config.file_watcher = ActiveSupport::EventedFileUpdateChecker - config.eager_load = false + config.eager_load = ENV["DISCOURSE_ZEITWERK_EAGER_LOAD"] == "1" # Use the schema_cache.yml file generated during db:migrate (via db:schema:cache:dump) config.active_record.use_schema_cache_dump = true diff --git a/config/environments/test.rb b/config/environments/test.rb index e80e4f7a648..aa539725753 100644 --- a/config/environments/test.rb +++ b/config/environments/test.rb @@ -44,7 +44,7 @@ Discourse::Application.configure do config.assets.compile = true config.assets.digest = false - config.eager_load = false + config.eager_load = ENV["DISCOURSE_ZEITWERK_EAGER_LOAD"] == "1" if ENV["RAILS_ENABLE_TEST_LOG"] config.logger = Logger.new(STDOUT)