Rails 7 -> 8.0.5

[abcampo-iry]

Status

• Closes: Not fully closing
• Related to 1291

Points for consideration:

• Upgrade of syntax to comply with 8.x version

What's changed?

It has been upgraded rails 7 to rails 8.0.4 following the conversations from the Issue above and #784

[github-actions]

Test coverage

91.05% line coverage reported by SimpleCov.
Run: https://github.com/RaspberryPiFoundation/editor-api/actions/runs/25787243937

[Copilot]

Pull request overview

Upgrades the application from Rails 7.2 to Rails 8.x, updating configuration defaults, strong-parameter handling, and tests to match new Rails behaviors (notably ActionController::Parameters#expect and updated exception messages).

Changes:

• Bump Rails dependency and framework defaults to Rails 8 (Gemfile/Gemfile.lock + config.load_defaults).
• Replace require(...).permit(...) strong params with params.expect(...) in several API controllers.
• Adjust request/spec helpers for rake task loading/reenabling and update brittle expectation(s) affected by Rails 8.

Reviewed changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
spec/requests/test_utilities_spec.rb	Updates rake task handling around request specs.
spec/requests/api_controller_spec.rb	Updates expected ParameterMissing error message for Rails 8.
spec/rails_helper.rb	Loads rake tasks once per suite and reenables tasks around type: :task specs.
spec/features/school_class/listing_school_classes_spec.rb	Makes assertions order-independent by locating classes by name.
Gemfile.lock	Resolves Rails and related gems to Rails 8.x versions.
Gemfile	Updates Rails dependency requirement to ~> 8.0.4.
config/application.rb	Updates config.load_defaults to 8.0.
app/controllers/api/subscriptions_controller.rb	Switches strong params to params.expect.
app/controllers/api/schools_controller.rb	Switches strong params to params.expect.
app/controllers/api/school_teachers_controller.rb	Switches strong params to params.expect.
app/controllers/api/school_students_controller.rb	Switches strong params to params.expect.
app/controllers/api/school_classes_controller.rb	Switches strong params to params.expect.
app/controllers/api/projects/remixes_controller.rb	Switches strong params to params.expect (includes nested params).
app/controllers/api/google_auth_controller.rb	Switches strong params to params.expect.
app/controllers/api/class_members_controller.rb	Switches strong params to params.expect.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[zetter-rpf]

👍 Just one question from your past self

Also, do you think we should we do any additional testing or monitoring given it's a major upgrade?

[abcampo-iry]

👍 Just one question from your past self

Also, do you think we should we do any additional testing or monitoring given it's a major upgrade?

@zetter-rpf Yes, after deploy, I’d watch logs and request metrics for increased 4xx/5xx rates, ActionController::ParameterMissing, unexpected auth failures, Google OAuth exchange failures, and any latency/error-rate changes on core endpoints.

[zetter-rpf]

@zetter-rpf Yes, after deploy, I’d watch logs and request metrics for increased 4xx/5xx rates, ActionController::ParameterMissing, unexpected auth failures, Google OAuth exchange failures, and any latency/error-rate changes on core endpoints.

Great. I think you can see most of that in Sentry, but let me know if I can help with that as there's also similar information in betterstack that we could look at together.