Add padding to video duration (#1812)

* Add proper padding in video_duration_in_minutes

* Add proper padding in video_duration_in_minutes

* Remove ffmpeg from travis for now

app/models/article.rb

@@ -137,9 +137,8 @@ class Article < ApplicationRecord
                   per_environment: true,
                   enqueue: :trigger_delayed_index do
       attributes :title, :path, :class_name, :comments_count, :reading_time,
-        :tag_list, :positive_reactions_count, :id, :hotness_score, :score,
-        :readable_publish_date, :flare_tag, :user_id, :organization_id,
-        :cloudinary_video_url, :video_duration_in_minutes
+        :tag_list, :positive_reactions_count, :id, :hotness_score, :score, :readable_publish_date,
+        :flare_tag, :user_id, :organization_id, :cloudinary_video_url, :video_duration_in_minutes
       attribute :published_at_int do
         published_at.to_i
       end
@@ -395,6 +394,7 @@ def cloudinary_video_url
   def video_duration_in_minutes
     minutes = (video_duration_in_seconds.to_i / 60) % 60
     seconds = video_duration_in_seconds.to_i % 60
+    seconds = "0#{seconds}" if seconds.to_s.size == 1
     "#{minutes}:#{seconds}"
   end
 

spec/models/article_spec.rb

@@ -226,6 +226,16 @@ def build_and_validate_article(*args)
       article.save
       expect(article).to be_valid
     end
+
+    it "has padded video_duration_in_minutes" do
+      article.video_duration_in_seconds = 1141
+      expect(article.video_duration_in_minutes).to eq("19:01")
+    end
+
+    it "has correctly non-padded seconds in video_duration_in_minutes" do
+      article.video_duration_in_seconds = 1161
+      expect(article.video_duration_in_minutes).to eq("19:21")
+    end
   end
 
   describe "queries" do