From 57aa65263a8925efedb9bd8cb8a0aeb530dcda4d Mon Sep 17 00:00:00 2001 From: Martin Lensment Date: Mon, 3 Aug 2015 12:55:23 +0300 Subject: [PATCH] Refactor model creation to fabricator #2806 --- spec/models/zonefile_setting_spec.rb | 34 +++------------------------- 1 file changed, 3 insertions(+), 31 deletions(-) diff --git a/spec/models/zonefile_setting_spec.rb b/spec/models/zonefile_setting_spec.rb index 5ac895a8e..d7a11bf85 100644 --- a/spec/models/zonefile_setting_spec.rb +++ b/spec/models/zonefile_setting_spec.rb @@ -2,27 +2,8 @@ require 'rails_helper' describe ZonefileSetting do it 'generates the zonefile' do - ZonefileSetting.where({ - origin: 'ee', - ttl: 43200, - refresh: 3600, - retry: 900, - expire: 1209600, - minimum_ttl: 3600, - email: 'hostmaster.eestiinternet.ee', - master_nameserver: 'ns.tld.ee' - }).first_or_create! - - ZonefileSetting.where({ - origin: 'pri.ee', - ttl: 43200, - refresh: 3600, - retry: 900, - expire: 1209600, - minimum_ttl: 3600, - email: 'hostmaster.eestiinternet.ee', - master_nameserver: 'ns.tld.ee' - }).first_or_create! + Fabricate(:zonefile_setting) + Fabricate(:zonefile_setting, origin: 'pri.ee') d = Fabricate(:domain_with_dnskeys, name: 'testpri.ee') d.nameservers << Nameserver.new({ @@ -56,16 +37,7 @@ describe ZonefileSetting do end it 'does not allow deleting zone when it has existing domains' do - zs = ZonefileSetting.where({ - origin: 'ee', - ttl: 43200, - refresh: 3600, - retry: 900, - expire: 1209600, - minimum_ttl: 3600, - email: 'hostmaster.eestiinternet.ee', - master_nameserver: 'ns.tld.ee' - }).first_or_create! + zs = Fabricate(:zonefile_setting) d = Fabricate(:domain)