keepr 0.3.0 → 0.6.0

data/lib/keepr/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Keepr
-  VERSION = '0.3.0'
+  VERSION = '0.6.0'
 end

data/spec/factories/account.rb

@@ -1,7 +1,9 @@
-FactoryGirl.define do
+# frozen_string_literal: true
+
+FactoryBot.define do
   factory :account, class: Keepr::Account do
-    number 12345
-    kind :asset
-    name 'Foo'
+    number { 12_345 }
+    kind { :asset }
+    name { 'Foo' }
   end
 end

data/spec/factories/cost_center.rb

@@ -1,6 +1,8 @@
-FactoryGirl.define do
+# frozen_string_literal: true
+
+FactoryBot.define do
   factory :cost_center, class: Keepr::CostCenter do
-    number 'FZ1'
-    name 'Kleintransporter'
+    number { 'FZ1' }
+    name { 'Kleintransporter' }
   end
 end

data/spec/factories/group.rb

@@ -1,6 +1,8 @@
-FactoryGirl.define do
+# frozen_string_literal: true
+
+FactoryBot.define do
   factory :group, class: Keepr::Group do
-    target :asset
-    name 'Foo'
+    target { :asset }
+    name { 'Foo' }
   end
 end

data/spec/factories/tax.rb

@@ -1,8 +1,10 @@
-FactoryGirl.define do
+# frozen_string_literal: true
+
+FactoryBot.define do
   factory :tax, class: Keepr::Tax do
-    name 'USt19'
-    description 'Umsatzsteuer 19%'
-    value 19.0
-    keepr_account { FactoryGirl.create :account, :number => 1776 }
+    name { 'USt19' }
+    description { 'Umsatzsteuer 19%' }
+    value { 19.0 }
+    keepr_account { FactoryBot.create :account, number: 1776 }
   end
 end

data/spec/keepr/account_export_spec.rb

@@ -1,26 +1,29 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Keepr::AccountExport do
-  let!(:account_1000)  { FactoryGirl.create :account, :kind => :asset,     :number => 1000,  :name => 'Kasse'                     }
-  let!(:account_1776)  { FactoryGirl.create :account, :kind => :liability, :number => 1776,  :name => 'Umsatzsteuer 19 %'         }
-  let!(:account_4920)  { FactoryGirl.create :account, :kind => :expense,   :number => 4920,  :name => 'Telefon'                   }
-  let!(:account_8400)  { FactoryGirl.create :account, :kind => :revenue,   :number => 8400,  :name => 'Erlöse 19 %'               }
-  let!(:account_9000)  { FactoryGirl.create :account, :kind => :neutral,   :number => 9000,  :name => 'Saldenvorträge Sachkonten' }
-  let!(:account_10000) { FactoryGirl.create :account, :kind => :creditor,  :number => 10000, :name => 'Diverse Kreditoren'        }
-  let!(:account_70000) { FactoryGirl.create :account, :kind => :debtor,    :number => 70000, :name => 'Diverse Debitoren'         }
+  let!(:account_1000)  { FactoryBot.create :account, kind: :asset,     number:  1000,  name: 'Kasse'                     }
+  let!(:account_1776)  { FactoryBot.create :account, kind: :liability, number:  1776,  name: 'Umsatzsteuer 19 %'         }
+  let!(:account_4920)  { FactoryBot.create :account, kind: :expense,   number:  4920,  name: 'Telefon'                   }
+  let!(:account_8400)  { FactoryBot.create :account, kind: :revenue,   number:  8400,  name: 'Erlöse 19 %'               }
+  let!(:account_9000)  { FactoryBot.create :account, kind: :forward,   number:  9000,  name: 'Saldenvorträge Sachkonten' }
+  let!(:account_10000) { FactoryBot.create :account, kind: :creditor,  number: 10_000, name: 'Diverse Kreditoren'        }
+  let!(:account_70000) { FactoryBot.create :account, kind: :debtor,    number: 70_000, name: 'Diverse Debitoren'         }
 
   let(:scope) { Keepr::Account.all }
 
