Serve reports using Rails rather than Webrick.

Author: gregschmit

Gemfile

@@ -76,7 +76,6 @@ group :development do
   gem "pry-rails"
   gem "rubocop-shopify", require: false
   gem "ruby-lsp", require: false
-  gem "webrick"
   gem "web-console"
 
   # Vendoring external assets.

Gemfile.lock

@@ -353,7 +353,6 @@ GEM
       activemodel (>= 6.0.0)
       bindex (>= 0.4.0)
       railties (>= 6.0.0)
-    webrick (1.9.1)
     websocket-driver (0.7.7)
       base64
       websocket-extensions (>= 0.1.0)
@@ -400,7 +399,6 @@ DEPENDENCIES
   stackprof
   translate_enum
   web-console
-  webrick
 
 RUBY VERSION
    ruby 3.2.2p53

Procfile.dev

@@ -1,4 +1,3 @@
 web: env RUBY_DEBUG_OPEN=true bin/rails server -p 3000
 worker: bin/jobs
 reloader: bin/reloader
-reports: bin/reports_server

README.md

@@ -153,8 +153,8 @@ using RVM. Then run `bin/setup` to install the appropriate gems and set things u
 
 The top-level `bin/rails` proxies all Rails commands to the test project, so you can operate it via
 the usual commands (e.g., `rails test`, `rails console`). For development, use `bin/dev` to run the
-web server and the job queue, which serves the test app  and coverage/brakeman reports:
+web server and the job queue, which serves the test app and coverage/brakeman reports:
 
 - Test App: [http://127.0.0.1:3000](http://127.0.0.1:3000)
-- Test App API: [http://127.0.0.1:3000/api](http://127.0.0.1:3000/api)
-- Reports: [http://127.0.0.1:3001](http://127.0.0.1:3001)
+- API: [http://127.0.0.1:3000/api](http://127.0.0.1:3000/api)
+- Reports: [http://127.0.0.1:3000/reports](http://127.0.0.1:3000/reports)

bin/reloader

@@ -23,7 +23,7 @@ def reload(skip_seed: false)
   system("kill -USR1 $(cat #{PUMA_PID_PATH})")
 
   # Run the test suite (generates coverage report).
-  system("#{"SKIP_SEED=1" if skip_seed} bin/rails test".strip)
+  system("#{"SKIP_SEED=1" if skip_seed} SIMPLECOV_INLINE_ASSETS=1 bin/rails test".strip)
 
   # Generate brakeman report.
   system("bin/brakeman")

bin/reports_server

@@ -1,4 +1,4 @@
 ---
 :output_files:
-  - reports/brakeman.html
+  - test/public/reports/brakeman.html
 :combine_locations: false

config/brakeman.yml

@@ -19,7 +19,7 @@
 
   # The `rest_framework` project directory should be the root for coverage purposes.
   root ".."
-  coverage_dir "reports/coverage"
+  coverage_dir "test/public/reports/coverage"
 
   # Filter out everything but the lib directory.
   add_filter "app/"