cdn_var_substitutor.rb in katello-4.14.0.rc1

- old
+ new

@@ -1,10 +1,10 @@
 module Katello
   module Util
     class CdnVarSubstitutor
-      attr_reader :good_listings
-      attr_reader :bad_listings
+      attr_reader :good_listings, :bad_listings
+
       # cdn_resource - an object providing access to CDN. It has to
       # provide a get method that takes a path (e.g.
       # /content/rhel/6.2/listing) and returns the body response)
       def initialize(cdn_resource)
         @resource = cdn_resource
@@ -66,10 +66,12 @@
             end
           end
 
           futures.each do |future|
             resolved << future.value
-            Rails.logger.error("Failed at scanning for repository: #{future.reason}") if future.rejected?
+            if future.rejected?
+              fail Errors::CdnSubstitutionError, "Failed at scanning for repository: #{future.reason}"
+            end
           end
         end
 
         find_substitutions(resolved.compact.flatten)
       end