-  let(:export) {
-    Keepr::AccountExport.new(scope,
-      'Berater'     => 1234567,
-      'Mandant'     => 78901,
-      'WJ-Beginn'   => Date.new(2016,1,1),
+  let(:export) do
+    Keepr::AccountExport.new(
+      scope,
+      'Berater'     => 1_234_567,
+      'Mandant'     => 78_901,
+      'WJ-Beginn'   => Date.new(2016, 1, 1),
       'Bezeichnung' => 'Keepr-Konten'
-    ) do |account|
+    ) do
       { 'Sprach-ID' => 'de-DE' }
     end
-  }
+  end
 
   describe :to_s do
     subject { export.to_s }
@@ -29,43 +32,43 @@ describe Keepr::AccountExport do
       subject.lines[2..-1].map { |line| line.encode(Encoding::UTF_8) }
     end
 
-    it "should return CSV lines" do
+    it 'should return CSV lines' do
       subject.lines.each { |line| expect(line).to include(';') }
     end
 
-    it "should include header data" do
+    it 'should include header data' do
       expect(subject.lines[0]).to include('1234567;')
       expect(subject.lines[0]).to include('78901;')
-      expect(subject.lines[0]).to include('Keepr-Konten;')
+      expect(subject.lines[0]).to include('"Keepr-Konten";')
     end
 
-    it "should include all accounts except debtor/creditor" do
+    it 'should include all accounts except debtor/creditor' do
       expect(account_lines.count).to eq(5)
 
       expect(account_lines[0]).to include('1000;')
-      expect(account_lines[0]).to include('Kasse;')
+      expect(account_lines[0]).to include('"Kasse";')
 
       expect(account_lines[1]).to include('1776;')
-      expect(account_lines[1]).to include('Umsatzsteuer 19 %;')
+      expect(account_lines[1]).to include('"Umsatzsteuer 19 %";')
 
       expect(account_lines[2]).to include('4920;')
-      expect(account_lines[2]).to include('Telefon;')
+      expect(account_lines[2]).to include('"Telefon";')
 
       expect(account_lines[3]).to include('8400;')
-      expect(account_lines[3]).to include('Erlöse 19 %;')
+      expect(account_lines[3]).to include('"Erlöse 19 %";')
 
       expect(account_lines[4]).to include('9000;')
-      expect(account_lines[4]).to include('Saldenvorträge Sachkonten;')
+      expect(account_lines[4]).to include('"Saldenvorträge Sachkonten";')
     end
 
-    it "should include data from block" do
-      expect(account_lines[0]).to include(';de-DE')
-      expect(account_lines[1]).to include(';de-DE')
+    it 'should include data from block' do
+      expect(account_lines[0]).to include(';"de-DE"')
+      expect(account_lines[1]).to include(';"de-DE"')
     end
   end
 
   describe :to_file do
-    it "should create CSV file" do
+    it 'should create CSV file' do
       Dir.mktmpdir do |dir|
         filename = "#{dir}/EXTF_Kontenbeschriftungen.csv"
         export.to_file(filename)

data/spec/keepr/account_spec.rb

@@ -1,89 +1,90 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Keepr::Account do
   describe :number_as_string do
-    it "should return number with leading zeros for low values" do
-      account = Keepr::Account.new(:number => 999)
+    it 'should return number with leading zeros for low values' do
+      account = Keepr::Account.new(number: 999)
       expect(account.number_as_string).to eq('0999')
     end
 
-    it "should return number unchanged for high values" do
-      account = Keepr::Account.new(:number => 70000)
+    it 'should return number unchanged for high values' do
+      account = Keepr::Account.new(number: 70_000)
       expect(account.number_as_string).to eq('70000')
     end
   end
 
   describe :to_s do
-    it "should format" do
-      account = Keepr::Account.new(:number => 27, :name => 'Software')
+    it 'should format' do
+      account = Keepr::Account.new(number: 27, name: 'Software')
       expect(account.to_s).to eq('0027 (Software)')
     end
   end
 end
 
 describe Keepr::Account do
-  let!(:account_1000) { FactoryGirl.create(:account, :number => 1000) }
-  let!(:account_1200) { FactoryGirl.create(:account, :number => 1200) }
+  let!(:account_1000) { FactoryBot.create(:account, number: 1000) }
+  let!(:account_1200) { FactoryBot.create(:account, number: 1200) }
 
   before :each do
