diff --git a/spec/models/account_spec.rb b/spec/models/account_spec.rb index 83e40b86e1..46cef0b094 100644 --- a/spec/models/account_spec.rb +++ b/spec/models/account_spec.rb @@ -1,14 +1,6 @@ require 'rails_helper' RSpec.describe Account do - it 'has versions' do - with_versioning do - price = build(:account) - price.save! - expect(price.versions.size).to be(1) - end - end - describe 'registrar validation', db: false do subject(:account) { described_class.new } diff --git a/spec/models/admin_user_spec.rb b/spec/models/admin_user_spec.rb index f25ea77bd8..1cb8b5b127 100644 --- a/spec/models/admin_user_spec.rb +++ b/spec/models/admin_user_spec.rb @@ -1,16 +1,6 @@ require 'rails_helper' RSpec.describe AdminUser do - context 'with invalid attribute' do - before do - @admin_user = described_class.new - end - - it 'should not have any versions' do - @admin_user.versions.should == [] - end - end - context 'with valid attributes' do before do @admin_user = create(:admin_user) diff --git a/spec/models/api_user_spec.rb b/spec/models/api_user_spec.rb index 89feeac6d3..e27cc87712 100644 --- a/spec/models/api_user_spec.rb +++ b/spec/models/api_user_spec.rb @@ -17,10 +17,6 @@ ]) end - it 'should not have any versions' do - @api_user.versions.should == [] - end - it 'should be active by default' do @api_user.active.should == true end @@ -41,16 +37,6 @@ @api_user.valid? @api_user.errors.full_messages.should match_array([]) end - - it 'should have one version' do - with_versioning do - @api_user.versions.should == [] - @api_user.username = 'newusername' - @api_user.save - @api_user.errors.full_messages.should match_array([]) - @api_user.versions.size.should == 1 - end - end end describe '::min_password_length', db: false do diff --git a/spec/models/bank_statement_spec.rb b/spec/models/bank_statement_spec.rb index d1d54d01fb..2f2062dab6 100644 --- a/spec/models/bank_statement_spec.rb +++ b/spec/models/bank_statement_spec.rb @@ -13,10 +13,6 @@ "Iban is missing" ]) end - - it 'should not have any versions' do - @bank_statement.versions.should == [] - end end context 'with valid attributes' do @@ -67,15 +63,5 @@ bs.bank_transactions.unbinded.count.should == 4 bs.not_binded?.should == true end - - it 'should have one version' do - with_versioning do - @bank_statement.versions.should == [] - @bank_statement.bank_code = 'new_code' - @bank_statement.save - @bank_statement.errors.full_messages.should match_array([]) - @bank_statement.versions.size.should == 1 - end - end end end diff --git a/spec/models/contact_spec.rb b/spec/models/contact_spec.rb index 5913083c93..52469b28f8 100644 --- a/spec/models/contact_spec.rb +++ b/spec/models/contact_spec.rb @@ -5,16 +5,6 @@ create(:zone, origin: 'ee') end - context 'about class' do - it 'should have versioning enabled?' do - Contact.paper_trail_enabled_for_model?.should == true - end - - it 'should have custom log prexied table name for versions table' do - ContactVersion.table_name.should == 'log_contacts' - end - end - context 'with invalid attribute' do before :example do @contact = Contact.new @@ -28,10 +18,6 @@ @contact.updator.should == nil end - it 'should not have any versions' do - @contact.versions.should == [] - end - it 'should not accept long code' do @contact.code = 'verylongcode' * 100 @contact.valid? @@ -66,16 +52,6 @@ @contact = create(:contact) end - it 'should have one version' do - with_versioning do - @contact.versions.reload.should == [] - @contact.name = 'New name' - @contact.save - @contact.errors.full_messages.should match_array([]) - @contact.versions.size.should == 1 - end - end - it 'should not overwrite code' do old_code = @contact.code @contact.code = 'CID:REG1:should-not-overwrite-old-code-12345' diff --git a/spec/models/dnskey_spec.rb b/spec/models/dnskey_spec.rb index 5c0f7859fa..d219682149 100644 --- a/spec/models/dnskey_spec.rb +++ b/spec/models/dnskey_spec.rb @@ -34,10 +34,6 @@ @dnskey.errors.full_messages.should match_array([ ]) end - - it 'should not have any versions' do - @dnskey.versions.should == [] - end end context 'with valid attributes' do @@ -66,15 +62,6 @@ @dnskey.errors.full_messages.should match_array([]) end - # TODO: figure out why not working - # it 'should have one version' do - # with_versioning do - # @dnskey.versions.should == [] - # @dnskey.touch_with_version - # @dnskey.versions.size.should == 1 - # end - # end - it 'generates correct DS digest and DS key tag for ria.ee' do d = create(:domain, name: 'ria.ee', dnskeys: [@dnskey]) dk = d.dnskeys.last diff --git a/spec/models/domain_contact_spec.rb b/spec/models/domain_contact_spec.rb index fc7ec010ed..f5476c49bd 100644 --- a/spec/models/domain_contact_spec.rb +++ b/spec/models/domain_contact_spec.rb @@ -49,18 +49,6 @@ it 'should have Tech name' do @domain_contact.name.should == 'Tech' end - - it 'should have one version' do - @domain_contact = create(:domain_contact) - - with_versioning do - @domain_contact.versions.reload.should == [] - @domain_contact.contact = create(:contact) - @domain_contact.save! - @domain_contact.errors.full_messages.should match_array([]) - @domain_contact.versions.size.should == 1 - end - end end context 'with valid attributes with tech domain contact' do @@ -82,18 +70,6 @@ it 'should have Tech name' do @domain_contact.name.should == 'Tech' end - - it 'should have one version' do - @domain_contact = create(:domain_contact) - - with_versioning do - @domain_contact.versions.reload.should == [] - @domain_contact.contact = create(:contact) - @domain_contact.save! - @domain_contact.errors.full_messages.should match_array([]) - @domain_contact.versions.size.should == 1 - end - end end context 'with valid attributes with admin domain contact' do @@ -115,15 +91,5 @@ it 'should have Tech name' do @domain_contact.name.should == 'Admin' end - - it 'should have one version' do - with_versioning do - @domain_contact.versions.reload.should == [] - @domain_contact.contact = create(:contact) - @domain_contact.save - @domain_contact.errors.full_messages.should match_array([]) - @domain_contact.versions.size.should == 1 - end - end end end diff --git a/spec/models/domain_spec.rb b/spec/models/domain_spec.rb index bed293474c..8bf58bdd0b 100644 --- a/spec/models/domain_spec.rb +++ b/spec/models/domain_spec.rb @@ -33,10 +33,6 @@ @domain = Domain.new end - it 'should not have any versions' do - @domain.versions.should == [] - end - it 'should not have whois body' do @domain.whois_record.should == nil end @@ -489,21 +485,6 @@ expect(d.statuses.count).to eq(1) expect(d.statuses.first).to eq(DomainStatus::CLIENT_DELETE_PROHIBITED) end - - with_versioning do - context 'when saved' do - before(:each) do - domain = create(:domain, nameservers_attributes: [attributes_for(:nameserver), - attributes_for(:nameserver)]) - end - - it 'creates domain version' do - expect(DomainVersion.count).to eq(1) - expect(ContactVersion.count).to eq(3) - expect(NameserverVersion.count).to eq(2) - end - end - end end RSpec.describe Domain do diff --git a/spec/models/keyrelay_spec.rb b/spec/models/keyrelay_spec.rb index 5c335941bd..fc388dfede 100644 --- a/spec/models/keyrelay_spec.rb +++ b/spec/models/keyrelay_spec.rb @@ -41,10 +41,6 @@ "Only one parameter allowed: relative or absolute" ]) end - - it 'should not have any versions' do - @keyrelay.versions.should == [] - end end context 'with valid attributes' do @@ -63,16 +59,6 @@ @keyrelay.errors.full_messages.should match_array([]) end - it 'should have one version' do - with_versioning do - @keyrelay.versions.should == [] - @keyrelay.auth_info_pw = 'newpw' - @keyrelay.save - @keyrelay.errors.full_messages.should match_array([]) - @keyrelay.versions.size.should == 1 - end - end - it 'is in pending status' do @keyrelay.status.should == 'pending' end diff --git a/spec/models/white_ip_spec.rb b/spec/models/white_ip_spec.rb index bb286c0caf..fa28991f11 100644 --- a/spec/models/white_ip_spec.rb +++ b/spec/models/white_ip_spec.rb @@ -23,22 +23,6 @@ end end - context 'with valid attributes' do - before :all do - @white_ip = create(:white_ip) - end - - it 'should have one version' do - with_versioning do - @white_ip.versions.should == [] - @white_ip.ipv4 = '192.168.1.2' - @white_ip.save - @white_ip.errors.full_messages.should match_array([]) - @white_ip.versions.size.should == 1 - end - end - end - describe '#include_ip?' do context 'when given ip v4 exists' do before do