calendarium-romanum 0.4.0 → 0.5.0

data/spec/celebration_spec.rb

@@ -20,4 +20,24 @@ describe CR::Celebration do
       end
     end
   end
+
+  describe '#change' do
+    let(:c) { described_class.new('title') }
+
+    it 'produces a new instance' do
+      c2 = c.change rank: CR::Ranks::SOLEMNITY_GENERAL
+      expect(c2).not_to be c
+    end
+
+    it 'sets specified properties' do
+      c2 = c.change rank: CR::Ranks::SOLEMNITY_GENERAL
+      expect(c2.rank).not_to eq c.rank
+      expect(c2.rank).to be CR::Ranks::SOLEMNITY_GENERAL
+    end
+
+    it 'copies the rest' do
+      c2 = c.change rank: CR::Ranks::SOLEMNITY_GENERAL
+      expect(c2.title).to be c.title # reference copying is no problem, Celebrations are immutable
+    end
+  end
 end

data/spec/cli_spec.rb

@@ -1,26 +1,155 @@
 require 'spec_helper'
 require 'calendarium-romanum/cli'
 
-describe CalendariumRomanum::CLI do
-  let(:path_universal_la) { File.expand_path('../../data/universal-la.txt', __FILE__) }
-  let(:path_universal_en) { File.expand_path('../../data/universal-en.txt', __FILE__) }
+describe CalendariumRomanum::CLI, type: :aruba do
+  let(:path_universal_la) { CR::Data::GENERAL_ROMAN_LATIN.path }
+  let(:path_universal_en) { CR::Data::GENERAL_ROMAN_ENGLISH.path }
+
+  before :each do
+    prepend_environment_variable('RUBYLIB', File.expand_path('../../lib', __FILE__) + ':')
+  end
+
   describe 'subcommands' do
     describe 'errors' do
-      it 'raises no exception' do
-        described_class.start(['errors', path_universal_la])
+      describe 'no errors detected' do
+        before(:each) { run "calendariumrom errors #{path_universal_la}" }
+        it { expect(last_command).to be_successfully_executed }
+        it { expect(all_output).to be_empty }
+      end
+
+      describe 'non-existent file' do
+        before(:each) { run 'calendariumrom errors does-not-exist.txt' }
+        it { expect(all_output).to include 'No such file or directory' }
+        it { expect(last_command).to have_exit_status 1 }
       end
 
-      it 'fails on a non-existent file' do
-        expect do
-          described_class.start(['errors', 'does-not-exist.txt'])
-        end.to raise_exception Errno::ENOENT
+      describe 'errors detected' do
+        before(:each) do
+          write_file 'data.txt', file_content
+          run 'calendariumrom errors data.txt'
+        end
+
+        describe 'completely broken record' do
+          let(:file_content) { 'invalid content' }
+
+          it { expect(all_output).to eq "L1: Syntax error, line skipped 'invalid content'\n" }
+          it { expect(last_command).to have_exit_status 1 }
+        end
+
+        describe 'wrong month title (month does not exist)' do
+          let(:file_content) { "= 13\n" }
+
+          it { expect(all_output).to start_with 'L1: Invalid month' }
+          it { expect(last_command).to have_exit_status 1 }
+        end
       end
     end
 
     describe 'cmp' do
-      it 'raises no exception' do
-        described_class.start(['cmp', path_universal_la, path_universal_en])
+      describe 'contents match' do
+        before(:each) { run "calendariumrom cmp #{path_universal_la} #{path_universal_en}" }
+
+        it { expect(all_output).to be_empty }
+        it { expect(last_command).to be_successfully_executed }
+      end
+
+      describe 'contents do not match' do
+        before(:each) do
+          write_file 'cal1.txt', content1
+          write_file 'cal2.txt', content2
+          run 'calendariumrom cmp cal1.txt cal2.txt'
+        end
+
+        describe 'in rank' do
+          let(:content1) { '1/11 : St. None, abbot' }
+          let(:content2) { '1/11 m : St. None, abbot' }
+
+          it { expect(all_output).to include '1/11' }
+          it { expect(all_output).to include 'St. None, abbot' }
+          it { expect(all_output).to include 'differs in rank' }
+          it { expect(last_command).to be_successfully_executed }
+        end
+
+        describe 'in colour' do
+          let(:content1) { '1/11 : St. None, abbot' }
+          let(:content2) { '1/11 R : St. None, abbot and martyr' }
+
+          it { expect(all_output).to include '1/11' }
+          it { expect(all_output).to include 'St. None, abbot' }
+          it { expect(all_output).to include 'differs in colour' }
+          it { expect(last_command).to be_successfully_executed }
+        end
+
+        describe 'in optional memorial count' do
+          let(:content1) { '1/11 : St. None, abbot' }
+          let(:content2) { "1/11 : St. None, abbot\n1/11 : St. Nulla, abbess" }
+
+          it { expect(all_output).to include '1/11' }
+          it { expect(all_output).to include 'St. Nulla, abbess' }
+          it { expect(all_output).to include 'only in cal2.txt' }
+          it { expect(last_command).to be_successfully_executed }
+        end
+
+        describe 'only in one source' do
+          let(:content1) { '' }
+          let(:content2) { '1/11 : St. None, abbot' }
+
+          it { expect(all_output).to include '1/11' }
+          it { expect(all_output).to include 'St. None, abbot' }
+          it { expect(all_output).to include 'only in cal2.txt' }
+          it { expect(last_command).to be_successfully_executed }
+        end
       end
     end