-    Keepr::Journal.create! :date => Date.yesterday,
-                           :permanent => true,
-                           :keepr_postings_attributes => [
-                             { :keepr_account => account_1000, :amount => 20, :side => 'debit' },
-                             { :keepr_account => account_1200, :amount => 20, :side => 'credit' }
-                            ]
-
-    Keepr::Journal.create! :date => Date.yesterday,
-                           :keepr_postings_attributes => [
-                             { :keepr_account => account_1000, :amount =>  10, :side => 'credit' },
-                             { :keepr_account => account_1200, :amount =>  10, :side => 'debit' },
-                            ]
-
-    Keepr::Journal.create! :date => Date.today,
-                           :keepr_postings_attributes => [
-                             { :keepr_account => account_1000, :amount => 200, :side => 'debit' },
-                             { :keepr_account => account_1200, :amount => 200, :side => 'credit' }
-                            ]
-
-    Keepr::Journal.create! :date => Date.today,
-                           :keepr_postings_attributes => [
-                             { :keepr_account => account_1000, :amount => 100, :side => 'credit' },
-                             { :keepr_account => account_1200, :amount => 100, :side => 'debit' },
-                            ]
+    Keepr::Journal.create! date: Date.yesterday,
+                           permanent: true,
+                           keepr_postings_attributes: [
+                             { keepr_account: account_1000, amount: 20, side: 'debit' },
+                             { keepr_account: account_1200, amount: 20, side: 'credit' }
+                           ]
+
+    Keepr::Journal.create! date: Date.yesterday,
+                           keepr_postings_attributes: [
+                             { keepr_account: account_1000, amount:  10, side: 'credit' },
+                             { keepr_account: account_1200, amount:  10, side: 'debit' }
+                           ]
+
+    Keepr::Journal.create! date: Date.current,
+                           keepr_postings_attributes: [
+                             { keepr_account: account_1000, amount: 200, side: 'debit' },
+                             { keepr_account: account_1200, amount: 200, side: 'credit' }
+                           ]
+
+    Keepr::Journal.create! date: Date.current,
+                           keepr_postings_attributes: [
+                             { keepr_account: account_1000, amount: 100, side: 'credit' },
+                             { keepr_account: account_1200, amount: 100, side: 'debit' }
+                           ]
   end
 
   describe 'validations' do
-    let!(:result_group)    { FactoryGirl.create(:group, :target => :liability, :is_result => true) }
-    let!(:liability_group) { FactoryGirl.create(:group, :target => :liability) }
-    let!(:asset_group)     { FactoryGirl.create(:group, :target => :asset) }
+    let!(:result_group)    { FactoryBot.create(:group, target: :liability, is_result: true) }
+    let!(:liability_group) { FactoryBot.create(:group, target: :liability) }
+    let!(:asset_group)     { FactoryBot.create(:group, target: :asset) }
 
-    it "should not allow assigning to result group" do
-      account = FactoryGirl.build(:account, :keepr_group => result_group)
+    it 'should not allow assigning to result group' do
+      account = FactoryBot.build(:account, keepr_group: result_group)
       expect(account).to_not be_valid
-      expect(account.errors.added? :keepr_group_id, :no_group_allowed_for_result).to eq(true)
+      expect(account.errors.added?(:keepr_group_id, :no_group_allowed_for_result)).to eq(true)
     end
 
-    it "should not allow assigning asset account to liability group" do
-      account = FactoryGirl.build(:account, :kind => :asset, :keepr_group => liability_group)
+    it 'should not allow assigning asset account to liability group' do
+      account = FactoryBot.build(:account, kind: :asset, keepr_group: liability_group)
       expect(account).to_not be_valid
-      expect(account.errors.added? :kind, :group_mismatch).to eq(true)
+      expect(account.errors.added?(:kind, :group_mismatch)).to eq(true)
     end
 
-    it "should not allow assigning liability account to asset group" do
-      account = FactoryGirl.build(:account, :kind => :liability, :keepr_group => asset_group)
+    it 'should not allow assigning liability account to asset group' do
+      account = FactoryBot.build(:account, kind: :liability, keepr_group: asset_group)
       expect(account).to_not be_valid
