Upgrade to Rails 8.1.2 with latest gem versions (#18)

* Upgrade to Rails 8.1.2 with latest gem versions

- Rails 7.2 → 8.1.2 (latest)
- puma 6.x → 7.2.0
- will_paginate 3.3 → 4.0.1
- rspec-rails 6.1 → 8.0.3
- sqlite3 → 2.9.0
- annotate → annotaterb 4.22.0 (Rails 8 compatible successor)
- Update config.load_defaults to 8.1
- Remove deprecated active_support.deprecation config (removed in Rails 8)
- Add x86_64-linux platform to Gemfile.lock for CI
- CI: use rake db:migrate and set SECRET_KEY_BASE for all steps

All 119 tests passing.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Move test SECRET_KEY_BASE to config and restore deprecation warnings (#19)

* Initial plan

* Apply PR review feedback: move SECRET_KEY_BASE to test.rb and restore deprecation warnings

Co-authored-by: ngallegos <1068686+ngallegos@users.noreply.github.com>

* Add security warning comment for test secret key

Co-authored-by: ngallegos <1068686+ngallegos@users.noreply.github.com>

* Use realistic-length secret key for better production simulation

Co-authored-by: ngallegos <1068686+ngallegos@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ngallegos <1068686+ngallegos@users.noreply.github.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: ngallegos <1068686+ngallegos@users.noreply.github.com>

Author: 3 people

.github/workflows/ci.yml

@@ -22,10 +22,9 @@ jobs:
       - name: Set up database
         env:
           RAILS_ENV: test
-        run: bundle exec rails db:migrate
+        run: bundle exec rake db:migrate
 
       - name: Run tests
         env:
           RAILS_ENV: test
-          SECRET_KEY_BASE: test_secret_key_base_for_ci_only
         run: bundle exec rspec

Gemfile

@@ -1,21 +1,20 @@
 source 'https://rubygems.org'
 
-gem 'rails', '~> 7.2'
-gem 'sqlite3', '~> 2.0'
-gem 'puma', '~> 6.0'
+gem 'rails', '~> 8.1'
+gem 'sqlite3', '~> 2.9'
+gem 'puma', '~> 7.0'
 gem 'bcrypt', '~> 3.1'
-gem 'gravatar_image_tag', '~> 1.2'
-gem 'will_paginate', '~> 3.3'
-gem 'google_visualr', '~> 2.3'
+gem 'will_paginate', '~> 4.0'
+gem 'google_visualr', '~> 2.5'
 
 group :development do
-  gem 'annotate', '~> 3.2'
+  gem 'annotaterb', '~> 4.0'
 end
 
 group :development, :test do
-  gem 'rspec-rails', '~> 6.1'
-  gem 'factory_bot_rails', '~> 6.4'
-  gem 'faker', '~> 3.2'
+  gem 'rspec-rails', '~> 8.0'
+  gem 'factory_bot_rails', '~> 6.5'
+  gem 'faker', '~> 3.6'
   gem 'rails-controller-testing'
 end
 

Gemfile.lock

@@ -1,87 +1,87 @@
 GEM
   remote: https://rubygems.org/
   specs:
-    actioncable (7.2.3)
-      actionpack (= 7.2.3)
-      activesupport (= 7.2.3)
+    action_text-trix (2.1.16)
+      railties
+    actioncable (8.1.2)
+      actionpack (= 8.1.2)
+      activesupport (= 8.1.2)
       nio4r (~> 2.0)
       websocket-driver (>= 0.6.1)
       zeitwerk (~> 2.6)
-    actionmailbox (7.2.3)
-      actionpack (= 7.2.3)
-      activejob (= 7.2.3)
-      activerecord (= 7.2.3)
-      activestorage (= 7.2.3)
-      activesupport (= 7.2.3)
+    actionmailbox (8.1.2)
+      actionpack (= 8.1.2)
+      activejob (= 8.1.2)
+      activerecord (= 8.1.2)
+      activestorage (= 8.1.2)
+      activesupport (= 8.1.2)
       mail (>= 2.8.0)
-    actionmailer (7.2.3)
-      actionpack (= 7.2.3)
-      actionview (= 7.2.3)
-      activejob (= 7.2.3)
-      activesupport (= 7.2.3)
+    actionmailer (8.1.2)
+      actionpack (= 8.1.2)
+      actionview (= 8.1.2)
+      activejob (= 8.1.2)
+      activesupport (= 8.1.2)
       mail (>= 2.8.0)
       rails-dom-testing (~> 2.2)
-    actionpack (7.2.3)
-      actionview (= 7.2.3)
-      activesupport (= 7.2.3)
-      cgi
+    actionpack (8.1.2)
+      actionview (= 8.1.2)
+      activesupport (= 8.1.2)
       nokogiri (>= 1.8.5)
-      racc
-      rack (>= 2.2.4, < 3.3)
+      rack (>= 2.2.4)
       rack-session (>= 1.0.1)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.2)
       rails-html-sanitizer (~> 1.6)
       useragent (~> 0.16)
-    actiontext (7.2.3)
-      actionpack (= 7.2.3)
-      activerecord (= 7.2.3)
-      activestorage (= 7.2.3)
-      activesupport (= 7.2.3)
+    actiontext (8.1.2)
+      action_text-trix (~> 2.1.15)
+      actionpack (= 8.1.2)
+      activerecord (= 8.1.2)
+      activestorage (= 8.1.2)
+      activesupport (= 8.1.2)
       globalid (>= 0.6.0)
       nokogiri (>= 1.8.5)
-    actionview (7.2.3)
-      activesupport (= 7.2.3)
+    actionview (8.1.2)
+      activesupport (= 8.1.2)
       builder (~> 3.1)
-      cgi
       erubi (~> 1.11)
       rails-dom-testing (~> 2.2)
       rails-html-sanitizer (~> 1.6)
-    activejob (7.2.3)
-      activesupport (= 7.2.3)
+    activejob (8.1.2)
+      activesupport (= 8.1.2)
       globalid (>= 0.3.6)
-    activemodel (7.2.3)
-      activesupport (= 7.2.3)
-    activerecord (7.2.3)
-      activemodel (= 7.2.3)
-      activesupport (= 7.2.3)
+    activemodel (8.1.2)
+      activesupport (= 8.1.2)
+    activerecord (8.1.2)
+      activemodel (= 8.1.2)
+      activesupport (= 8.1.2)
       timeout (>= 0.4.0)
-    activestorage (7.2.3)
-      actionpack (= 7.2.3)
-      activejob (= 7.2.3)
-      activerecord (= 7.2.3)
-      activesupport (= 7.2.3)
+    activestorage (8.1.2)
+      actionpack (= 8.1.2)
+      activejob (= 8.1.2)
+      activerecord (= 8.1.2)
+      activesupport (= 8.1.2)
       marcel (~> 1.0)
-    activesupport (7.2.3)
+    activesupport (8.1.2)
       base64
-      benchmark (>= 0.3)
       bigdecimal
       concurrent-ruby (~> 1.0, >= 1.3.1)
       connection_pool (>= 2.2.5)
       drb
       i18n (>= 1.6, < 2)
+      json
       logger (>= 1.4.2)
       minitest (>= 5.1)
       securerandom (>= 0.3)
       tzinfo (~> 2.0, >= 2.0.5)
+      uri (>= 0.13.1)
     addressable (2.8.8)
       public_suffix (>= 2.0.2, < 8.0)
-    annotate (3.2.0)
-      activerecord (>= 3.2, < 8.0)
-      rake (>= 10.4, < 14.0)
+    annotaterb (4.22.0)
+      activerecord (>= 6.0.0)
+      activesupport (>= 6.0.0)
     base64 (0.3.0)
     bcrypt (3.1.21)
-    benchmark (0.5.0)
     bigdecimal (4.0.1)
     builder (3.3.0)
     capybara (3.40.0)
@@ -93,7 +93,6 @@ GEM
       rack-test (>= 0.6.3)
       regexp_parser (>= 1.5, < 3.0)
       xpath (~> 3.2)
-    cgi (0.5.1)
     concurrent-ruby (1.3.6)
     connection_pool (3.0.2)
     crass (1.0.6)
@@ -112,7 +111,6 @@ GEM
     globalid (1.3.0)
       activesupport (>= 6.1)
     google_visualr (2.5.1)
-    gravatar_image_tag (1.2.0)
     i18n (1.14.8)
       concurrent-ruby (~> 1.0)
     io-console (0.8.2)
@@ -121,6 +119,7 @@ GEM
       prism (>= 1.3.0)
       rdoc (>= 4.0.0)
       reline (>= 0.4.2)
+    json (2.18.1)
     logger (1.7.0)
     loofah (2.25.0)
       crass (~> 1.0.2)
@@ -162,7 +161,7 @@ GEM
       date
       stringio
     public_suffix (7.0.2)
-    puma (6.6.1)
+    puma (7.2.0)
       nio4r (~> 2.0)
     racc (1.8.1)
     rack (3.2.5)
@@ -173,20 +172,20 @@ GEM
       rack (>= 1.3)
     rackup (2.3.1)
       rack (>= 3)
-    rails (7.2.3)
-      actioncable (= 7.2.3)
-      actionmailbox (= 7.2.3)
-      actionmailer (= 7.2.3)
-      actionpack (= 7.2.3)
-      actiontext (= 7.2.3)
-      actionview (= 7.2.3)
-      activejob (= 7.2.3)
-      activemodel (= 7.2.3)
-      activerecord (= 7.2.3)
-      activestorage (= 7.2.3)
-      activesupport (= 7.2.3)
+    rails (8.1.2)
+      actioncable (= 8.1.2)
+      actionmailbox (= 8.1.2)
+      actionmailer (= 8.1.2)
+      actionpack (= 8.1.2)
+      actiontext (= 8.1.2)
+      actionview (= 8.1.2)
+      activejob (= 8.1.2)
+      activemodel (= 8.1.2)
+      activerecord (= 8.1.2)
+      activestorage (= 8.1.2)
+      activesupport (= 8.1.2)
       bundler (>= 1.15.0)
-      railties (= 7.2.3)
+      railties (= 8.1.2)
     rails-controller-testing (1.0.5)
       actionpack (>= 5.0.1.rc1)
       actionview (>= 5.0.1.rc1)
@@ -198,10 +197,9 @@ GEM
     rails-html-sanitizer (1.6.2)
       loofah (~> 2.21)
       nokogiri (>= 1.15.7, != 1.16.7, != 1.16.6, != 1.16.5, != 1.16.4, != 1.16.3, != 1.16.2, != 1.16.1, != 1.16.0.rc1, != 1.16.0)
-    railties (7.2.3)
-      actionpack (= 7.2.3)
-      activesupport (= 7.2.3)
-      cgi
+    railties (8.1.2)
+      actionpack (= 8.1.2)
+      activesupport (= 8.1.2)
       irb (~> 1.13)
       rackup (>= 1.0.0)
       rake (>= 12.2)
@@ -224,10 +222,10 @@ GEM
     rspec-mocks (3.13.7)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-rails (6.1.5)
-      actionpack (>= 6.1)
-      activesupport (>= 6.1)
-      railties (>= 6.1)
+    rspec-rails (8.0.3)
+      actionpack (>= 7.2)
+      activesupport (>= 7.2)
+      railties (>= 7.2)
       rspec-core (~> 3.13)
       rspec-expectations (~> 3.13)
       rspec-mocks (~> 3.13)
@@ -244,12 +242,13 @@ GEM
     tsort (0.2.0)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
+    uri (1.1.1)
     useragent (0.16.11)
     websocket-driver (0.8.0)
       base64
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
-    will_paginate (3.3.1)
+    will_paginate (4.0.1)
     xpath (3.2.0)
       nokogiri (~> 1.8)
     zeitwerk (2.7.4)
@@ -260,19 +259,18 @@ PLATFORMS
   x86_64-linux
 
 DEPENDENCIES
-  annotate (~> 3.2)
+  annotaterb (~> 4.0)
   bcrypt (~> 3.1)
   capybara (~> 3.40)
-  factory_bot_rails (~> 6.4)
-  faker (~> 3.2)
-  google_visualr (~> 2.3)
-  gravatar_image_tag (~> 1.2)
-  puma (~> 6.0)
-  rails (~> 7.2)
+  factory_bot_rails (~> 6.5)
+  faker (~> 3.6)
+  google_visualr (~> 2.5)
+  puma (~> 7.0)
+  rails (~> 8.1)
   rails-controller-testing
-  rspec-rails (~> 6.1)
-  sqlite3 (~> 2.0)
-  will_paginate (~> 3.3)
+  rspec-rails (~> 8.0)
+  sqlite3 (~> 2.9)
+  will_paginate (~> 4.0)
 
 BUNDLED WITH
   4.0.3

config/application.rb

@@ -6,7 +6,7 @@
 
 module Run500Miles
   class Application < Rails::Application
-    config.load_defaults 7.2
+    config.load_defaults 8.1
 
     # Filter sensitive parameters from logs
     config.filter_parameters += [:password]

config/environments/development.rb

@@ -12,7 +12,7 @@
   config.action_mailer.raise_delivery_errors = false
   config.action_mailer.perform_caching = false
 
-  config.active_support.deprecation = :log
+  config.active_support.report_deprecations = true
   config.active_record.migration_error = :page_load
   config.active_record.verbose_query_logs = true
 end

config/environments/test.rb

@@ -13,4 +13,9 @@
   config.active_support.disallowed_deprecation = :raise
   config.active_support.disallowed_deprecation_warnings = []
   config.active_record.maintain_test_schema = true
+
+  # Set deterministic secret key for test environment
+  # WARNING: This is only for test environment - never use in production!
+  # Using a realistic-length key to better simulate production behavior
+  config.secret_key_base = '173b11beda45abe0420fe63478ddc88a7b81de671837caffc6e9f31adce2d64bf2aba4ed886f10c8e22909fd55d08077859d3deb6c796776ff91e0b905566480'
 end

db/schema.rb

@@ -10,37 +10,37 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.2].define(version: 2012_01_09_000251) do
+ActiveRecord::Schema[8.1].define(version: 2012_01_09_000251) do
   create_table "activities", force: :cascade do |t|
