Merge branch 'master' of github.com:internetee/registry

app/helpers/epp/contacts_helper.rb

@@ -33,10 +33,10 @@ module Epp::ContactsHelper
       @contact.destroy
       render '/epp/contacts/delete'
     rescue NoMethodError => e
-      epp_errors << {code: '2303', msg: "Object does not exist"}
+      epp_errors << { code: '2303', msg: t('errors.messages.epp_obj_does_not_exist') }
       render '/epp/error'
     rescue
-      epp_errors << {code: '2400', msg: "Command failed"}
+      epp_errors << {code: '2400', msg: t('errors.messages.epp_command_failed')}
       render '/epp/error'
     end
   end
@@ -48,7 +48,7 @@ module Epp::ContactsHelper
     if @contacts.any?
       render '/epp/contacts/check'
     else
-      epp_errors << {code: '2303', msg: "Object does not exist"}
+      epp_errors << { code: '2303', msg: t('errors.messages.epp_obj_does_not_exist') }
       render 'epp/error'
     end
   end
@@ -62,7 +62,7 @@ module Epp::ContactsHelper
     if @contact
       render '/epp/contacts/info'
     else
-      epp_errors << { code: '2303', msg: 'Object does not exist' }
+      epp_errors << { code: '2303', msg: t('errors.messages.epp_obj_does_not_exist') }
       render 'epp/error'
     end
   end

config/locales/en.yml

@@ -49,3 +49,5 @@ en:
     messages:
       epp_domain_reserved: 'Domain name is reserved or restricted'
       epp_domain_taken: 'Domain name already exists'
+      epp_obj_does_not_exist: 'Object does not exist'
+      epp_command_failed: 'Command failed'

spec/epp/contact_spec.rb

@@ -54,6 +54,13 @@ describe 'EPP Contact', epp: true do
       response = epp_request('contacts/check.xml')
       expect(response[:result_code]).to eq('1000')
       expect(response[:msg]).to eq('Command completed successfully')
+      ids = response[:parsed].css('resData chkData id')
+
+      expect(ids[0].attributes['avail'].text).to eq('0')
+      expect(ids[1].attributes['avail'].text).to eq('1')
+
+      expect(ids[0].text).to eq('sh8913')
+      expect(ids[1].text).to eq('sh8914')
 
     end
 
@@ -64,15 +71,15 @@ describe 'EPP Contact', epp: true do
     end
 
     it 'returns info about contact' do
-      contact = Fabricate(:contact, :name => "Johnny Awesome")
+      Fabricate(:contact, :name => "Johnny Awesome")
       Fabricate(:address)
 
       response = epp_request('contacts/info.xml')
-      contact_res = response[:parsed].css('resData chkData')
+      contact = response[:parsed].css('resData chkData')
 
       expect(response[:result_code]).to eq('1000')
       expect(response[:msg]).to eq('Command completed successfully')
-      expect(contact_res.css('name').first.text).to eq('Johnny Awesome')
+      expect(contact.css('name').first.text).to eq('Johnny Awesome')
 
     end
   end