lib/skating_system/ranking.rb spec/spec/ranking_spec.rb @@ -1,4 +1,5 @@ require 'skating_system/performance_results' +require 'skating_system/ranking' require 'skating_system/scorer' module SkatingSystem lib/skating_system.rb @@ -15,6 +15,22 @@ module SkatingSystem end end + def ranking + return @ranking if @ranking + @ranking =SkatingSystem::Ranking.new + @marks.each do |entree, data| + @ranking.add(data[:result], entree) + end + @ranking + end + + def [](key) + @marks[key] + end + + + + private def sum_marks(entree, position) sum=0 @marks[entree].each do |judge, place| @@ -59,12 +75,7 @@ module SkatingSystem end candidates end - - def [](key) - @marks[key] - end - - private + #Tallys the number of judges who marked each entree in each place or heigher # and writes the results to @tally # eg given the results lib/skating_system/performance_results.rb @@ -2,12 +2,12 @@ Gem::Specification.new do |s| s.platform = Gem::Platform::RUBY s.name = "skating-system" - s.version = "0.0.1" + s.version = "0.0.2" s.author = "Laurie Young" s.email = "skating_system@wildfalcon.com" s.summary = "Implemetation of the skating system." - s.files = ['lib/skating_system.rb', 'lib/skating_system/scorer.rb', 'lib/skating_system/performance_results.rb', - 'spec/spec.opts', 'spec/spec_helper.rb', 'spec/spec/scorer_spec.rb', 'spec/spec/performance_results_spec.rb'] + s.files = ['lib/skating_system.rb', 'lib/skating_system/scorer.rb', 'lib/skating_system/performance_results.rb', 'lib/skating_system/ranking.rb', + 'spec/spec.opts', 'spec/spec_helper.rb', 'spec/spec/scorer_spec.rb', 'spec/spec/performance_results_spec.rb', 'spec/spec/ranking_spec.rb'] s.require_path = "lib" s.autorequire = "skating_system" s.has_rdoc = true skating-system.gemspec @@ -60,4 +60,48 @@ describe SkatingSystem::PerformanceResults, "tallying the marks" do end end +describe SkatingSystem::PerformanceResults, "getting the rankings" do + before(:each) do + @performance_resuts = SkatingSystem::PerformanceResults.new + + @judge1 = mock("Judge") + @judge2 = mock("Judge") + @judge3 = mock("Judge") + + @entree1 = mock("Entree") + @entree2 = mock("Entree") + @entree3 = mock("Entree") + + # A B C 1 2 3 + # 1 1 1 3 2 2 3 1 + # 2 2 3 2 0 2 3 2 + # 3 3 2 3 0 1 3 3 + @couples_marks = { @entree1=>{@judge1=>1, @judge2=>1, @judge3=>3 }, + @entree2=>{@judge1=>2, @judge2=>3, @judge3=>2 }, + @entree3=>{@judge1=>3, @judge2=>2, @judge3=>3 }} + + SkatingSystem::Ranking.stub!(:new).and_return(@ranking = mock(SkatingSystem::Ranking)) + @ranking.stub!(:add) + + @performance_resuts.score(@couples_marks) + end + + it "should return a Ranking when asked for one" do + @performance_resuts.ranking.should be(@ranking) + end + + it "should add couple 1 to the ranking in first place" do + @ranking.should_receive(:add).with(1, @entree1) + @performance_resuts.ranking + end + it "should add couple 2 to the ranking in second place" do + @ranking.should_receive(:add).with(2, @entree2) + @performance_resuts.ranking + end + + it "should add couple 3 to the ranking in third place" do + @ranking.should_receive(:add).with(3, @entree3) + @performance_resuts.ranking + end +end spec/spec/performance_results_spec.rb @@ -2,7 +2,9 @@ dir = File.dirname(__FILE__) lib_path = File.expand_path("#{dir}/../lib") $LOAD_PATH.unshift lib_path unless $LOAD_PATH.include?(lib_path) +require 'rubygems' require 'skating_system' +require 'ruby-debug' Spec::Runner.configure do |config| spec/spec_helper.rb 3356bef2367217512be5061596354eaa667ef840 Laurie Young laurie@wildfalcon.com http://github.com/wildfalcon/skating-system/commit/f100ccfa6dd1869c7bc78eac54c7f9c32f53a87d f100ccfa6dd1869c7bc78eac54c7f9c32f53a87d 2008-07-03T14:31:08-07:00 2008-07-03T14:31:08-07:00 Added a ranking return for performance results, and updated performance results to include first have of rules 7 8b7e07757d759fdf5e5935e103888ab80aefc013 Laurie Young laurie@wildfalcon.com