jpmobile 7.0.0 → 7.0.3

data/lib/jpmobile/mobile/android.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::GoogleEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /Android/
+    USER_AGENT_REGEXP = /Android/.freeze
   end
 end

data/lib/jpmobile/mobile/au.rb

@@ -6,15 +6,17 @@ module Jpmobile::Mobile
   class Au < AbstractMobile
     # 対応するUser-Agentの正規表現
     # User-Agent文字列中に "UP.Browser" を含むVodafoneの端末があるので注意が必要
-    USER_AGENT_REGEXP = %r{^(?:KDDI|UP.Browser/.+?)-(.+?) }
+    USER_AGENT_REGEXP = %r{^(?:KDDI|UP.Browser/.+?)-(.+?) }.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@ezweb\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@ezweb\.ne\.jp/.freeze
     # 簡易位置情報取得に対応していないデバイスID
     # http://www.au.kddi.com/ezfactory/tec/spec/eznavi.html
     LOCATION_UNSUPPORTED_DEVICE_ID = %w[PT21 TS25 KCTE TST9 KCU1 SYT5 KCTD TST8 TST7 KCTC SYT4 KCTB KCTA TST6 KCT9 TST5 TST4 KCT8 SYT3 KCT7 MIT1 MAT3 KCT6 TST3 KCT5 KCT4 SYT2 MAT1 MAT2 TST2 KCT3 KCT2 KCT1 TST1 SYT1].freeze
     # GPS取得に対応していないデバイスID
     GPS_UNSUPPORTED_DEVICE_ID = %w[PT21 KC26 SN28 SN26 KC23 SA28 TS25 SA25 SA24 SN23 ST14 KC15 SN22 KC14 ST13 SN17 SY15 CA14 HI14 TS14 KC13 SN15 SN16 SY14 ST12 TS13 CA13 MA13 HI13 SN13 SY13 SN12 SN14 ST11 DN11 SY12 KCTE TST9 KCU1 SYT5 KCTD TST8 TST7 KCTC SYT4 KCTB KCTA TST6 KCT9 TST5 TST4 KCT8 SYT3 KCT7 MIT1 MAT3 KCT6 TST3 KCT5 KCT4 SYT2 MAT1 MAT2 TST2 KCT3 KCT2 KCT1 TST1 SYT1].freeze
 
+    TARGET_PARAMS = %w[ver datum unit lat lon alt time smaj smin vert majaa fm].freeze
+
     # EZ番号(サブスクライバID)があれば返す。無ければ +nil+ を返す。
     def subno
       @request.env['HTTP_X_UP_SUBNO']
@@ -27,7 +29,7 @@ module Jpmobile::Mobile
       return @__posotion = nil if params['lat'].nil? || params['lat'] == '' || params['lon'].nil? || params['lon'] == ''
 
       l = Jpmobile::Position.new
-      l.options = params.select {|x, _| %w[ver datum unit lat lon alt time smaj smin vert majaa fm].include?(x) }
+      l.options = params.select {|x, _| TARGET_PARAMS.include?(x) }
       case params['unit']
       when '1'
         l.lat = params['lat'].to_f
@@ -83,11 +85,7 @@ module Jpmobile::Mobile
           'none'
         end
 
-      if protocol.start_with?('https')
-        false
-      else
-        true
-      end
+      !protocol.start_with?('https')
     end
 
     # 文字コード変換

data/lib/jpmobile/mobile/black_berry.rb

@@ -4,6 +4,6 @@ module Jpmobile::Mobile
   # ==BlackBerry
   class BlackBerry < SmartPhone
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /BlackBerry/
+    USER_AGENT_REGEXP = /BlackBerry/.freeze
   end
 end

data/lib/jpmobile/mobile/ddipocket.rb

