Working domain renewal

app/models/domain.rb

@@ -125,13 +125,14 @@ class Domain < ActiveRecord::Base
   end
 
   def renew(cur_exp_date, period, unit='y')
-    if cur_exp_date == valid_to
+    if cur_exp_date.to_date == valid_to
-      valid_to = period.to_i.years
+      self.valid_to = self.valid_to + period.to_i.years
+      self.period = period
+      save
     else
       errors[:base] << {msg: I18n.t('errors.messages.epp_exp_dates_do_not_match'), obj: 'domain', val: cur_exp_date}
+      false
     end
-
-    save
   end
 
   class << self

spec/epp/domain_spec.rb

@@ -134,7 +134,13 @@ describe 'EPP Domain', epp: true do
     context 'with valid domain' do
       before(:each) { Fabricate(:domain, name: 'example.ee') }
 
-
+      it 'renews a domain' do
+        response = epp_request('domains/renew.xml')
+        exDate = response[:parsed].css('renData exDate').text
+        name = response[:parsed].css('renData name').text
+        expect(exDate).to eq ('2015-08-07 00:00:00 UTC')
+        expect(name).to eq ('example.ee')
+      end
     end
 
     it 'checks a domain' do

spec/epp/requests/domains/renew.xml

@@ -5,7 +5,7 @@
       <domain:renew
        xmlns:domain="urn:ietf:params:xml:ns:domain-1.0">
         <domain:name>example.ee</domain:name>
-        <domain:curExpDate>2000-04-03</domain:curExpDate>
+        <domain:curExpDate>2014-08-07</domain:curExpDate>
         <domain:period unit="y">1</domain:period>
       </domain:renew>
     </renew>

spec/fabricators/domain_fabricator.rb

@@ -1,5 +1,6 @@
 Fabricator(:domain) do
   name { "#{Faker::Internet.domain_word}.ee" }
+  valid_to Date.new(2014, 8, 7)
   period 1
   owner_contact(fabricator: :contact)
 end