-    t.string "comment"
-    t.integer "user_id"
     t.date "activity_date"
+    t.integer "activity_type"
+    t.string "comment"
+    t.datetime "created_at", precision: nil
     t.float "distance"
     t.integer "hours"
+    t.string "location", default: ""
     t.integer "minutes"
-    t.datetime "created_at", precision: nil
     t.datetime "updated_at", precision: nil
-    t.string "location", default: ""
-    t.integer "activity_type"
+    t.integer "user_id"
     t.index ["user_id", "created_at"], name: "index_activities_on_user_id_and_created_at"
   end
 
   create_table "configurations", force: :cascade do |t|
-    t.string "key"
-    t.text "value"
     t.datetime "created_at", precision: nil
+    t.string "key"
     t.datetime "updated_at", precision: nil
+    t.text "value"
   end
 
   create_table "users", force: :cascade do |t|
-    t.string "fname"
-    t.string "lname"
-    t.string "email"
+    t.boolean "admin", default: false
     t.datetime "created_at", precision: nil
-    t.datetime "updated_at", precision: nil
+    t.string "email"
     t.string "encrypted_password"
+    t.string "fname"
+    t.string "lname"
     t.string "salt"
-    t.boolean "admin", default: false
+    t.datetime "updated_at", precision: nil
     t.string "user_type", default: "2"
     t.index ["email"], name: "index_users_on_email", unique: true
     t.index ["user_type"], name: "index_users_on_user_type"