@@ -4,7 +4,7 @@ module Jpmobile::Mobile
   # スーパクラスはWillcom。
   class Ddipocket < Willcom
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(DDIPOCKET}
+    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(DDIPOCKET}.freeze
 
     MAIL_ADDRESS_REGEXP = nil # DdipocketはEmail判定だとWillcomと判定させたい
   end

data/lib/jpmobile/mobile/docomo.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # ==DoCoMo携帯電話
   class Docomo < AbstractMobile
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /^DoCoMo/
+    USER_AGENT_REGEXP = /^DoCoMo/.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@docomo\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@docomo\.ne\.jp/.freeze
     # メールのデフォルトのcharset
     MAIL_CHARSET = 'Shift_JIS'.freeze
     # テキスト部分の content-transfer-encoding

data/lib/jpmobile/mobile/emobile.rb

@@ -3,9 +3,9 @@
 module Jpmobile::Mobile
   # ==EMOBILE携帯電話
   class Emobile < AbstractMobile
-    USER_AGENT_REGEXP = %r{^emobile/|^Mozilla/5.0 \(H11T; like Gecko; OpenBrowser\) NetFront/3.4$|^Mozilla/4.0 \(compatible; MSIE 6.0; Windows CE; IEMobile 7.7\) S11HT$}
+    USER_AGENT_REGEXP = %r{^emobile/|^Mozilla/5.0 \(H11T; like Gecko; OpenBrowser\) NetFront/3.4$|^Mozilla/4.0 \(compatible; MSIE 6.0; Windows CE; IEMobile 7.7\) S11HT$}.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@emnet\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@emnet\.ne\.jp/.freeze
     # EMnet対応端末から通知されるユニークなユーザIDを取得する。
     def em_uid
       @request.env['HTTP_X_EM_UID']

data/lib/jpmobile/mobile/ipad.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::UnicodeEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /iPad/
+    USER_AGENT_REGEXP = /iPad/.freeze
   end
 end

data/lib/jpmobile/mobile/iphone.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::UnicodeEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /iPhone/
+    USER_AGENT_REGEXP = /iPhone/.freeze
   end
 end

data/lib/jpmobile/mobile/softbank.rb

@@ -5,14 +5,16 @@ module Jpmobile::Mobile
   # Vodafoneのスーパクラス。
   class Softbank < AbstractMobile
     # 対応するuser-agentの正規表現
-    USER_AGENT_REGEXP = /^(?:SoftBank|Semulator)/
+    USER_AGENT_REGEXP = /^(?:SoftBank|Semulator)/.freeze
     # 対応するメールアドレスの正規表現　ディズニーモバイル対応
-    MAIL_ADDRESS_REGEXP = /.+@(?:softbank\.ne\.jp|disney\.ne\.jp)/
+    MAIL_ADDRESS_REGEXP = /.+@(?:softbank\.ne\.jp|disney\.ne\.jp)/.freeze
     # メールのデフォルトのcharset
     MAIL_CHARSET = 'Shift_JIS'.freeze
     # テキスト部分の content-transfer-encoding
     MAIL_CONTENT_TRANSFER_ENCODING = '8bit'.freeze
 
+    TARGET_PARAMS = ['pos', 'geo', 'x-acr'].freeze
+
     # 製造番号を返す。無ければ +nil+ を返す。
     def serial_number
       @request.env['HTTP_USER_AGENT'] =~ /SN(.+?) /
@@ -36,7 +38,7 @@ module Jpmobile::Mobile
       l = Jpmobile::Position.new
       l.lat = ((Regexp.last_match(1) == 'N') ? 1 : -1) * Jpmobile::Position.dms2deg(Regexp.last_match(2), Regexp.last_match(3), Regexp.last_match(4))
       l.lon = ((Regexp.last_match(5) == 'E') ? 1 : -1) * Jpmobile::Position.dms2deg(Regexp.last_match(6), Regexp.last_match(7), Regexp.last_match(8))
