Skip to content

Commit 1e53322

Browse files
fix MHV user account error body (#20170)
1 parent 0006ea2 commit 1e53322

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

app/services/mhv/user_account/errors.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ def initialize(message, body = nil)
2828
end
2929

3030
def as_json
31-
[{ title: message, detail: body[:message], code: body[:errorCode] }]
31+
[{ title: message, detail: body['message'], code: body['errorCode'] }]
3232
end
3333
end
3434
end

spec/controllers/v0/user/mhv_user_accounts_controller_spec.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@
107107
end
108108

109109
context 'when there is an MHV client error' do
110-
let(:mhv_error_body) { { errorCode: mhv_error_code, message: mhv_error_message } }
110+
let(:mhv_error_body) { { 'errorCode' => mhv_error_code, 'message' => mhv_error_message } }
111111
let(:mhv_error_code) { 'some-code' }
112112
let(:mhv_error_message) { 'some-error-message' }
113113

spec/services/mhv/user_account/creator_spec.rb

+9-3
Original file line numberDiff line numberDiff line change
@@ -138,13 +138,19 @@
138138
icn:
139139
}
140140
end
141+
let(:mhv_error_body) { { 'message' => 'some-message', 'errorCode' => 'some-code' } }
141142

142143
before do
143-
allow(mhv_client).to receive(:create_account).and_raise(Common::Client::Errors::ClientError, 'error')
144+
allow(mhv_client).to receive(:create_account).and_raise(
145+
Common::Client::Errors::ClientError.new('error', 400, mhv_error_body)
146+
)
144147
end
145148

146-
it 'logs and raises an error' do
147-
expect { subject.perform }.to raise_error(MHV::UserAccount::Errors::MHVClientError)
149+
it 'logs and raises an error with expected body' do
150+
expect { subject.perform }.to raise_error(MHV::UserAccount::Errors::MHVClientError) do |error|
151+
expect(error.body).to eq(mhv_error_body)
152+
end
153+
148154
expect(Rails.logger).to have_received(:error).with(expected_log_message, expected_log_payload)
149155
end
150156
end

0 commit comments

Comments
 (0)