empty? changed to length.zero? to respect expiry

Author: benngarcia

lib/prometheus_exporter/server/good_job_collector.rb

@@ -23,7 +23,7 @@ def type
     end
 
     def metrics
-      return [] if good_job_metrics.empty?
+      return [] if good_job_metrics.length.zero?
 
       good_job_metrics.each(&method(:process_metric))
       gauges.values

lib/prometheus_exporter/server/metrics_container.rb

@@ -2,6 +2,8 @@
 
 module PrometheusExporter::Server
   class MetricsContainer
+    # Since MetricsContainer defines #each, we can include Enumerable
+    include Enumerable
     METRIC_MAX_AGE = 60
     METRIC_EXPIRE_ATTR = "_expire_at"
 
@@ -34,10 +36,6 @@ def size(&blk)
     end
     alias_method :length, :size
 
-    def map(&blk)
-      wrap_expire(:map, &blk)
-    end
-
     def each(&blk)
       wrap_expire(:each, &blk)
     end