-      l.options = params.select {|x, _| ['pos', 'geo', 'x-acr'].include?(x) }
+      l.options = params.select {|x, _| TARGET_PARAMS.include?(x) }
 
       @__position = l
     end

data/lib/jpmobile/mobile/vodafone.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # スーパクラスはSoftbank。
   class Vodafone < Softbank
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /^(Vodafone|Vemulator)/
+    USER_AGENT_REGEXP = /^(Vodafone|Vemulator)/.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@[dhtcrknsq]\.vodafone\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@[dhtcrknsq]\.vodafone\.ne\.jp/.freeze
 
     # cookieに対応しているか？
     def supports_cookie?

data/lib/jpmobile/mobile/willcom.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # Ddipocketのスーパクラス。
   class Willcom < AbstractMobile
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(WILLCOM}
+    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(WILLCOM}.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@((.+\.)?pdx\.ne\.jp|willcom\.com)/
+    MAIL_ADDRESS_REGEXP = /.+@((.+\.)?pdx\.ne\.jp|willcom\.com)/.freeze
 
     # 位置情報があれば Position のインスタンスを返す。無ければ +nil+ を返す。
     def position

data/lib/jpmobile/mobile/windows_phone.rb

@@ -4,6 +4,6 @@ module Jpmobile::Mobile
   # ==WindowsPhone
   class WindowsPhone < SmartPhone
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /Windows Phone/
+    USER_AGENT_REGEXP = /Windows Phone/.freeze
   end
 end

data/lib/jpmobile/position.rb

@@ -10,7 +10,7 @@ end
 module Jpmobile
   # 位置情報
   class Position
-    if Object.const_defined?('GeoKit')
+    if Object.const_defined?(:GeoKit)
       # GeoKitが読み込まれている場合はMappableにする
       include ::GeoKit::Mappable
       def self.acts_as_mappable
@@ -35,7 +35,7 @@ module Jpmobile
 
     # 度分秒を度に変換する。
     def self.dms2deg(d, m, s)
-      d.to_i + m.to_i.to_f / 60 + s.to_f / 3600
+      d.to_i + (m.to_i.to_f / 60) + (s.to_f / 3600)
     end
 
     # 日本測地系から世界測地系に変換する。

data/lib/jpmobile/rails.rb

@@ -2,6 +2,7 @@ ActiveSupport.on_load(:action_controller) do
   require 'jpmobile/docomo_guid'
   require 'jpmobile/filter'
   require 'jpmobile/helpers'
+  require 'jpmobile/method_less_action_support'
   require 'jpmobile/trans_sid'
   require 'jpmobile/hook_test_request'
   ActionDispatch::Request.prepend Jpmobile::Encoding

data/lib/jpmobile/template_details.rb

@@ -19,8 +19,8 @@ module Jpmobile
         requested.formats_idx[@format],
         requested.locale_idx[@locale],
         requested.variants_idx[@variant],
-        requested.handlers_idx[@handler],
         requested.mobile_idx[@mobile],
+        requested.handlers_idx[@handler],
       ]
     end
 

data/lib/jpmobile/version.rb

@@ -1,3 +1,3 @@
 module Jpmobile
-  VERSION = '7.0.0'.freeze
+  VERSION = '7.0.3'.freeze
 end

data/lib/tasks/jpmobile_tasks.rake

@@ -84,12 +84,10 @@ namespace :test do
     unless skip
       # for cookie_only option
       config_path = File.join(rails_root, 'config', 'initializers', 'session_store.rb')
-      File.open(config_path, 'w') do |file|
-        file.write <<-SESSION_CONFIG
+      File.write(config_path, <<-SESSION_CONFIG)
         Rails.application.config.session_store :active_record_store, :key => '_session_id'
         Rails.application.config.session_options = { :cookie_only => false }
-        SESSION_CONFIG
-      end
+      SESSION_CONFIG
     end
 
     unless skip

data/spec/unit/decorated_mail_spec.rb

