Commit a1bd9d6ffebafc0181876d9badc629e9a0e0fd6e
1 parent
b60df0c3
Exists in
colab
and in
4 other branches
Restored comment about Devise's parameter test coverage
Showing
1 changed file
with
3 additions
and
0 deletions
Show diff stats
app/controllers/application_controller.rb
| @@ -28,6 +28,9 @@ class ApplicationController < ActionController::Base | @@ -28,6 +28,9 @@ class ApplicationController < ActionController::Base | ||
| 28 | 28 | ||
| 29 | protected | 29 | protected |
| 30 | 30 | ||
| 31 | + # We don't have a way to test this unless we have the Devise controllers among our code. | ||
| 32 | + # Since creating the controllers looks wronger than not testing this two | ||
| 33 | + # lines. I think we can live without 100% of coverage | ||
| 31 | # :nocov: | 34 | # :nocov: |
| 32 | def configure_permitted_parameters | 35 | def configure_permitted_parameters |
| 33 | devise_parameter_sanitizer.for(:sign_up) << :name | 36 | devise_parameter_sanitizer.for(:sign_up) << :name |