Skip to content

Commit

Permalink
Fix test names
Browse files Browse the repository at this point in the history
  • Loading branch information
Artur Beljajev committed May 21, 2018
1 parent bcba344 commit 65c50fd
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 10 deletions.
10 changes: 5 additions & 5 deletions test/integration/whois_records/html_test.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require 'test_helper'

class PrivatePersonWhoisRecordHTMLTest < ActionDispatch::IntegrationTest
def test_HTML_returns_404_for_missing_domains
def test_html_returns_404_for_missing_domains
visit('/v1/missing-domain.test')

assert_equal(404, page.status_code)
Expand All @@ -24,7 +24,7 @@ def test_returns_minimal_html
)
end

def test_HTML_has_disclaimer_text
def test_html_has_disclaimer_text
visit('/v1/privatedomain.test')

assert_text(
Expand All @@ -42,7 +42,7 @@ def test_HTML_has_disclaimer_text
)
end

def test_HTML_for_private_person_does_not_contain_personal_data
def test_html_for_private_person_does_not_contain_personal_data
visit('/v1/privatedomain.test')
assert_text(
<<-TEXT.squish
Expand All @@ -65,7 +65,7 @@ def test_HTML_for_private_person_does_not_contain_personal_data
)
end

def test_HTML_for_company_with_failed_captcha
def test_html_for_company_with_failed_captcha
# Setup
Recaptcha.configuration.skip_verify_env.delete("test")
headers = page.driver.options[:headers]
Expand Down Expand Up @@ -115,7 +115,7 @@ def test_HTML_for_company_with_failed_captcha
Recaptcha.configuration.skip_verify_env = ['test', 'cucumber']
end

def test_HTML_for_company_with_whitelist_IP
def test_html_for_company_with_whitelist_ip
visit('/v1/company-domain.test')

assert_text(
Expand Down
10 changes: 5 additions & 5 deletions test/integration/whois_records/json_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
class WhoisRecordJsonTest < ActionDispatch::IntegrationTest
# By default, all requests are done from 127.0.0.1, which is inside the
# whitelist
def test_JSON_does_not_include_disclosed_field_when_not_on_whitelist
def test_json_does_not_include_disclosed_field_when_not_on_whitelist
get('/v1/company-domain.test.json', {}, { 'REMOTE_ADDR': '1.2.3.4' })

response_json = JSON.parse(response.body)
Expand All @@ -20,7 +20,7 @@ def test_json_returns_404_for_missing_domains
assert_equal('missing-domain.test', response_json['name'])
end

def test_POST_requests_works_as_get
def test_post_requests_works_as_get
post('/v1/missing-domain.test.json')

assert_equal(404, response.status)
Expand All @@ -47,7 +47,7 @@ def test_discarded_returns_minimal_json
refute(response_json.has_key?('registrant'))
end

def test_JSON_does_not_include_private_person_contact_data
def test_json_does_not_include_private_person_contact_data
get('/v1/privatedomain.test.json')

response_json = JSON.parse(response.body)
Expand All @@ -56,7 +56,7 @@ def test_JSON_does_not_include_private_person_contact_data
assert_equal('Private Person', response_json['registrant'])
end

def test_JSON_includes_legal_person_contacts_data
def test_json_includes_legal_person_contacts_data
get('/v1/company-domain.test.json')

response_json = JSON.parse(response.body)
Expand All @@ -80,7 +80,7 @@ def test_JSON_includes_legal_person_contacts_data
assert_equal(expected_tech_contacts, response_json['tech_contacts'])
end

def test_JSON_all_fields_are_present
def test_json_all_fields_are_present
expected_response = {
'admin_contacts': [{'changed': 'Not Disclosed',
'email': 'Not Disclosed',
Expand Down

0 comments on commit 65c50fd

Please sign in to comment.