jpmobile 5.0.0 → 5.0.1

data/spec/unit/spec_helper.rb

@@ -4,20 +4,20 @@ require 'rspec/its'
 $LOAD_PATH.unshift(File.expand_path(File.join(File.dirname(__FILE__), '..', '..', 'lib')))
 require 'jpmobile'
 begin
-  require File.dirname(__FILE__)+'/../../vendor/jpmobile-ipaddresses/lib/jpmobile-ipaddresses'
+  require File.dirname(__FILE__) + '/../../vendor/jpmobile-ipaddresses/lib/jpmobile-ipaddresses'
 rescue LoadError
-  puts "IP Address test requires jpmobile-ipaddresses module"
+  puts 'IP Address test requires jpmobile-ipaddresses module'
 end
 begin
-  require File.dirname(__FILE__)+'/../../vendor/jpmobile-terminfo/lib/jpmobile-terminfo'
+  require File.dirname(__FILE__) + '/../../vendor/jpmobile-terminfo/lib/jpmobile-terminfo'
 rescue LoadError
-  puts "Terminal display information test requires jpmobile-terminfo module"
+  puts 'Terminal display information test requires jpmobile-terminfo module'
 end
 
 RSpec.configure do |config|
-  config.filter_run :focus => true
+  config.filter_run focus: true
   config.run_all_when_everything_filtered = true
   config.color = true
-  config.filter_run_excluding :broken => true
+  config.filter_run_excluding broken: true
   # config.full_backtrace = true
 end

data/spec/unit/util_spec.rb

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
 require 'stringio'
 require 'nkf'
@@ -7,62 +6,78 @@ describe Jpmobile::Util do
   include Jpmobile::Util
 
   describe 'utf8_to_sjis' do
-    it "utf8_to_sjis で変換できない文字列が含んでいた場合?に変換される" do
-      expect(utf8_to_sjis("اللغة العربية")).to eq(sjis("????? ???????"))
+    it 'utf8_to_sjis で変換できない文字列が含んでいた場合?に変換される' do
+      expect(utf8_to_sjis('اللغة العربية')).to eq(sjis('????? ???????'))
     end
 
-    it "utf8_to_sjis ですべての改行コードが CRLF に変更されること" do
+    it 'utf8_to_sjis ですべての改行コードが CRLF に変更されること' do
       expect(utf8_to_sjis("UTF8\rSAMPLE\nTEXT\r\n")).to eq(sjis("UTF8\r\nSAMPLE\r\nTEXT\r\n"))
     end
 
-    it "U+FFE3が0x8150に変換されること" do
-      expect(utf8_to_sjis([0xffe3].pack("U"))).to eq(sjis("\x81\x50"))
+    it 'U+FFE3が0x8150に変換されること' do
+      expect(utf8_to_sjis([0xffe3].pack('U'))).to eq(sjis("\x81\x50"))
     end
 
-    it "U+203Eが0x8150に変換されること" do
-      expect(utf8_to_sjis([0x203e].pack("U"))).to eq(sjis("\x81\x50"))
+    it 'U+203Eが0x8150に変換されること' do
+      expect(utf8_to_sjis([0x203e].pack('U'))).to eq(sjis("\x81\x50"))
     end
 
-    it "U+2014が0x815Cに変換されること" do
-      expect(utf8_to_sjis([0x2014].pack("U"))).to eq(sjis("\x81\x5C"))
+    it 'U+2014が0x815Cに変換されること' do
+      expect(utf8_to_sjis([0x2014].pack('U'))).to eq(sjis("\x81\x5C"))
     end
 
-    it "U+2212が0x817Cに変換されること" do
-      expect(utf8_to_sjis([0x2212].pack("U"))).to eq(sjis("\x81\x7C"))
+    it 'U+2212が0x817Cに変換されること' do
+      expect(utf8_to_sjis([0x2212].pack('U'))).to eq(sjis("\x81\x7C"))
     end
 
-    it "utf8_to_sjis で変換できない文字列が含んでいた場合?に変換される" do
-      expect(utf8_to_jis("اللغة العربية")).to eq(jis("????? ???????"))
+    it 'utf8_to_sjis で変換できない文字列が含んでいた場合?に変換される' do
+      expect(utf8_to_jis('اللغة العربية')).to eq(jis('????? ???????'))
+    end
+
+    it 'frozenでも通過すること' do
+      expect { utf8_to_jis('漢字'.freeze) }.not_to raise_error
     end
   end
 
   describe 'sjis_to_utf8' do