@@ -14,7 +14,7 @@ describe 'decorated mails' do
     @mail.from      = 'ちはやふる <info@jpmobile-rails.org>'
     @mail.to        = 'むすめふさほせ <info+to@jpmobile-rails.org>'
 
-    @photo = File.open(File.join(__dir__, 'email-fixtures/photo.jpg')).read
+    @photo = File.read(File.join(__dir__, 'email-fixtures/photo.jpg'))
     @mail.attachments.inline['photo.jpg'] = @photo
     @inline_url = @mail.attachments['photo.jpg'].url
   end

data/spec/unit/mail_spec.rb

@@ -285,7 +285,7 @@ describe 'Jpmobile::Mail' do
       @mobile = Jpmobile::Mobile::AbstractMobile.new(nil, nil)
       @mail.mobile = @mobile
       @mail.to = 'むすめふさほせ <info+to@jpmobile-rails.org>'
-      @photo = File.open(File.join(__dir__, 'email-fixtures/photo.jpg')).read
+      @photo = File.read(File.join(__dir__, 'email-fixtures/photo.jpg'))
     end
 
     it 'should encodes itself successfully' do
@@ -318,7 +318,7 @@ describe 'Jpmobile::Mail' do
       @mobile = Jpmobile::Mobile::AbstractMobile.new(nil, nil)
       @mail.mobile = @mobile
       @mail.to = 'むすめふさほせ <info+to@jpmobile-rails.org>'
-      @photo = File.open(File.join(__dir__, 'email-fixtures/photo.jpg')).read
+      @photo = File.read(File.join(__dir__, 'email-fixtures/photo.jpg'))
     end
 
     it 'wave dash converting correctly' do

data/spec/unit/receive_mail_spec.rb

@@ -12,7 +12,7 @@ describe 'Jpmobile::Mail#receive' do
 
   describe 'PC mail' do
     before(:each) do
-      @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/pc-mail-single.eml')).read)
+      @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/pc-mail-single.eml')))
     end
 
     it 'subject should be parsed correctly' do
@@ -49,7 +49,7 @@ describe 'Jpmobile::Mail#receive' do
   describe 'multipart' do
     describe 'PC mail' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/pc-mail-multi.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/pc-mail-multi.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -68,7 +68,7 @@ describe 'Jpmobile::Mail#receive' do
 
     describe 'PC mail without subject' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/pc-mail-attached-without-subject.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/pc-mail-attached-without-subject.eml')))
       end
 
       it 'body should be parsed correctly' do
@@ -84,7 +84,7 @@ describe 'Jpmobile::Mail#receive' do
     describe 'Docomo' do
       context 'with sjis decomail' do
         before(:each) do
-          @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/docomo-gmail-sjis.eml')).read)
+          @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/docomo-gmail-sjis.eml')))
         end
 
         it 'subject should be parsed correctly' do
@@ -111,7 +111,7 @@ describe 'Jpmobile::Mail#receive' do
 
       context 'with jis decomail' do
         before(:each) do
-          @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/docomo-decomail.eml')).read)
+          @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/docomo-decomail.eml')))
         end
 
         it 'does not contain charset within multipart Content-Type' do
@@ -129,7 +129,7 @@ describe 'Jpmobile::Mail#receive' do
 
     describe 'Au' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/au-decomail.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/au-decomail.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -151,7 +151,7 @@ describe 'Jpmobile::Mail#receive' do
     describe 'Softbank' do
       context 'Shift_JIS' do
         before(:each) do
-          @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/softbank-gmail-sjis.eml')).read)
+          @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/softbank-gmail-sjis.eml')))
         end
 
         it 'subject should be parsed correctly' do
@@ -170,7 +170,7 @@ describe 'Jpmobile::Mail#receive' do
 
       context 'UTF-8' do
         before(:each) do
