Merge pull request #1461 from PRX/fix/publish_transcripts

Publish RSS after copying transcript

Author: cavis

app/models/transcript.rb

@@ -22,6 +22,7 @@ class Transcript < ApplicationRecord
   has_many :tasks, as: :owner
 
   before_validation :initialize_attributes, on: :create
+  after_save :publish!
 
   validates :original_url, presence: true
 
@@ -98,6 +99,10 @@ def copy_media(force = false)
     end
   end
 
+  def publish!
+    episode&.publish! if status_complete? && status_previously_changed?
+  end
+
   def retryable?
     if %w[started created processing retrying].include?(status)
       last_event = task&.updated_at || updated_at || Time.now

test/models/transcript_test.rb

@@ -57,6 +57,27 @@
     end
   end
 
+  describe "#publish!" do
+    let(:transcript) { create(:transcript, status: "started") }
+
+    it "publishes the episode when complete and status has changed" do
+      publish = Minitest::Mock.new
+      publish.expect(:call, nil) { raise "should not have called" }
+
+      transcript.episode.stub(:publish!, publish) do
+        transcript.update(status: "processing")
+      end
+
+      publish = Minitest::Mock.new
+      publish.expect(:call, nil)
+      transcript.episode.stub(:publish!, publish) do
+        transcript.update(status: "complete")
+      end
+
+      assert publish.verify
+    end
+  end
+
   describe "rss_mime_type" do
     it "returns the mime type for the transcript format" do
       assert_equal "text/html", transcript.rss_mime_type