+
+    describe 'query' do
+      describe 'invalid data file from file system' do
+        before(:each) do
+          write_file 'invalid.txt', 'Foo bar baz'
+          run 'calendariumrom query --calendar invalid.txt 2017-10-03'
+        end
+
+        it { expect(all_output).to include 'Invalid file format.' }
+        it { expect(last_command).to have_exit_status 1 }
+      end
+
+      describe 'correct data file from file system' do
+        before(:each) { run "calendariumrom query --calendar #{path_universal_la} 2017-10-03" }
+
+        it { expect(all_output).to include 'season: Ordinary Time' }
+        it { expect(last_command).to be_successfully_executed }
+      end
+
+      describe 'correct season naming' do
+        before(:each) { run 'calendariumrom query 2017-10-03' }
+
+        it { expect(all_output).to include 'season: Ordinary Time' }
+        it { expect(last_command).to be_successfully_executed }
+      end
+
+      describe 'correct month querying' do
+        before(:each) { run 'calendariumrom query 2015-06' }
+
+        it { expect(all_output).to include 'Saint Cyril of Alexandria' }
+        it { expect(all_output).to include 'Saint Anthony of Padua, priest and doctor' }
+        it { expect(last_command).to be_successfully_executed }
+      end
+
+      describe 'correct year querying' do
+        before(:each) { run 'calendariumrom query 2013' }
+
+        it { expect(all_output).to include 'Saint John the Apostle and evangelist' }
+        it { expect(all_output).to include 'Saint Paul of the Cross, priest' }
+        it { expect(last_command).to be_successfully_executed }
+      end
+    end
+
+    describe 'calendars' do
+      before(:each) { run 'calendariumrom calendars' }
+
+      it { expect(all_output).to include 'universal-en' }
+      it { expect(all_output).to include 'czech-praha-cs' }
+      it { expect(last_command).to be_successfully_executed }
+    end
   end
 end

data/spec/date_parser_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+describe CR::Util::DateParser do
+  describe 'new' do
+    it 'should accept a full date YYYY/MM/DD' do
+      date_str = '2017/10/06'
+      expect do
+        described_class.new(date_str)
+      end.not_to raise_exception
+    end
+
+    it 'should accept a full date YYYY-MM-DD' do
+      date_str = '2017-10-06'
+      expect do
+        described_class.new(date_str)
+      end.not_to raise_exception
+    end
+
+    it 'should accept a month range YYYY/MM' do
+      date_str = '2017/10'
+      expect do
+        described_class.new(date_str)
+      end.not_to raise_exception
+    end
+
+    it 'should accept a month range YYYY-MM' do
+      date_str = '2017-10'
+      expect do
+        described_class.new(date_str)
+      end.not_to raise_exception
+    end
+
+    it 'should accept a year range YYYY' do
+      date_str = '2017'
+      expect do
+        described_class.new(date_str)
+      end.not_to raise_exception
+    end
+
+    it 'should not accept an invalid month' do
+      date_str = '2017-15'
+      expect do
+        described_class.new(date_str)
+      end.to raise_exception(ArgumentError)
+    end
+
+    it 'should not accept an invalid day' do
+      date_str = '2017-10-48'
+      expect do
+        described_class.new(date_str)
+      end.to raise_exception(ArgumentError)
+    end
+
+    it 'should not accept a not-matching string' do
+      date_str = 'foobar'
+      expect do
+        described_class.new(date_str)
+      end.to raise_exception(ArgumentError)
+    end
+
+    it 'should not accept an empty string' do
+      date_str = ''
+      expect do
+        described_class.new(date_str)
+      end.to raise_exception(ArgumentError)
+    end
+  end
+end

data/spec/date_spec.rb

@@ -4,14 +4,14 @@ require 'spec_helper'
 describe Date do
   describe '#-' do
     it 'returns Rational' do
-      date_diff = Date.new(2013,5,5) - Date.new(2013,5,1)
+      date_diff = Date.new(2013, 5, 5) - Date.new(2013, 5, 1)
       expect(date_diff).to be_a Rational
       expect(date_diff.numerator).to eq 4
     end
   end
 
   describe 'range' do
-    let(:range) { (Date.new(2000,1,1) .. Date.new(2010,1,1)) }
+    let(:range) { (Date.new(2000, 1, 1) .. Date.new(2010, 1, 1)) }
 
     # please note that the RSpec 'include' matcher cannot
     # be used here, because it does't simply test
@@ -20,11 +20,11 @@ describe Date do
     # of DateTime in a Date range.
     describe 'test Date inclusion' do
       it 'includes' do
