Resolve issue related to wrong specs and non counting extra_working_days

lib/business/calendar.rb

@@ -178,6 +178,13 @@ def business_days_between(date1, date2)
         in_range && on_biz_day
       end
 
+      num_biz_days += extra_working_dates.count do |extra_day|
+        in_range = full_weeks_range.cover?(extra_day)
+        # Add holiday if it is marked as extra working day
+        on_weekend = !working_days.include?(extra_day.strftime("%a").downcase)
+        in_range && on_weekend
+      end
+
       remaining_range = (date2 - remaining_days...date2)
       # Loop through each day in remaining_range and count if a business day
       num_biz_days + remaining_range.count { |a| business_day?(a) }

spec/business/calendar_spec.rb

@@ -628,7 +628,7 @@
             let(:date_1) { "Wed 28/5/2014" }
             let(:date_2) { "Thu 12/6/2014" }
 
-            it { is_expected.to eq(11) }
+            it { is_expected.to eq(12) }
           end
         end
 
@@ -651,7 +651,7 @@
             let(:date_1) { "Wed 25/6/2014" }
             let(:date_2) { "Tue 8/7/2014" }
 
-            it { is_expected.to eq(8) }
+            it { is_expected.to eq(9) }
           end
         end
       end
@@ -683,7 +683,7 @@
             let(:date_1) { "Sat 31/5/2014" }
             let(:date_2) { "Fri 13/6/2014" }
 
-            it { is_expected.to eq(8) }
+            it { is_expected.to eq(9) }
           end
         end
 
@@ -698,7 +698,7 @@
             let(:date_1) { "Sat 31/5/2014" }
             let(:date_2) { "Sun 8/6/2014" }
 
-            it { is_expected.to be(5) }
+            it { is_expected.to eq(6) }
           end
 
           context "including business, weekend days, and holidays" do
@@ -709,9 +709,9 @@
 
           context "including business, weekend, holiday days & working date" do
             let(:date_1) { "Sat 31/5/2014" }
-            let(:date_2) { "Sun 14/6/2014" }
+            let(:date_2) { "Sun 15/6/2014" }
 
-            it { is_expected.to be(9) }
+            it { is_expected.to be(10) }
           end
         end
 
@@ -726,7 +726,7 @@
             let(:date_1) { "Sat 31/5/2014" }
             let(:date_2) { "Thu 12/6/2014" }
 
-            it { is_expected.to eq(8) }
+            it { is_expected.to eq(9) }
           end
         end
 
@@ -746,9 +746,9 @@
           end
 
           context "including business, weekend days, holidays & working date" do
-            let(:date_2) { "Tue 13/6/2014" }
+            let(:date_2) { "Fri 13/6/2014" }
 
-            it { is_expected.to eq(8) }
+            it { is_expected.to eq(9) }
           end
         end
       end
@@ -807,10 +807,10 @@
           end
 
           context "including business/weekend days, holidays & working date" do
-            let(:date_1) { "27/5/2014" }
+            let(:date_1) { "Tue 27/5/2014" }
             let(:date_2) { "Thu 12/6/2014" }
 
-            it { is_expected.to eq(11) }
+            it { is_expected.to eq(12) }
           end
         end
 
@@ -850,13 +850,13 @@
           context "including working date, working & weekend days" do
             let(:date_2) { "Tue 10/6/2014" }
 
-            it { is_expected.to eq(6) }
+            it { is_expected.to eq(7) }
           end
 
           context "including working date, working & weekend days & holiday" do
-            let(:date_2) { "Tue 13/6/2014" }
+            let(:date_2) { "Fri 13/6/2014" }
 
-            it { is_expected.to eq(8) }
+            it { is_expected.to eq(9) }
           end
         end
 
@@ -913,7 +913,7 @@
             let(:date_1) { "Sat 28/6/2014" }
             let(:date_2) { "Sat 5/7/2014" }
 
-            it { is_expected.to eq(4) }
+            it { is_expected.to eq(5) }
           end
         end
       end