diff options
| author | George Claghorn <george.claghorn@gmail.com> | 2019-02-14 13:36:04 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-02-14 13:36:04 -0500 |
| commit | eaf6d188e4b8c72cce6d08170678557e8c3d1713 (patch) | |
| tree | 80bdfa0d5c8ec24120a36f0245b5440deea44367 | |
| parent | 5f7180676bd60b44566a24b293376d54ff5d0d94 (diff) | |
| parent | d3f92261902d09adf54dbc211e3a1080d943440b (diff) | |
| download | rails-eaf6d188e4b8c72cce6d08170678557e8c3d1713.tar.gz rails-eaf6d188e4b8c72cce6d08170678557e8c3d1713.tar.bz2 rails-eaf6d188e4b8c72cce6d08170678557e8c3d1713.zip | |
Merge pull request #35268 from abhaynikam/35252-add-delegate-for-path-for-method
Fixes missing method `path_for` when using MirrorService with DiskService as the primary service
| -rw-r--r-- | activestorage/lib/active_storage/service/mirror_service.rb | 2 | ||||
| -rw-r--r-- | activestorage/test/service/mirror_service_test.rb | 4 |
2 files changed, 5 insertions, 1 deletions
diff --git a/activestorage/lib/active_storage/service/mirror_service.rb b/activestorage/lib/active_storage/service/mirror_service.rb index 75274f81b3..aa41df304e 100644 --- a/activestorage/lib/active_storage/service/mirror_service.rb +++ b/activestorage/lib/active_storage/service/mirror_service.rb @@ -9,7 +9,7 @@ module ActiveStorage class Service::MirrorService < Service attr_reader :primary, :mirrors - delegate :download, :download_chunk, :exist?, :url, to: :primary + delegate :download, :download_chunk, :exist?, :url, :path_for, to: :primary # Stitch together from named services. def self.build(primary:, mirrors:, configurator:, **options) #:nodoc: diff --git a/activestorage/test/service/mirror_service_test.rb b/activestorage/test/service/mirror_service_test.rb index 94c751a4ff..249a5652fb 100644 --- a/activestorage/test/service/mirror_service_test.rb +++ b/activestorage/test/service/mirror_service_test.rb @@ -61,4 +61,8 @@ class ActiveStorage::Service::MirrorServiceTest < ActiveSupport::TestCase @service.url(@key, expires_in: 2.minutes, disposition: :inline, filename: filename, content_type: "text/plain") end end + + test "path for file in primary service" do + assert_equal @service.primary.path_for(@key), @service.path_for(@key) + end end |