Add download action for tracks (#487)

app/controllers/tracks_controller.rb

@@ -1,7 +1,7 @@
 class TracksController < ApplicationController
   include ActionController::Live
 
-  before_action :set_track, only: %i[show update destroy audio merge]
+  before_action :set_track, only: %i[show update destroy audio download merge]
 
   has_scope :by_filter, as: 'filter'
   has_scope :by_album, as: 'album_id'
@@ -78,6 +78,13 @@ def audio
     end
   end
 
+  def download
+    audio_file = @track.audio_file
+    raise ActiveRecord::RecordNotFound.new('track has no audio', 'audio') unless audio_file.present? && audio_file.check_self
+
+    send_file audio_file.full_path
+  end
+
   def merge
     @track.merge(Track.find(params[:source_id]))
     # We don't do error handling here. The merge action isn't supposed to fail.

app/policies/track_policy.rb

@@ -33,6 +33,10 @@ def audio?
     index?
   end
 
+  def download?
+    audio?
+  end
+
   def merge?
     create?
   end

config/routes.rb

@@ -159,6 +159,7 @@
       end
       member do
         get 'audio'
+        get 'download'
         post 'merge'
       end
     end

test/controllers/tracks_controller_test.rb

@@ -257,6 +257,16 @@ class TracksControllerTest < ActionDispatch::IntegrationTest
 
     assert_response :success
   end
+
+  test 'should download to user' do
+    location = Location.create(path: Rails.root.join('test/files'))
+    audio_file = create(:audio_file, location:, filename: '/base.flac')
+    track = create(:track, audio_file:)
+
+    get download_track_url(track)
+
+    assert_response :success
+  end
 end
 
 class TracksControllerAudioTest < ActionDispatch::IntegrationTest