-    it "0x8150がU+FFE3に変換されること" do
-      expect(sjis_to_utf8(sjis("\x81\x50"))).to eq([0xffe3].pack("U"))
+    it '0x8150がU+FFE3に変換されること' do
+      expect(sjis_to_utf8(sjis("\x81\x50"))).to eq([0xffe3].pack('U'))
     end
 
-    it "sjis_to_utf8 ですべての改行コードが LF に変更されること" do
+    it 'sjis_to_utf8 ですべての改行コードが LF に変更されること' do
       expect(sjis_to_utf8("SJIS\rSAMPLE\nTEXT\r\n")).to eq(utf8("SJIS\nSAMPLE\nTEXT\n"))
     end
+
+    it 'frozenでも通過すること' do
+      expect { sjis_to_utf8('漢字'.encode('SJIS').freeze) }.not_to raise_error
+    end
   end
 
   describe 'utf8_to_jis' do
-    it "utf8_to_jis ですべての改行コードが CRLF に変更されること" do
+    it 'utf8_to_jis ですべての改行コードが CRLF に変更されること' do
       expect(utf8_to_jis("UTF8\rSAMPLE\nTEXT\r\n")).to eq(jis("UTF8\r\nSAMPLE\r\nTEXT\r\n"))
     end
+
+    it 'frozenでも通過すること' do
+      expect { utf8_to_jis('漢字'.freeze) }.not_to raise_error
+    end
   end
 
   describe 'jis_string_regexp' do
-    it "jis_string_regexpでISO-2022-JPの文字列がマッチすること" do
-      expect(jis_string_regexp.match(ascii_8bit(utf8_to_jis("abcしからずんばこじをえずdef")))).not_to be_nil
-      expect(jis_to_utf8(jis("\x1b\x24\x42#{$1}\x1b\x28\x42"))).to eq("しからずんばこじをえず")
+    it 'jis_string_regexpでISO-2022-JPの文字列がマッチすること' do
+      expect(jis_string_regexp.match(ascii_8bit(utf8_to_jis('abcしからずんばこじをえずdef')))).not_to be_nil
+      expect(jis_to_utf8(jis("\x1b\x24\x42#{Regexp.last_match(1)}\x1b\x28\x42"))).to eq('しからずんばこじをえず')
     end
   end
 
   describe 'jis_to_utf8' do
-    it "jis_to_utf8 ですべての改行コードが LF に変更されること" do
+    it 'jis_to_utf8 ですべての改行コードが LF に変更されること' do
       expect(jis_to_utf8("JIS\rSAMPLE\nTEXT\r\n")).to eq(utf8("JIS\nSAMPLE\nTEXT\n"))
     end
+
+    it 'frozenでも通過すること' do
+      expect { jis_to_utf8('漢字'.encode('ISO-2022-JP').freeze) }.not_to raise_error
+    end
   end
 
   describe '#force_encode' do
@@ -71,59 +86,123 @@ describe Jpmobile::Util do
     end
 
     it 'does not enter infinite loop on retry' do
-      expect{ force_encode("\e\x28\x49\x9a\x43\x3D\x44\e\x28\x42".force_encoding('ISO-2022-JP'), 'iso-2022-jp', 'UTF-8') }.to raise_error ::Encoding::InvalidByteSequenceError
+      expect { force_encode("\e\x28\x49\x9a\x43\x3D\x44\e\x28\x42".force_encoding('ISO-2022-JP'), 'iso-2022-jp', 'UTF-8') }.to raise_error ::Encoding::InvalidByteSequenceError
+    end
+
+    it 'frozenでも通過すること' do
+      expect { force_encode('漢字'.encode('ISO-2022-JP').freeze, 'iso-2022-jp', 'UTF-8') }.not_to raise_error
+    end
+  end
+
+  describe '#fold_text' do
+    it 'UTF-8の日本語文字列が指定文字数で折り返された配列で返ること' do
+      expect(fold_text('長い日本語の題名で折り返されるかようにするには事前に分割していないとダメなことがわかりましたよ', 15)).
+        to eq(
+          %w[
+            長い日本語の題名で折り返される
+            かようにするには事前に分割して
+            いないとダメなことがわかりまし
+            たよ
+          ]
+        )
+    end
+
+    it 'UTF-8の短い文字列は折り返されないこと' do
+      expect(fold_text('短い', 15)).to eq(['短い'])
+    end
+
+    it 'frozenでも通過すること' do
+      expect { fold_text(('漢字' * 10).freeze, 15) }.not_to raise_error
     end
   end
 
