Extract Duration#seconds_to_formatted_duration

app/helpers/talks_helper.rb

@@ -1,11 +1,5 @@
 module TalksHelper
   def seconds_to_formatted_duration(seconds)
-    parts = ActiveSupport::Duration.build(seconds).parts
-
-    values = [parts[:hours], parts.fetch(:minutes, 0), parts.fetch(:seconds, 0)].select(&:present?)
-
-    return "??:??" if values.any?(&:negative?)
-
-    values.map { |value| value.to_s.rjust(2, "0") }.join(":")
+    Duration.seconds_to_formatted_duration(seconds)
   end
 end

app/lib/duration.rb

@@ -0,0 +1,21 @@
+class Duration
+  def self.seconds_to_formatted_duration(seconds)
+    if seconds.is_a?(Integer)
+      parts = ActiveSupport::Duration.build(seconds).parts
+    elsif seconds.is_a?(ActiveSupport::Duration)
+      parts = seconds.parts
+    else
+      raise "seconds (`#{seconds.inspect}`) is not an Integer or ActiveSupport::Duration"
+    end
+
+    values = [
+      parts.fetch(:hours, nil),
+      parts.fetch(:minutes, 0),
+      parts.fetch(:seconds, 0)
+    ].compact
+
+    return "??:??" if values.any?(&:negative?)
+
+    values.map { |value| value.to_s.rjust(2, "0") }.join(":")
+  end
+end

app/models/static/video.rb

@@ -28,13 +28,7 @@ def duration_fs
     end
 
     def duration_to_formatted_cue(duration)
-      parts = [
-        duration.parts.fetch(:hours, nil),
-        duration.parts.fetch(:minutes, 0),
-        duration.parts.fetch(:seconds, 0)
-      ].compact
-
-      parts.map { |x| x.to_s.rjust(2, "0") }.join(":")
+      Duration.seconds_to_formatted_duration(duration)
     end
 
     def duration

app/models/talk.rb

@@ -324,7 +324,7 @@ def formatted_date
   end
 
   def formatted_duration
-    duration.parts.values.map { |x| x.to_s.rjust(2, "0") }.join(":")
+    Duration.seconds_to_formatted_duration(duration)
   end
 
   def duration