-          @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/softbank-gmail-utf8.eml')).read)
+          @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/softbank-gmail-utf8.eml')))
         end
 
         it 'subject should be parsed correctly' do
@@ -190,7 +190,7 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'bounced mail' do
       it 'should parse sub-part charset correctly' do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/bounce_with_utf8_part.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/bounce_with_utf8_part.eml')))
         expect(@mail.parts.first.charset).to match(/iso-2022-jp/i)
         expect(@mail.parts.last.charset).to  match(/utf-8/i)
       end
@@ -199,7 +199,7 @@ describe 'Jpmobile::Mail#receive' do
 
   describe 'Docomo' do
     before(:each) do
-      @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/docomo-emoji.eml')).read)
+      @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/docomo-emoji.eml')))
     end
 
     it 'subject should be parsed correctly' do
@@ -234,7 +234,7 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'JIS mail' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/docomo-jis.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/docomo-jis.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -250,7 +250,7 @@ describe 'Jpmobile::Mail#receive' do
   describe 'Au' do
     context 'au-emoji.eml' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/au-emoji.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/au-emoji.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -286,36 +286,36 @@ describe 'Jpmobile::Mail#receive' do
 
     it 'should not be raised when parsing incoming email #41' do
       expect {
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/au-email.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/au-email.eml')))
       }.not_to raise_error
     end
 
     it 'should not be raised when parsing incoming email #45' do
       expect {
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/au-decomail2.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/au-decomail2.eml')))
       }.not_to raise_error
     end
 
     it 'should not be raised when parsing incoming email - include kigou' do
       expect {
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/au-kigou.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/au-kigou.eml')))
       }.not_to raise_error
     end
 
     context 'From au iPhone' do
       it 'charset should be UTF-8' do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/iphone-message.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/iphone-message.eml')))
         expect(@mail.mobile).to be_a(Jpmobile::Mobile::Au)
         expect(@mail.charset).to match(/utf-8/i)
       end
 
       it 'should be encoded correctly' do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/iphone-message.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/iphone-message.eml')))
         expect(@mail.encoded).to match(Regexp.escape("%[\e$B1`;yL>\e(B]%\e$B$N\e(B%[\e$BJ]8n<TL>\e(B]%"))
       end
 
       it 'should decode cp932-encoded mail correctly' do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-cp932.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-cp932.eml')))
         expect(@mail.subject).to eq 'Re: 【NKSC】test'
         expect(@mail.body.to_s).to eq 'テストです。㈱'
       end
