From c22232ce07f980a33ddf59cfca95b1b4e9943e7c Mon Sep 17 00:00:00 2001 From: Oleg Hasjanov Date: Fri, 14 May 2021 15:00:12 +0300 Subject: [PATCH] added tests --- app/models/concerns/domain/releasable.rb | 1 - app/models/domain_status.rb | 2 +- test/models/domain/releasable/auctionable_test.rb | 13 ++++++++++++- test/models/domain/releasable/discardable_test.rb | 12 +++++++++++- 4 files changed, 24 insertions(+), 4 deletions(-) diff --git a/app/models/concerns/domain/releasable.rb b/app/models/concerns/domain/releasable.rb index b859b8102..606a146bc 100644 --- a/app/models/concerns/domain/releasable.rb +++ b/app/models/concerns/domain/releasable.rb @@ -4,7 +4,6 @@ module Domain::Releasable class_methods do def release_domains releasable_domains.each do |domain| - domain.release yield domain if block_given? end diff --git a/app/models/domain_status.rb b/app/models/domain_status.rb index 6fcd759ab..6210da2fa 100644 --- a/app/models/domain_status.rb +++ b/app/models/domain_status.rb @@ -160,7 +160,7 @@ class DomainStatus < ApplicationRecord ['TechChangeProhibited', SERVER_TECH_CHANGE_PROHIBITED], ['UpdateProhibited', SERVER_UPDATE_PROHIBITED], ['DeleteProhibited', SERVER_DELETE_PROHIBITED], - ['ServerReleaseProhibited', SERVER_RELEASE_PROHIBITED], + ['ReleaseProhibited', SERVER_RELEASE_PROHIBITED], ] end diff --git a/test/models/domain/releasable/auctionable_test.rb b/test/models/domain/releasable/auctionable_test.rb index 97d66be4f..675ad3c8e 100644 --- a/test/models/domain/releasable/auctionable_test.rb +++ b/test/models/domain/releasable/auctionable_test.rb @@ -99,10 +99,21 @@ class DomainReleasableAuctionableTest < ActiveJob::TestCase assert_not @domain.domain_name.at_auction? end - def test_ignores_domains_with_server_delete_prohibited_status + def test_does_not_ignore_domains_with_server_delete_prohibited_status @domain.update!(delete_date: '2010-07-04', statuses: [DomainStatus::SERVER_DELETE_PROHIBITED]) travel_to Time.zone.parse('2010-07-05') + assert_changes 'Domain.count' do + Domain.release_domains + end + + assert @domain.domain_name.at_auction? + end + + def test_ignores_domains_with_server_release_prohibited_status + @domain.update!(delete_date: '2010-07-04', statuses: [DomainStatus::SERVER_RELEASE_PROHIBITED]) + travel_to Time.zone.parse('2010-07-05') + assert_no_difference 'Domain.count' do Domain.release_domains end diff --git a/test/models/domain/releasable/discardable_test.rb b/test/models/domain/releasable/discardable_test.rb index a5ff9309e..e8bc5364a 100644 --- a/test/models/domain/releasable/discardable_test.rb +++ b/test/models/domain/releasable/discardable_test.rb @@ -51,13 +51,23 @@ class DomainReleasableDiscardableTest < ActiveSupport::TestCase end end - def test_ignores_domains_with_server_delete_prohibited_status + def test_does_not_ignore_domains_with_server_delete_prohibited_status @domain.update!(delete_date: '2010-07-04', statuses: [DomainStatus::SERVER_DELETE_PROHIBITED]) travel_to Time.zone.parse('2010-07-05') Domain.release_domains @domain.reload + assert @domain.discarded? + end + + def test_ignores_domains_with_server_release_prohibited_status + @domain.update!(delete_date: '2010-07-04', statuses: [DomainStatus::SERVER_RELEASE_PROHIBITED]) + travel_to Time.zone.parse('2010-07-05') + + Domain.release_domains + @domain.reload + assert_not @domain.discarded? end