-      expect(account.errors.added? :kind, :group_mismatch).to eq(true)
+      expect(account.errors.added?(:kind, :group_mismatch)).to eq(true)
     end
 
-    it "should not allow assigning neutral account to asset group" do
-      account = FactoryGirl.build(:account, :kind => :neutral, :keepr_group => asset_group)
+    it 'should not allow assigning forward account to asset group' do
+      account = FactoryBot.build(:account, kind: :forward, keepr_group: asset_group)
       expect(account).to_not be_valid
-      expect(account.errors.added? :kind, :group_conflict).to eq(true)
+      expect(account.errors.added?(:kind, :group_conflict)).to eq(true)
     end
 
-    it "should allow target match" do
-      account = FactoryGirl.build(:account, :kind => :asset, :keepr_group => asset_group)
+    it 'should allow target match' do
+      account = FactoryBot.build(:account, kind: :asset, keepr_group: asset_group)
       expect(account).to be_valid
     end
   end
@@ -131,7 +132,7 @@ describe Keepr::Account do
 
     context 'with date option' do
       it 'should work with Date' do
-        account1, account2 = Keepr::Account.with_sums(:date => Date.yesterday)
+        account1, account2 = Keepr::Account.with_sums(date: Date.yesterday)
 
         expect(account1.number).to eq(1000)
         expect(account1.sum_amount).to eq(10)
@@ -140,7 +141,7 @@ describe Keepr::Account do
       end
 
       it 'should work with Range' do
-        account1, account2 = Keepr::Account.with_sums(:date => Date.today..Date.tomorrow)
+        account1, account2 = Keepr::Account.with_sums(date: Date.today..Date.tomorrow)
 
         expect(account1.number).to eq(1000)
         expect(account1.sum_amount).to eq(100)
@@ -149,14 +150,14 @@ describe Keepr::Account do
       end
 
       it 'should raise for other class' do
-        expect { Keepr::Account.with_sums(:date => Time.current) }.to raise_error(ArgumentError)
-        expect { Keepr::Account.with_sums(:date => :foo)         }.to raise_error(ArgumentError)
+        expect { Keepr::Account.with_sums(date: Time.current) }.to raise_error(ArgumentError)
+        expect { Keepr::Account.with_sums(date: :foo)         }.to raise_error(ArgumentError)
       end
     end
 
     context 'with permanent_only option' do
       it 'should filter the permanent journals' do
-        account1, account2 = Keepr::Account.with_sums(:permanent_only => true)
+        account1, account2 = Keepr::Account.with_sums(permanent_only: true)
 
         expect(account1.number).to eq(1000)
         expect(account1.sum_amount).to eq(20)
@@ -175,17 +176,17 @@ describe Keepr::Account do
 end
 
 describe Keepr::Account, 'with subaccounts' do
-  let!(:account_1400) { FactoryGirl.create(:account, :number => 1400) }
-  let!(:account_10000) { FactoryGirl.create(:account, :number => 10000, :parent => account_1400) }
-  let!(:account_10001) { FactoryGirl.create(:account, :number => 10001, :parent => account_1400) }
-  let!(:account_8400) { FactoryGirl.create(:account, :number => 8400) }
+  let!(:account_1400) { FactoryBot.create(:account, number: 1400) }
+  let!(:account_10000) { FactoryBot.create(:account, number: 10_000, parent: account_1400) }
+  let!(:account_10001) { FactoryBot.create(:account, number: 10_001, parent: account_1400) }
+  let!(:account_8400) { FactoryBot.create(:account, number: 8400) }
 
   before :each do
-    Keepr::Journal.create! :date => Date.yesterday,
-                           :keepr_postings_attributes => [
-                             { :keepr_account => account_10000, :amount => 20, :side => 'debit' },
-                             { :keepr_account => account_8400, :amount => 20, :side => 'credit' }
-                            ]
+    Keepr::Journal.create! date: Date.yesterday,
+                           keepr_postings_attributes: [
+                             { keepr_account: account_10000, amount: 20, side: 'debit' },
+                             { keepr_account: account_8400, amount: 20, side: 'credit' }
+                           ]
   end
 
   describe :keepr_postings do
