Browse Source

make it not return http 400 when passing and empty source argument (#12259)

* make it not return http 400 when passing and empty source argument

* create a spec for the empty source hash bug

* compact checks for nil, empty? parameters

* use nil.blank? instead checking for nil
master
Jennifer Glauche 4 years ago
committed by Eugen Rochko
parent
commit
fd93a9c871
2 changed files with 14 additions and 1 deletions
  1. +1
    -1
      app/controllers/api/v1/accounts/credentials_controller.rb
  2. +13
    -0
      spec/controllers/api/v1/accounts/credentials_controller_spec.rb

+ 1
- 1
app/controllers/api/v1/accounts/credentials_controller.rb View File

@ -25,7 +25,7 @@ class Api::V1::Accounts::CredentialsController < Api::BaseController
end
def user_settings_params
return nil unless params.key?(:source)
return nil if params[:source].blank?
source_params = params.require(:source)

+ 13
- 0
spec/controllers/api/v1/accounts/credentials_controller_spec.rb View File

@ -59,6 +59,19 @@ describe Api::V1::Accounts::CredentialsController do
end
end
describe 'with empty source list' do
before do
patch :update, params: {
display_name: "I'm a cat",
source: {},
}, as: :json
end
it 'returns http success' do
expect(response).to have_http_status(200)
end
end
describe 'with invalid data' do
before do
patch :update, params: { note: 'This is too long. ' * 30 }

Loading…
Cancel
Save