mobile-enhancements 0.0.3 → 0.0.4

data/lib/mobile_enhancements/railtie.rb

@@ -12,6 +12,8 @@ module MobileEnhancements
       ActionController::Base.send(:include, MobileEnhancements::HelperDelegation)
       # setup the layout calculation
       ActionController::Base.layout :determine_layout
+      # include the UrlHelper into action view
+      ActionView::Base.send(:include, UrlHelper)
       # if we have a custom mobile format
       if format = MobileEnhancements.configuration.mobile_format
         # register it as an alias to the HTML mime-type

data/lib/mobile_enhancements/version.rb

@@ -1,3 +1,3 @@
 module MobileEnhancements
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/spec/controllers/test_controller_spec.rb

@@ -1,12 +1,26 @@
 require "spec_helper"
 
 describe TestController do
+  render_views
+
   context "via the mobile UI" do
     describe "GET /#{MobileEnhancements.configuration.mobile_path_prefix}/two" do
       before { get :two, mobile: MobileEnhancements.configuration.mobile_path_prefix }
       
       it "should render 'two, #{MobileEnhancements.configuration.mobile_format}'" do
-        expect(@response.body).to eq "two, #{MobileEnhancements.configuration.mobile_format}"
+        expect(response.body).to eq "two, #{MobileEnhancements.configuration.mobile_format}"
+      end
+    end
+    
+    describe "GET /#{MobileEnhancements.configuration.mobile_path_prefix}/path_rendering" do
+      before { get :path_rendering, mobile: MobileEnhancements.configuration.mobile_path_prefix }
+      
+      it "should render a mobile path" do
+        expect(response.body).to eq "/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
+      end
+      
+      it "should render the path_rendering template" do
+        expect(response).to render_template "path_rendering"
       end
     end
     
@@ -14,7 +28,7 @@ describe TestController do
       before { get :path_checking, mobile: MobileEnhancements.configuration.mobile_path_prefix }
       
       it "should render a mobile URL" do
-        expect(@response.body).to eq "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
+        expect(response.body).to eq "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
       end
     end
     
@@ -22,7 +36,7 @@ describe TestController do
       before { get :redirect_checking, mobile: MobileEnhancements.configuration.mobile_path_prefix }
       
       it "should redirect to a mobile URL" do
-        expect(@response.headers["Location"]).to eq "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
+        expect(response).to redirect_to "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
       end
     end
     
@@ -30,7 +44,7 @@ describe TestController do
       before { get :fixed_redirect_checking, mobile: MobileEnhancements.configuration.mobile_path_prefix }
       
       it "should redirect to a mobile URL" do
-        expect(@response.headers["Location"]).to eq "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
+        expect(response).to redirect_to "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
       end
     end
   end
@@ -40,7 +54,19 @@ describe TestController do
       before { get :two }
       
       it "should render 'two, html'" do
-        expect(@response.body).to eq "two, html"
+        expect(response.body).to eq "two, html"
+      end
+    end
+    
+    describe "GET /path_rendering" do
+      before { get :path_rendering }
+      
+      it "should render a desktop path" do
+        expect(response.body).to eq "/two"
+      end
+      
+      it "should render the path_rendering template" do
+        expect(response).to render_template "path_rendering"
       end
     end
     
@@ -48,7 +74,7 @@ describe TestController do
       before { get :path_checking }
       
       it "should render a desktop URL" do
-        expect(@response.body).to eq "http://test.host/two"
+        expect(response.body).to eq "http://test.host/two"
       end
     end
     
@@ -56,7 +82,7 @@ describe TestController do
       before { get :redirect_checking }
       
       it "should redirect to a desktop URL" do
-        expect(@response.headers["Location"]).to eq "http://test.host/two"
+        expect(response).to redirect_to "http://test.host/two"
       end
     end
     
@@ -64,7 +90,7 @@ describe TestController do
       before { get :fixed_redirect_checking }
       
       it "should redirect to a mobile URL" do
-        expect(@response.headers["Location"]).to eq "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
+        expect(response).to redirect_to "http://test.host/#{MobileEnhancements.configuration.mobile_path_prefix}/two"
       end
     end
   end

data/spec/internal/app/controllers/test_controller.rb

@@ -6,6 +6,8 @@ class TestController < ActionController::Base
   alias_method :three, :one
   alias_method :four, :one
   
+  def path_rendering; end
+  
   def path_checking
     render text: optional_mobile_url
   end

data/spec/internal/app/views/layouts/application.html.erb

@@ -0,0 +1 @@
+<%= yield %>

data/spec/internal/app/views/layouts/application.mobile.erb

@@ -0,0 +1 @@
+<%= yield %>

data/spec/internal/app/views/test/path_rendering.html.erb

@@ -0,0 +1 @@
+<%= optional_mobile_path %>

data/spec/internal/app/views/test/path_rendering.mobile.erb

@@ -0,0 +1 @@
+<%= optional_mobile_path %>

data/spec/internal/config/routes.rb

@@ -6,6 +6,7 @@ Rails.application.routes.draw do
     get "five", to: "test#path_checking"
     get "six", to: "test#redirect_checking"
     get "seven", to: "test#fixed_redirect_checking"
+    get "eight", to: "test#path_rendering"
   end
   
   mobile_only do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mobile-enhancements
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -150,6 +150,10 @@ files:
 - mobile-enhancements.gemspec
 - spec/controllers/test_controller_spec.rb
 - spec/internal/app/controllers/test_controller.rb
+- spec/internal/app/views/layouts/application.html.erb
+- spec/internal/app/views/layouts/application.mobile.erb
+- spec/internal/app/views/test/path_rendering.html.erb
+- spec/internal/app/views/test/path_rendering.mobile.erb
 - spec/internal/config/initializers/mobile_enhancements.rb
 - spec/internal/config/routes.rb
 - spec/mobile_enhancements/configuration_spec.rb
@@ -186,6 +190,10 @@ summary: A few features to aid Rails apps with separate mobile-targeted UIs.
 test_files:
 - spec/controllers/test_controller_spec.rb
 - spec/internal/app/controllers/test_controller.rb
+- spec/internal/app/views/layouts/application.html.erb
+- spec/internal/app/views/layouts/application.mobile.erb
+- spec/internal/app/views/test/path_rendering.html.erb
+- spec/internal/app/views/test/path_rendering.mobile.erb
 - spec/internal/config/initializers/mobile_enhancements.rb
 - spec/internal/config/routes.rb
 - spec/mobile_enhancements/configuration_spec.rb