Merge pull request #1572 from bquorning/spec-append-and-prepend-view-path

Spec append and prepend view path

Author: JonRowe

lib/rspec/rails/view_rendering.rb

@@ -71,17 +71,17 @@ def self.call(_template)
       # @private
       module EmptyTemplates
         def prepend_view_path(new_path)
-          lookup_context.view_paths.unshift(*_path_decorator(new_path))
+          lookup_context.view_paths.unshift(*_path_decorator(*new_path))
         end
 
         def append_view_path(new_path)
-          lookup_context.view_paths.push(*_path_decorator(new_path))
+          lookup_context.view_paths.push(*_path_decorator(*new_path))
         end
 
       private
 
-        def _path_decorator(path)
-          EmptyTemplateResolver.new(path)
+        def _path_decorator(*paths)
+          paths.map { |path| EmptyTemplateResolver.new(path) }
         end
       end
 

spec/rspec/rails/view_rendering_spec.rb

@@ -121,5 +121,27 @@ def example.controller
         end
       end
     end
+
+    context 'when render_views? is false' do
+      let(:controller) { ActionController::Base.new }
+
+      before { controller.extend(ViewRendering::EmptyTemplates) }
+
+      it 'supports manipulating view paths' do
+        controller.prepend_view_path 'app/views'
+        controller.append_view_path 'app/others'
+        expect(controller.view_paths.map(&:to_s)).to match_paths 'app/views', 'app/others'
+      end
+
+      it 'supports manipulating view paths with arrays' do
+        controller.prepend_view_path ['app/views', 'app/legacy_views']
+        controller.append_view_path ['app/others', 'app/more_views']
+        expect(controller.view_paths.map(&:to_s)).to match_paths 'app/views', 'app/legacy_views', 'app/others', 'app/more_views'
+      end
+
+      def match_paths(*paths)
+        eq paths.map { |path| File.expand_path path }
+      end
+    end
   end
 end