@@ -209,42 +210,46 @@ describe Keepr::Account, 'with subaccounts' do
 
   describe :with_sums do
     it 'should calc balance' do
-      expect(Keepr::Account.with_sums.
-                     select(&:sum_amount).
-                     map { |a| [a.number, a.sum_amount] }).
-                     to eq([[8400, -20], [10000, 20]])
+      expect(Keepr::Account.with_sums
+                     .select(&:sum_amount)
+                     .map { |a| [a.number, a.sum_amount] })
+        .to eq([[8400, -20], [10_000, 20]])
     end
   end
 
   describe :merged_with_sums do
     it 'should calc merged balance' do
-      expect(Keepr::Account.merged_with_sums.
-                     select(&:sum_amount).
-                     map { |a| [a.number, a.sum_amount] }).
-                     to eq([[1400, 20], [8400, -20]])
+      expect(Keepr::Account.merged_with_sums
+                     .select(&:sum_amount)
+                     .map { |a| [a.number, a.sum_amount] })
+        .to eq([[1400, 20], [8400, -20]])
     end
   end
 end
 
 describe Keepr::Account, 'with tax' do
-  let!(:tax_account) { Keepr::Account.create! :number => 1776,
-                                              :name => 'Umsatzsteuer 19%',
-                                              :kind => :asset }
-
-  let!(:tax) { Keepr::Tax.create! :name          => 'USt19',
-                                  :description   => 'Umsatzsteuer 19%',
-                                  :value         => 19.0,
-                                  :keepr_account => tax_account }
-
-  it "should link to tax" do
-    account = Keepr::Account.new :number    => 8400,
-                                 :name      => 'Erlöse 19% USt',
-                                 :kind      => :revenue,
-                                 :keepr_tax => tax
+  let!(:tax_account) do
+    Keepr::Account.create! number: 1776,
+                           name: 'Umsatzsteuer 19%',
+                           kind: :asset
+  end
+
+  let!(:tax) do
+    Keepr::Tax.create! name: 'USt19',
+                       description: 'Umsatzsteuer 19%',
+                       value: 19.0,
+                       keepr_account: tax_account
+  end
+
+  it 'should link to tax' do
+    account = Keepr::Account.new number: 8400,
+                                 name: 'Erlöse 19% USt',
+                                 kind: :revenue,
+                                 keepr_tax: tax
     expect(account).to be_valid
   end
 
-  it "should avoid circular reference" do
+  it 'should avoid circular reference' do
     tax_account.keepr_tax_id = tax.id
     expect(tax_account).to be_invalid
   end

data/spec/keepr/active_record_extension_spec.rb

@@ -1,26 +1,28 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe Keepr::ActiveRecordExtension do
-  let(:account_1000) { FactoryGirl.create(:account, :number => 1000, :kind => :asset) }
-  let(:account_1200) { FactoryGirl.create(:account, :number => 1200, :kind => :asset) }
+  let(:account_1000) { FactoryBot.create(:account, number: 1000, kind: :asset) }
+  let(:account_1200) { FactoryBot.create(:account, number: 1200, kind: :asset) }
 
   describe 'ledger with associated account' do
-    let(:ledger) { Ledger.create! :bank_name => 'Sparkasse' }
-    let!(:account) { ledger.create_keepr_account! :number => '1250', :kind => :asset, :name => 'Girokonto' }
+    let(:ledger) { Ledger.create! bank_name: 'Sparkasse' }
+    let!(:account) { ledger.create_keepr_account! number: '1250', kind: :asset, name: 'Girokonto' }
 
     it 'has keepr_account' do
       expect(ledger.keepr_account).to eq(account)
     end
 
     it 'has keepr_postings' do
-      journal = Keepr::Journal.create! :keepr_postings_attributes => [
-                               { :keepr_account => account,      :amount => 30, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 30, :side => 'credit' }
-                             ]
-      other_journal = Keepr::Journal.create! :keepr_postings_attributes => [
-                               { :keepr_account => account_1000, :amount => 20, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 20, :side => 'credit' }
-                             ]
+      Keepr::Journal.create! keepr_postings_attributes: [
+        { keepr_account: account,      amount: 30, side: 'debit'  },
+        { keepr_account: account_1200, amount: 30, side: 'credit' }
+      ]
+      Keepr::Journal.create! keepr_postings_attributes: [
+        { keepr_account: account_1000, amount: 20, side: 'debit'  },
+        { keepr_account: account_1200, amount: 20, side: 'credit' }
+      ]
 
       expect(ledger.keepr_postings.count).to eq(1)
       expect(ledger.keepr_postings.first.amount).to eq(30)
