zydronium/internetee-registry
1
0
Fork 0
mirror of https://github.com/internetee/registry.git synced 2025-06-06 20:55:44 +02:00
Code Issues Projects Releases Packages Wiki Activity

Require new contact to be valid

Browse source 
#662
This commit is contained in:
Artur Beljajev 2018-06-04 03:06:29 +03:00
parent 125bb4bb4a
commit 263cb2e123
3 changed files with 19 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

7
app/api/repp/domain_contacts_v1.rb
View file

@ -23,6 +23,13 @@ module Repp
:bad_request)
end
if new_contact.invalid?
error!({ error: { type: 'invalid_request_error',
param: 'new_contact_id',
message: 'New contact must be valid' } },
:bad_request)
end
if current_contact == new_contact
error!({ error: { type: 'invalid_request_error',
message: 'New contact ID must be different from current' \

2
test/fixtures/contacts.yml vendored
View file

@ -76,7 +76,7 @@ not_in_use:
invalid:
name: any
code: any
code: invalid
email: invalid@invalid.test
auth_info: any
registrar: bestnames

11
test/integration/api/domain_contacts_test.rb
View file

@ -95,6 +95,17 @@ class APIDomainContactsTest < ActionDispatch::IntegrationTest
JSON.parse(response.body, symbolize_names: true)
end
def test_disallow_invalid_new_contact
patch '/repp/v1/domains/contacts', { current_contact_id: 'william-001',
new_contact_id: 'invalid' },
{ 'HTTP_AUTHORIZATION' => http_auth_key }
assert_response :bad_request
assert_equal ({ error: { type: 'invalid_request_error',
param: 'new_contact_id',
message: 'New contact must be valid' } }),
JSON.parse(response.body, symbolize_names: true)
end
def test_disallow_self_replacement
patch '/repp/v1/domains/contacts', { current_contact_id: 'william-001',
new_contact_id: 'william-001' },