Handle market holidays during holding sync (#1292)

* Handle market holidays during holding sync

* Use informal holidays instead of custom override

Gemfile

@@ -48,6 +48,7 @@ gem "csv"
 gem "redcarpet"
 gem "stripe"
 gem "intercom-rails"
+gem "holidays"
 
 group :development, :test do
   gem "debug", platforms: %i[mri windows]

Gemfile.lock

@@ -183,6 +183,7 @@ GEM
       thor (>= 1.0.0)
     hashdiff (1.1.1)
     highline (3.0.1)
+    holidays (8.8.0)
     hotwire-livereload (1.4.1)
       actioncable (>= 6.0.0)
       listen (>= 3.0.0)
@@ -486,6 +487,7 @@ DEPENDENCIES
   faraday-multipart
   faraday-retry
   good_job
+  holidays
   hotwire-livereload
   i18n-tasks
   image_processing (>= 1.2)

app/models/gapfiller.rb

@@ -32,7 +32,11 @@ class Gapfiller
     attr_reader :date_range, :cache
 
     def should_gapfill?(date, record)
-      date.on_weekend? && record.nil?
+      (date.on_weekend? || holiday?(date)) && record.nil?
+    end
+
+    def holiday?(date)
+      Holidays.on(date, :federalreserve, :us, :informal).any?
     end
 
     def create_gapfilled_record(prev_record, date)

app/models/security/price/provided.rb

@@ -38,13 +38,15 @@ module Security::Price::Provided
 
         if response.success?
           response.prices.map do |price|
-            new_price = Security::Price.new \
+            new_price = Security::Price.find_or_initialize_by(
               ticker: ticker,
-              date: price[:date],
+              date: price[:date]
-              price: price[:price],
+            ) do |p|
-              currency: price[:currency]
+              p.price = price[:price]
+              p.currency = price[:currency]
+            end
 
-            new_price.save! if cache
+            new_price.save! if cache && new_price.new_record?
             new_price
           end
         else