NZOI / nztrain / 14556340216

Pull #289

github

Remove caching from controller permitted_params etc.

We don't call these methods multiple times during a single request, so
there is no point in caching the result. (The example from the
original strong_parameters gem doesn't use caching:
https://github.com/rails/strong_parameters.)

This also eliminates silly inconsistencies with what was cached -- in
some places it was @_permitted_params, while in other places it was
the intermediate @_permitted_attributes.

Also remove method `visible_attributes` from user_controller.rb, it's
unused since commit 8798951a (Refactor to remove the UserPresenter
service object (#165), 2023-02-18).

29 of 48 new or added lines in 15 files covered. (60.42%)

2243 of 5990 relevant lines covered (37.45%)

5.65 hits per line