Bypass validation on EPP domain:delete

#707

app/models/epp/domain.rb

@@ -556,8 +556,6 @@ class Epp::Domain < Domain
   end
 
   def epp_destroy(frame, user_id)
-    return false unless valid?
-
     check_discarded
 
     if doc = attach_legal_document(Epp::Domain.parse_legal_document_from_frame(frame))

test/fixtures/contacts.yml

@@ -34,5 +34,6 @@ acme_ltd:
 invalid:
   name: any
   code: any
+  email: invalid@invalid.test
   auth_info: any
   registrar: bestnames

test/fixtures/domain_contacts.yml

@@ -12,3 +12,13 @@ library_john:
   domain: library
   contact: john
   type: AdminDomainContact
+
+invalid_invalid_admin:
+  domain: invalid
+  contact: invalid
+  type: AdminDomainContact
+
+invalid_invalid_tech:
+  domain: invalid
+  contact: invalid
+  type: TechDomainContact

test/fixtures/domains.yml

@@ -27,3 +27,9 @@ library:
   valid_to: 2010-07-05
   period: 1
   period_unit: m
+
+invalid:
+  name: invalid.test
+  transfer_code: any
+  registrar: bestnames
+  registrant: invalid

test/integration/epp/domain/domain_delete_test.rb

@@ -0,0 +1,27 @@
+require 'test_helper'
+
+class EppDomainDeleteTest < ActionDispatch::IntegrationTest
+  def test_bypasses_domain_and_registrant_and_contacts_validation
+    request_xml = <<-XML
+      <?xml version="1.0" encoding="UTF-8" standalone="no"?>
+      <epp xmlns="https://epp.tld.ee/schema/epp-ee-1.0.xsd">
+        <command>
+          <delete>
+            <domain:delete xmlns:domain="https://epp.tld.ee/schema/domain-eis-1.0.xsd">
+              <domain:name>invalid.test</domain:name>
+            </domain:delete>
+          </delete>
+          <extension>
+            <eis:extdata xmlns:eis="https://epp.tld.ee/schema/eis-1.0.xsd">
+              <eis:legalDocument type="pdf">dGVzdCBmYWlsCg==</eis:legalDocument>
+            </eis:extdata>
+          </extension>
+        </command>
+      </epp>
+    XML
+
+    post '/epp/command/delete', { frame: request_xml }, { 'HTTP_COOKIE' => 'session=api_bestnames' }
+    assert_equal '1001', Nokogiri::XML(response.body).at_css('result')[:code]
+    assert_equal 1, Nokogiri::XML(response.body).css('result').size
+  end
+end

test/models/domain/domain_test.rb

@@ -8,4 +8,8 @@ class DomainTest < ActiveSupport::TestCase
   def test_valid_fixture
     assert @domain.valid?
   end
+
+  def test_invalid_fixture
+    assert domains(:invalid).invalid?
+  end
 end