-  it "fold_textでUTF-8の日本語文字列が指定文字数で折り返された配列で返ること" do
-    expect(fold_text('長い日本語の題名で折り返されるかようにするには事前に分割していないとダメなことがわかりましたよ', 15)).to eq([
-      '長い日本語の題名で折り返される',
-      'かようにするには事前に分割して',
-      'いないとダメなことがわかりまし',
-      'たよ'
-    ])
+  describe '#split_text' do
+    it 'UTF-8の日本語文字列が指定文字数で2つに分割されること' do
+      expect(split_text('長い日本語の題名で折り返されるかようにするには事前に分割していないとダメなことがわかりましたよ', 15)).
+        to eq(
+          %w[
+            長い日本語の題名で折り返される
+            かようにするには事前に分割していないとダメなことがわかりましたよ
+          ]
+        )
+    end
+
+    it 'nilかblankの場合はnilが返ること' do
+      expect(split_text('', 15)).to be_nil
+      expect(split_text(nil, 15)).to be_nil
+    end
+
+    it 'frozenでも通過すること' do
+      expect { split_text(('漢字' * 10).freeze, 15) }.not_to raise_error
+    end
   end
 
-  it "fold_textでUTF-8の短い文字列は折り返されないこと" do
-    expect(fold_text('短い', 15)).to eq(['短い'])
+  describe '#sjis' do
+    it 'frozenでも通過すること' do
+      expect { sjis('漢字'.freeze) }.not_to raise_error
+    end
   end
 
-  it "split_textでUTF-8の日本語文字列が指定文字数で2つに分割されること" do
-    expect(split_text('長い日本語の題名で折り返されるかようにするには事前に分割していないとダメなことがわかりましたよ', 15)).to eq([
-      '長い日本語の題名で折り返される',
-      'かようにするには事前に分割していないとダメなことがわかりましたよ'
-    ])
+  describe '#utf8' do
+    it 'frozenでも通過すること' do
+      expect { utf8('漢字'.freeze) }.not_to raise_error
+    end
   end
 
-  it "split_textでnilかblankの場合はnilが返ること" do
-    expect(split_text('', 15)).to be_nil
-    expect(split_text(nil, 15)).to be_nil
+  describe '#jis' do
+    it 'frozenでも通過すること' do
+      expect { jis('漢字'.freeze) }.not_to raise_error
+    end
+  end
+
+  describe '#jis_win' do
+    it 'frozenでも通過すること' do
+      expect { jis_win('漢字'.freeze) }.not_to raise_error
+    end
+  end
+
+  describe '#ascii_8bit' do
+    it 'frozenでも通過すること' do
+      expect { ascii_8bit('漢字'.freeze) }.not_to raise_error
+    end
+  end
+
+  describe '#ascii_compatible!' do
+    it 'frozenでも通過すること' do
+      expect { ascii_compatible!('漢字'.freeze) }.not_to raise_error
+    end
   end
 
-  it "全角チルダがjisに適切に変換されること" do
+  it '全角チルダがjisに適切に変換されること' do
     expect(ascii_8bit(utf8_to_jis("\xef\xbd\x9e"))).to eq(ascii_8bit("\x1b\x24\x42\x21\x41\x1b\x28\x42"))
-    expect(ascii_8bit(encode("\xef\xbd\x9e", "ISO-2022-JP"))).to eq(ascii_8bit("\x1b\x24\x42\x21\x41\x1b\x28\x42"))
+    expect(ascii_8bit(encode("\xef\xbd\x9e", 'ISO-2022-JP'))).to eq(ascii_8bit("\x1b\x24\x42\x21\x41\x1b\x28\x42"))
   end
 
   describe 'invert_table' do
     it 'pickups the first(least) of duplicated values when inverting hash table' do
-      hash = invert_table({
+      hash = invert_table(
+        {
           0xFE009 => 0xE63E,
           0xFE00A => 0xE63E,
           0xFE00B => 0x3013,
           0xFE010 => 0xE6B3,
-          0xFE038 => 0xE73F})
+          0xFE038 => 0xE73F,
+        }
+      )
       expect(hash[0xE63E]).to eq(0xFE009)
       expect(hash[0x3013]).to eq(0xFE00B)
     end
 
     it 'should not raise error when hash has Array keys' do
-      hash = invert_table({
+      hash = invert_table(
+        {
           [0x1F1E8, 0x1F1F3] => 0x3013,
           0x1F1FF            => 0x3013,
           0x1F526            => 0xE6FB,
           [0x0023, 0x20E3]   => 0xE6E0,
-          0x1F354            => 0xE673})
+          0x1F354            => 0xE673,
+        }
+      )
       expect(hash[0x3013]).to eq([0x1F1E8, 0x1F1F3])
       expect(hash[0xE6FB]).to eq(0x1F526)
     end

data/spec/unit/valid_ip_spec.rb

@@ -2,47 +2,47 @@ require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
 
 describe 'Jpmobile::Mobile::valid_ip?' do
   [
-    [ '210.153.84.1'    , :Docomo    ],
-    [ '210.153.84.254'  , :Docomo    ],
-    [ '210.230.128.225' , :Au        ],
-    [ '210.230.128.238' , :Au        ],
-    [ '123.108.237.1'   , :Softbank  ],
-    [ '123.108.237.31' , :Softbank  ],
-    [ '123.108.237.1'   , :Vodafone  ],
-    [ '123.108.237.31' , :Vodafone  ],
-    [ '61.198.142.1'    , :Willcom   ],
-    [ '61.198.142.254'  , :Willcom   ],
-    [ '61.198.142.1'    , :Ddipocket ],
-    [ '61.198.142.254'  , :Ddipocket ],
-    [ '117.55.1.224'    , :Emobile   ],
-    [ '117.55.1.254'    , :Emobile   ],
+    ['210.153.84.1', :Docomo],
+    ['210.153.84.254', :Docomo],
+    ['210.230.128.225', :Au],
+    ['210.230.128.238', :Au],
+    ['123.108.237.1', :Softbank],
+    ['123.108.237.31', :Softbank],
+    ['123.108.237.1', :Vodafone],
+    ['123.108.237.31', :Vodafone],
+    ['61.198.142.1', :Willcom],
+    ['61.198.142.254', :Willcom],
+    ['61.198.142.1', :Ddipocket],
+    ['61.198.142.254', :Ddipocket],
+    ['117.55.1.224', :Emobile],
+    ['117.55.1.254', :Emobile],
   ].each do |remote_ip, carrier|
-    it "should be return true if #{remote_ip} is in #{:carrier} address" do
+    it "should be return true if #{remote_ip} is in #{carrier} address" do
       expect(Jpmobile::Mobile.const_get(carrier).valid_ip?(remote_ip)).to eq(true)
     end
   end
 
   [
-    [ '127.0.0.1'       , :Docomo    ],
-    [ '210.153.83.1'    , :Docomo    ],
-    [ '210.153.83.254'  , :Docomo    ],
-    [ '127.0.0.1'       , :Au        ],
-    [ '210.169.41.1'    , :Au        ],
-    [ '210.169.41.254'  , :Au        ],
-    [ '127.0.0.1'       , :Softbank  ],
-    [ '123.108.238.1'   , :Softbank  ],
-    [ '123.108.238.254' , :Softbank  ],
-    [ '127.0.0.1'       , :Vodafone  ],
-    [ '123.108.238.1'   , :Vodafone  ],
-    [ '123.108.238.254' , :Vodafone  ],
-    [ '127.0.0.1'       , :Willcom   ],
-    [ '61.198.144.1'    , :Willcom   ],
-    [ '61.198.144.254'  , :Willcom   ],
-    [ '127.0.0.1'       , :Ddipocket ],
-    [ '61.198.144.1'    , :Ddipocket ],
-    [ '61.198.144.254'  , :Ddipocket ],
-    [ '127.0.0.1'       , :Emobile   ],
-    [ '117.55.1.223'    , :Emobile   ],
+    ['127.0.0.1', :Docomo],
+    ['210.153.83.1', :Docomo],
+    ['210.153.83.254', :Docomo],
+    ['127.0.0.1', :Au],
+    ['210.169.41.1', :Au],
+    ['210.169.41.254', :Au],
+    ['127.0.0.1', :Softbank],
+    ['123.108.238.1', :Softbank],
+    ['123.108.238.254', :Softbank],
+    ['127.0.0.1', :Vodafone],
+    ['123.108.238.1', :Vodafone],
+    ['123.108.238.254', :Vodafone],
+    ['127.0.0.1', :Willcom],
+    ['61.198.144.1', :Willcom],
+    ['61.198.144.254', :Willcom],
+    ['127.0.0.1', :Ddipocket],
+    ['61.198.144.1', :Ddipocket],
+    ['61.198.144.254', :Ddipocket],
+    ['127.0.0.1', :Emobile],
+    ['117.55.1.223', :Emobile],
   ].each do |remote_ip, carrier|
     it 'should not be return true if #{:remote_ip} is in #{carrier} address' do
       expect(Jpmobile::Mobile.const_get(carrier).valid_ip?(remote_ip)).not_to eq(true)