@@ -323,20 +323,20 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'From iPad' do
       it 'charset should be UTF-8' do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-mail3.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-mail3.eml')))
         expect(@mail.mobile).to be_a(Jpmobile::Mobile::AbstractMobile)
         expect(@mail.charset).to match(/utf-8/i)
       end
 
       it 'should be encoded correctly' do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-mail3.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-mail3.eml')))
         expect(@mail.encoded).to match(/BK\\J82~9T\$J\$7!2#5#1#2J8;z!2/)
       end
     end
 
     it 'should not raise when parsing attached email' do
       expect {
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/au-attached.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/au-attached.eml')))
         expect(@mail.encoded).to match('/9j/4AAQSkZJRgABAgAAZABkAAD/7AARRHVja3kAAQAEAAAAPQAA')
       }.not_to raise_error
     end
@@ -344,7 +344,7 @@ describe 'Jpmobile::Mail#receive' do
 
   describe 'Softbank' do
     before(:each) do
-      @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/softbank-emoji.eml')).read)
+      @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/softbank-emoji.eml')))
     end
 
     it 'subject should be parsed correctly' do
@@ -379,7 +379,7 @@ describe 'Jpmobile::Mail#receive' do
 
     describe 'blank-mail' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/softbank-blank.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/softbank-blank.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -395,7 +395,7 @@ describe 'Jpmobile::Mail#receive' do
   describe 'iPhone' do
     context 'JIS mail' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-jis.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-jis.eml')))
       end
 
       it 'body should be parsed correctly' do
@@ -405,7 +405,7 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'when the mail contains UTF-8 emojis' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-unicode-emoji.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-unicode-emoji.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -419,7 +419,7 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'when the mail contains circled-numbers' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, 'email-fixtures/iphone-circled-numbers-in-jis.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, 'email-fixtures/iphone-circled-numbers-in-jis.eml')))
       end
 
       it 'subject should be parsed correctly' do
@@ -435,7 +435,7 @@ describe 'Jpmobile::Mail#receive' do
   describe 'bounced mail' do
     context 'has jp address' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/bounced-jp.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/bounced-jp.eml')))
       end
 
       it 'mobile should abstract mobile' do
@@ -447,7 +447,7 @@ describe 'Jpmobile::Mail#receive' do
   describe 'non-Japanese mail' do
     context 'us-ascii' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/non-jp.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/non-jp.eml')))
       end
 
       it 'mobile should be nil' do
@@ -458,7 +458,7 @@ describe 'Jpmobile::Mail#receive' do
 
     context 'no From header' do
       before(:each) do
-        @mail = Mail.new(File.open(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/no-from.eml')).read)
+        @mail = Mail.new(File.read(File.join(__dir__, '../../test/rails/overrides/spec/fixtures/files/mobile_mailer/no-from.eml')))
       end
 
       it 'mobile should be nil' do

data/test/rails/overrides/app/controllers/method_less_action_support_controller.rb

@@ -0,0 +1,3 @@
+class MethodLessActionSupportController < ApplicationController
+  include Jpmobile::MethodLessActionSupport
+end

data/test/rails/overrides/app/mailers/decorated_mailer.rb

@@ -3,7 +3,7 @@ class DecoratedMailer < Jpmobile::Mailer::Base
   default to: 'info@jp.mobile'
 
   def deco_mail(to_mail)
-    attachments.inline['photo.jpg'] = File.open(File.join(Rails.root, 'spec/fixtures/files/mobile_mailer/photo.jpg')).read
+    attachments.inline['photo.jpg'] = File.read(File.join(Rails.root, 'spec/fixtures/files/mobile_mailer/photo.jpg'))
 
     mail(to: to_mail, subject: '題名', decorated: true)
   end

data/test/rails/overrides/config/routes.rb

@@ -1,3 +1,4 @@
+# rubocop:disable Performance/CollectionLiteralInLoop
 RailsRoot::Application.routes.draw do
   resources :users
   namespace :admin do
@@ -101,4 +102,7 @@ RailsRoot::Application.routes.draw do
       get "#{c}/#{a}", to: "#{c}##{a}"
     end
   end
+
+  get 'method_less_action_support', to: 'method_less_action_support#index'
 end
+# rubocop:enable Performance/CollectionLiteralInLoop

data/test/rails/overrides/spec/requests/method_less_action_support_spec.rb

@@ -0,0 +1,28 @@
+require 'rails_helper'
+
+describe 'Method-less mobile template only action', type: :request do
+  subject do
+    get '/method_less_action_support', headers: headers
+  end
+
+  let(:headers) do
+    {}
+  end
+
+  context 'when accessed with mobile User-Agent' do
+    before do
+      headers['User-Agent'] = 'Mozilla/5.0 (iPhone; U; CPU iPhone OS 3_0 like Mac OS X; ja-jp) AppleWebKit/528.18 (KHTML, like Gecko) Version/4.0 Mobile/7A341 Safari/528.16'
+    end
+
+    it 'successfully renders mobile template' do
+      subject
+      expect(response).to have_http_status(200)
+    end
+  end
+
+  context 'when accessed with non-mobile User-Agent' do
+    it 'raises AbstractController::ActionNotFound' do
+      expect { subject }.to raise_error(AbstractController::ActionNotFound)
+    end
+  end
+end