@@ -28,23 +30,23 @@ describe Keepr::ActiveRecordExtension do
   end
 
   describe 'contact with multiple associated accounts' do
-    let(:contact) { Contact.create! :name => 'John Doe' }
-    let(:account1) { contact.keepr_accounts.create! :number => '70001', :kind => :debtor, :name => "Doe's main account" }
-    let(:account2) { contact.keepr_accounts.create! :number => '70002', :kind => :debtor, :name => "Doe's second account" }
+    let(:contact) { Contact.create! name: 'John Doe' }
+    let(:account1) { contact.keepr_accounts.create! number: '70001', kind: :debtor, name: "Doe's main account" }
+    let(:account2) { contact.keepr_accounts.create! number: '70002', kind: :debtor, name: "Doe's second account" }
 
     it 'has multiple keepr_accounts' do
       expect(contact.keepr_accounts).to eq([account1, account2])
     end
 
     it 'has keepr_postings' do
-      journal = Keepr::Journal.create! :keepr_postings_attributes => [
-                               { :keepr_account => account1,     :amount => 30, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 30, :side => 'credit' }
-                             ]
-      other_journal = Keepr::Journal.create! :keepr_postings_attributes => [
-                               { :keepr_account => account_1000, :amount => 20, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 20, :side => 'credit' }
-                             ]
+      Keepr::Journal.create! keepr_postings_attributes: [
+        { keepr_account: account1,     amount: 30, side: 'debit'  },
+        { keepr_account: account_1200, amount: 30, side: 'credit' }
+      ]
+      Keepr::Journal.create! keepr_postings_attributes: [
+        { keepr_account: account_1000, amount: 20, side: 'debit'  },
+        { keepr_account: account_1200, amount: 20, side: 'credit' }
+      ]
 
       expect(contact.keepr_postings.count).to eq(1)
       expect(contact.keepr_postings.first.amount).to eq(30)
@@ -53,11 +55,11 @@ describe Keepr::ActiveRecordExtension do
 
   describe 'Document with associated journal' do
     subject do
-      document = Document.create! :number => 'RE-2013-10-12345'
-      Keepr::Journal.create! :accountable => document,
-                             :keepr_postings_attributes => [
-                               { :keepr_account => account_1000, :amount => 100.99, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 100.99, :side => 'credit' }
+      document = Document.create! number: 'RE-2013-10-12345'
+      Keepr::Journal.create! accountable: document,
+                             keepr_postings_attributes: [
+                               { keepr_account: account_1000, amount: 100.99, side: 'debit'  },
+                               { keepr_account: account_1200, amount: 100.99, side: 'credit' }
                              ]
       document
     end
@@ -69,16 +71,16 @@ describe Keepr::ActiveRecordExtension do
   end
 
   describe 'scopes' do
-    let!(:unbooked_document) { Document.create! :number => 'Unbooked' }
-    let!(:booked_document) {
-      document = Document.create! :number => 'Booked'
-      Keepr::Journal.create! :accountable => document,
-                             :keepr_postings_attributes => [
-                               { :keepr_account => account_1000, :amount => 100.99, :side => 'debit'  },
-                               { :keepr_account => account_1200, :amount => 100.99, :side => 'credit' }
+    let!(:unbooked_document) { Document.create! number: 'Unbooked' }
+    let!(:booked_document) do
+      document = Document.create! number: 'Booked'
+      Keepr::Journal.create! accountable: document,
+                             keepr_postings_attributes: [
+                               { keepr_account: account_1000, amount: 100.99, side: 'debit'  },
+                               { keepr_account: account_1200, amount: 100.99, side: 'credit' }
                              ]
       document
-    }
+    end
 
     describe :keepr_booked do
       subject { Document.keepr_booked }