data/spec/unit/variants_spec.rb

@@ -1,15 +1,14 @@
-# -*- coding: utf-8 -*-
 require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
 require 'mail'
 require 'jpmobile/mail'
 
 describe Jpmobile::Mobile::AbstractMobile do
-  subject{ Jpmobile::Mobile::AbstractMobile.new(nil, nil) }
+  subject { Jpmobile::Mobile::AbstractMobile.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to be === ['mobile'] }
+      it { is_expected.to eq(['mobile']) }
     end
   end
 
@@ -22,12 +21,12 @@ describe Jpmobile::Mobile::AbstractMobile do
 end
 
 describe Jpmobile::Mobile::Docomo do
-  subject{ Jpmobile::Mobile::Docomo.new(nil, nil) }
+  subject { Jpmobile::Mobile::Docomo.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['mobile_docomo', 'mobile']) }
+      it { is_expected.to eq(%w[mobile_docomo mobile]) }
     end
   end
 
@@ -39,12 +38,12 @@ describe Jpmobile::Mobile::Docomo do
 end
 
 describe Jpmobile::Mobile::Au do
-  subject{ Jpmobile::Mobile::Au.new(nil, nil) }
+  subject { Jpmobile::Mobile::Au.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['mobile_au', 'mobile']) }
+      it { is_expected.to eq(%w[mobile_au mobile]) }
     end
   end
 
@@ -56,12 +55,12 @@ describe Jpmobile::Mobile::Au do
 end
 
 describe Jpmobile::Mobile::Softbank do
-  subject{ Jpmobile::Mobile::Softbank.new(nil, nil) }
+  subject { Jpmobile::Mobile::Softbank.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['mobile_softbank', 'mobile']) }
+      it { is_expected.to eq(%w[mobile_softbank mobile]) }
     end
   end
 
@@ -73,12 +72,12 @@ describe Jpmobile::Mobile::Softbank do
 end
 
 describe Jpmobile::Mobile::Android do
-  subject{ Jpmobile::Mobile::Android.new(nil, nil) }
+  subject { Jpmobile::Mobile::Android.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['smart_phone_android', 'smart_phone']) }
+      it { is_expected.to eq(%w[smart_phone_android smart_phone]) }
     end
   end
 
@@ -90,12 +89,12 @@ describe Jpmobile::Mobile::Android do
 end
 
 describe Jpmobile::Mobile::AndroidTablet do
-  subject{ Jpmobile::Mobile::AndroidTablet.new(nil, nil) }
+  subject { Jpmobile::Mobile::AndroidTablet.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['tablet_android_tablet', 'tablet', 'smart_phone']) }
+      it { is_expected.to eq(%w[tablet_android_tablet tablet smart_phone]) }
     end
   end
 
@@ -107,12 +106,12 @@ describe Jpmobile::Mobile::AndroidTablet do
 end
 
 describe Jpmobile::Mobile::Iphone do
-  subject{ Jpmobile::Mobile::Iphone.new(nil, nil) }
+  subject { Jpmobile::Mobile::Iphone.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['smart_phone_iphone', 'smart_phone']) }
+      it { is_expected.to eq(%w[smart_phone_iphone smart_phone]) }
     end
   end
 
@@ -124,12 +123,12 @@ describe Jpmobile::Mobile::Iphone do
 end
 
 describe Jpmobile::Mobile::Ipad do
-  subject{ Jpmobile::Mobile::Ipad.new(nil, nil) }
+  subject { Jpmobile::Mobile::Ipad.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['tablet_ipad', 'tablet', 'smart_phone']) }
+      it { is_expected.to eq(%w[tablet_ipad tablet smart_phone]) }
     end
   end
 
@@ -141,12 +140,12 @@ describe Jpmobile::Mobile::Ipad do
 end
 
 describe Jpmobile::Mobile::WindowsPhone do
-  subject{ Jpmobile::Mobile::WindowsPhone.new(nil, nil) }
+  subject { Jpmobile::Mobile::WindowsPhone.new(nil, nil) }
 
   describe '#variants' do
     describe '#variants' do
       subject { super().variants }
-      it { is_expected.to eq(['smart_phone_windows_phone', 'smart_phone']) }
+      it { is_expected.to eq(%w[smart_phone_windows_phone smart_phone]) }
     end
   end