-        expect(range.include?(Date.new(2005,3,3))).to be true
+        expect(range.include?(Date.new(2005, 3, 3))).to be true
       end
 
       it 'excludes' do
-        expect(range.include?(Date.new(1995,3,3))).to be false
+        expect(range.include?(Date.new(1995, 3, 3))).to be false
       end
     end
 
@@ -32,22 +32,22 @@ describe Date do
       # DateTime without time details is treated just like Date
       describe 'without time specified' do
         it 'excludes' do
-          expect(range.include?(DateTime.new(1995,3,3))).to be false
+          expect(range.include?(DateTime.new(1995, 3, 3))).to be false
         end
 
         it 'includes' do
-          expect(range.include?(DateTime.new(2005,3,3))).to be true
+          expect(range.include?(DateTime.new(2005, 3, 3))).to be true
         end
       end
 
       # treatment of a DateTime with time details is different
       describe 'with time specified' do
         it 'excludes a DateTime not falling in the range' do
-          expect(range.include?(DateTime.new(1995,3,3,1,1,1))).to be false
+          expect(range.include?(DateTime.new(1995, 3, 3, 1, 1, 1))).to be false
         end
 
         it '... but also one *falling* in the range!' do
-          expect(range.include?(DateTime.new(2005,3,3,1,1,1))).to be false
+          expect(range.include?(DateTime.new(2005, 3, 3, 1, 1, 1))).to be false
         end
       end
     end

data/spec/day_spec.rb

@@ -1,16 +1,37 @@
 require_relative 'spec_helper'
 
 describe CR::Day do
-  describe '#==' do
-    let(:d) do
-      described_class.new(
-        date: Date.today,
-        season: CR::Seasons::ORDINARY,
-        season_week: 1,
-        celebrations: [CR::Celebration.new]
-      )
+  let(:d) do
+    described_class.new(
+      date: Date.today,
+      season: CR::Seasons::ORDINARY,
+      season_week: 1,
+      celebrations: [CR::Celebration.new],
+      vespers: vespers
+    )
+  end
+
+  let(:vespers) { nil }
+
+  describe '.new' do
+    it 'works without arguments' do
+      expect do
+        described_class.new
+      end.not_to raise_exception
     end
 
+    it 'makes a shallow copy of celebrations' do
+      celebrations = [CR::Celebration.new]
+      day = described_class.new celebrations: celebrations
+
+      expect(day.celebrations).to eq celebrations
+      expect(day.celebrations).not_to be celebrations
+
+      expect(day.celebrations[0]).to be celebrations[0]
+    end
+  end
+
+  describe '#==' do
     describe 'same content' do
       let(:d2) do
         described_class.new(
@@ -55,5 +76,33 @@ describe CR::Day do
         expect(d).not_to eq d2
       end
     end
+
+    describe 'different Vespers' do
+      let(:d2) do
+        described_class.new(
+          date: Date.today,
+          season: CR::Seasons::ORDINARY,
+          season_week: 1,
+          celebrations: [CR::Celebration.new],
+          vespers: CR::Temporale::CelebrationFactory.palm_sunday
+        )
+      end
+
+      it 'is different' do
+        expect(d).not_to eq d2
+      end
+    end
+  end
+
+  describe '#vespers_from_following?' do
+    describe 'vespers not set' do
+      it { expect(d.vespers_from_following?).to be false }
+    end
+
+    describe 'vespers set' do
+      let(:vespers) { CR::Celebration.new }
+
+      it { expect(d.vespers_from_following?).to be true }
+    end
   end
 end

data/spec/i18n_spec.rb

@@ -53,7 +53,6 @@ describe 'internationalization' do
     let(:locale) { 'de' }
 
     it 'attemt to switch to it fails by default' do
-
     end
   end
 end

data/spec/ordinalizer_spec.rb

@@ -1,21 +1,35 @@
 require_relative 'spec_helper'
 
 describe CalendariumRomanum::Ordinalizer do
-  describe '#english_ordinal' do
-    {
-      1 => '1st',
-      2 => '2nd',
-      3 => '3rd',
-      4 => '4th',
-      11 => '11th',
-      12 => '12th',
-      13 => '13th',
-      21 => '21st',
-      22 => '22nd',
-      23 => '23rd',
-    }.each_pair do |number, ordinal|
-      it number do
-        expect(described_class.english_ordinal(number)).to eq ordinal
+  describe '#ordinal' do
+    describe 'English' do
+      {
+        1 => '1st',
+        2 => '2nd',
+        3 => '3rd',
+        4 => '4th',
+        11 => '11th',
+        12 => '12th',
+        13 => '13th',
+        21 => '21st',
+        22 => '22nd',
+        23 => '23rd',
+      }.each_pair do |number, ordinal|
+        it number do
+          expect(described_class.ordinal(number, locale: :en)).to eq ordinal
+        end
+      end
+    end
+
+    describe 'French' do
+      {
+        1 => '1er',
+        2 => '2ème',
+        3 => '3ème',
+      }.each_pair do |number, ordinal|
+        it number do
+          expect(described_class.ordinal(number, locale: :fr)).to eq ordinal
+        end
       end
     end
   end