jp_prefecture 0.3.0 → 0.3.1

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.3.1 (June 11, 2013) ##
+
+* name で渡した文字列が変更されるバグを修正
+
 ## 0.3.0 (June 11, 2013) ##
 
 * 都道府県名(英語表記含む)から都道府県を検索できるようにした

data/lib/jp_prefecture/prefecture.rb

@@ -94,9 +94,10 @@ module JpPrefecture
 
     # 名前から都道府県コードを検索
     def self.find_code_by_name(name)
-      name.capitalize!
+      result = PREFECTURE_CODE_NAME.select { |_, v|
+        v.has_value?(name.capitalize)
+      }.first
 
-      result = PREFECTURE_CODE_NAME.select { |_, v| v.has_value?(name) }.first
       return if result.nil?
 
       result[0]

data/lib/jp_prefecture/version.rb

@@ -1,3 +1,3 @@
 module JpPrefecture
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/spec/prefecture_spec.rb

@@ -10,47 +10,75 @@ describe JpPrefecture::Prefecture do
   end
 
   describe '.find' do
-    shared_examples "都道府県が見つかる" do |arg|
-      let(:pref) { JpPrefecture::Prefecture.find(arg) }
-      it { pref.code.should eq 1 }
-      it { pref.name.should eq '北海道' }
-      it { pref.name_e.should eq 'Hokkaido' }
-    end
+    describe '検索結果について' do
+      shared_examples "都道府県が見つかる" do |arg|
+        let(:pref) { JpPrefecture::Prefecture.find(arg) }
+        it { pref.code.should eq 1 }
+        it { pref.name.should eq '北海道' }
+        it { pref.name_e.should eq 'Hokkaido' }
+      end
 
-    shared_examples '都道府県が見つからない' do |arg|
-      let(:pref) { JpPrefecture::Prefecture.find(arg) }
-      it { pref.should be_nil }
-    end
+      shared_examples '都道府県が見つからない' do |arg|
+        let(:pref) { JpPrefecture::Prefecture.find(arg) }
+        it { pref.should be_nil }
+      end
 
-    describe '都道府県コード' do
-      it_behaves_like "都道府県が見つかる", 1
-      it_behaves_like "都道府県が見つからない", 99
-      it_behaves_like "都道府県が見つかる", "1"
-      it_behaves_like "都道府県が見つかる", "01"
-      it_behaves_like "都道府県が見つからない", "99"
-    end
+      describe '都道府県コード' do
+        it_behaves_like "都道府県が見つかる", 1
+        it_behaves_like "都道府県が見つからない", 99
+        it_behaves_like "都道府県が見つかる", "1"
+        it_behaves_like "都道府県が見つかる", "01"
+        it_behaves_like "都道府県が見つからない", "99"
+      end
 
-    describe '都道府県コード(キーワード引数)' do
-      it_behaves_like "都道府県が見つかる", code: 1
-      it_behaves_like "都道府県が見つからない", code: 99
-      it_behaves_like "都道府県が見つかる", code: "1"
-      it_behaves_like "都道府県が見つかる", code: "01"
-      it_behaves_like "都道府県が見つからない", code: "99"
-    end
+      describe '都道府県コード(キーワード引数)' do
+        it_behaves_like "都道府県が見つかる", code: 1
+        it_behaves_like "都道府県が見つからない", code: 99
+        it_behaves_like "都道府県が見つかる", code: "1"
+        it_behaves_like "都道府県が見つかる", code: "01"
+        it_behaves_like "都道府県が見つからない", code: "99"
+      end
 
-    describe '都道府県名' do
-      it_behaves_like "都道府県が見つかる", name: "北海道"
-      it_behaves_like "都道府県が見つからない", name: "うどん県"
-    end
+      describe '都道府県名' do
+        it_behaves_like "都道府県が見つかる", name: "北海道"
+        it_behaves_like "都道府県が見つからない", name: "うどん県"
+      end
+
+      describe '都道府県名(英語表記)' do
+        it_behaves_like "都道府県が見つかる", name: "Hokkaido"
+        it_behaves_like "都道府県が見つからない", name: "Udon"
+      end
 
-    describe '都道府県名(英語表記)' do
-      it_behaves_like "都道府県が見つかる", name: "Hokkaido"
-      it_behaves_like "都道府県が見つからない", name: "Udon"
+      describe '都道府県名(英語表記-小文字)' do
+        it_behaves_like "都道府県が見つかる", name: "hokkaido"
+        it_behaves_like "都道府県が見つからない", name: "udon"
+      end
     end
 
-    describe '都道府県名(英語表記-小文字)' do
-      it_behaves_like "都道府県が見つかる", name: "hokkaido"
-      it_behaves_like "都道府県が見つからない", name: "udon"
+    describe '渡した変数について' do
+      context 'string の場合' do
+        it '値が変更されないこと' do
+          code = '1'
+          JpPrefecture::Prefecture.find(code)
+          code.should eq '1'
+        end
+      end
+
+      context 'code が string の場合' do
+        it '値が変更されないこと' do
+          code = '1'
+          JpPrefecture::Prefecture.find(code: code)
+          code.should eq '1'
+        end
+      end
+
+      context 'name の場合' do
+        it '値が変更されないこと' do
+          name = 'hokkaido'
+          JpPrefecture::Prefecture.find(name: name)
+          name.should eq 'hokkaido'
+        end
+      end
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jp_prefecture
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-10 00:00:00.000000000 Z
+date: 2013-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake