icu_tournament 1.2.2 → 1.2.3

data/lib/icu_tournament/federation.rb

@@ -56,7 +56,7 @@ module ICU
   #
   #   ICU::Federation.menu(:none => 'None')              # => [['None', ''], ['Afghanistan', 'AFG], ...]
   #
-  # The "None" option's code is the empty string and it come above the "top" option if both are specified.
+  # The "None" option's code is the empty string and it comes above the "top" option if both are specified.
   #
   class Federation
     attr_reader :code, :name
@@ -68,7 +68,7 @@ module ICU
       return nil unless str
       str = str.to_s
       return nil if str.length < 3
-      compile unless @@objects
+      compile
       str = str.strip.squeeze(' ').downcase
       return @@codes[str] if str.length == 3
       return @@names[str] if @@names[str]
@@ -82,7 +82,7 @@ module ICU
     end
 
     def self.menu(opts = {})
-      compile unless @@objects;
+      compile
       top, menu = nil, []
       @@objects.each {|o| opts[:top] == o.code ? top = [o.name, o.code] : menu.push([o.name, o.code]) }
       opts[:order] == 'code' ? menu.sort!{|a,b| a.last <=> b.last} : menu.sort!{|a,b| a.first <=> b.first}
@@ -90,6 +90,11 @@ module ICU
       menu.unshift([opts[:none], '']) if opts[:none]
       menu
     end
+    
+    def self.codes
+      compile
+      @@objects.map(&:code).sort
+    end
 
     def initialize(code, name) # :nodoc: because new is private
       @code = code

data/lib/icu_tournament/version.rb

@@ -2,6 +2,6 @@
 
 module ICU
   class Tournament
-    VERSION = "1.2.2"
+    VERSION = "1.2.3"
   end
 end

data/spec/federation_spec.rb

@@ -8,70 +8,70 @@ module ICU
         fed.code.should == 'IRL'
         fed.name.should == 'Ireland'
       end
-      
+
       it "should find a federation from code case insensitively" do
         fed = Federation.find('rUs')
         fed.code.should == 'RUS'
         fed.name.should == 'Russia'
       end
-      
+
       it "should find a federation despite irrelevant whitespace" do
         fed = Federation.find('  mex    ')
         fed.code.should == 'MEX'
         fed.name.should == 'Mexico'
       end
-      
+
       it "should return nil for an invalid code" do
         Federation.find('XYZ').should be_nil
       end
     end
-    
+
     context "#find using names" do
       it "should find a federation given a valid name" do
         fed = Federation.find('England')
         fed.code.should == 'ENG'
         fed.name.should == 'England'
       end
-      
+
       it "should find a federation from name case insensitively" do
         fed = Federation.find('franCE')
         fed.code.should == 'FRA'
         fed.name.should == 'France'
       end
-      
+
       it "should not be fooled by irrelevant whitespace" do
         fed = Federation.find(' united  states  of  america ')
         fed.code.should == 'USA'
         fed.name.should == 'United States of America'
       end
-      
+
       it "should return nil for an invalid name" do
         Federation.find('Mordor').should be_nil
       end
     end
-    
+
     context "#find using parts of names" do
       it "should find a federation given a substring which is unique and at least 4 characters" do
         fed = Federation.find('bosni')
         fed.code.should == 'BIH'
         fed.name.should == 'Bosnia and Herzegovina'
       end
-      
+
       it "should not be fooled by irrelevant whitespace" do
         fed = Federation.find('  arab    EMIRATES   ')
         fed.code.should == 'UAE'
         fed.name.should == 'United Arab Emirates'
       end
-      
+
       it "should not find a federation if the substring matches more than one" do
         Federation.find('land').should be_nil
       end
-      
+
       it "should return nil for any string smaller in length than 3" do
         Federation.find('ze').should be_nil
       end
     end
-    
+
     context "#find federations with alternative names" do
       it "should find Macedonia multiple ways" do
         Federation.find('MKD').name.should == 'Macedonia'
@@ -83,7 +83,7 @@ module ICU
         Federation.find('former yugoslav republic').name.should == 'Macedonia'
       end
     end
-      
+
     context "#find with alternative inputs" do
       it "should behave robustly with completely invalid inputs" do
         Federation.find().should be_nil
@@ -92,13 +92,13 @@ module ICU
         Federation.find(1).should be_nil
       end
     end
-    
+
     context "#new is private" do
       it "#new cannot be called directly" do
         lambda { Federation.new('IRL', 'Ireland') }.should raise_error(/private method/)
       end
     end
-    
+
     context "documentation examples" do
       it "should all be correct for valid input" do
         Federation.find('IRL').name.should == 'Ireland'
@@ -107,18 +107,18 @@ module ICU
         Federation.find('ongoli').name.should == 'Mongolia'
         Federation.find('  united   states   ').code.should == 'USA'
       end
-      
+
       it "should return nil for invalid input" do
         Federation.find('ZYX').should be_nil
         Federation.find('land').should be_nil
       end
     end
-    
+
     context "#menu" do
       before(:all) do
         @total = 173
       end
-      
+
       it "should return array of name-code pairs in order of name by default" do
         menu = Federation.menu
         menu.should have(@total).items
@@ -129,7 +129,7 @@ module ICU
         codes.index('AFG').should == 0
         codes.index('IRQ,IRL,ISR').should_not be_nil
       end
-      
+
       it "should be configuarble to order the list by codes" do
         menu = Federation.menu(:order => "code")
         menu.should have(@total).items
@@ -140,7 +140,7 @@ module ICU
         codes.index('AFG').should == 0
         codes.index('IRL,IRQ,ISL').should_not be_nil
       end
-      
+
       it "should be configuarble to have a selected country at the top" do
         menu = Federation.menu(:top => 'IRL')
         menu.should have(@total).items
@@ -151,7 +151,7 @@ module ICU
         codes.index('IRL,AFG').should == 0
         codes.index('IRQ,ISR').should_not be_nil
       end
-      
+
       it "should be configuarble to have 'None' entry at the top" do
         menu = Federation.menu(:none => 'None')
         menu.should have(@total + 1).items
@@ -160,7 +160,7 @@ module ICU
         names.index('None,Afghanistan').should == 0
         codes.index(',AFG').should == 0
       end
-      
+
       it "should be able to handle multiple configuarations" do
         menu = Federation.menu(:top => 'IRL', :order => 'code', :none => 'None')
         menu.should have(@total + 1).items
@@ -172,5 +172,20 @@ module ICU
         codes.index('IRQ,ISL').should_not be_nil
       end
     end
+
+    context "#codes" do
+      before(:all) do
+        @total = 173
+      end
+
+      it "should return array of codes ordered alphabetically" do
+        codes = Federation.codes
+        codes.should have(@total).items
+        all = codes.join(',')
+        puts all
+        all.index('AFG').should == 0
+        all.index('INA,IND,IRI,IRL,IRQ,ISL,ISR,ISV,ITA,IVB').should_not be_nil
+      end
+    end
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 2
-  - 2
-  version: 1.2.2
+  - 3
+  version: 1.2.3
 platform: ruby
 authors: 
 - Mark Orr
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-02 00:00:00 +00:00
+date: 2011-01-09 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency