@@ -1,7 +1,17 @@ class Postie < ActiveRecord::Base def Postie.find_by_pretty_id(pretty_id) - Postie.all.detect do |postie| - postie.name.downcase.gsub(/\W/, '') == pretty_id + postie = Postie.all.detect do |postie| + postie.pretty_id == pretty_id end + raise ActiveRecord::RecordNotFound unless postie + postie + end + + def pretty_id + self.name.downcase.gsub(/\W/, '') + end + + def to_param + self.pretty_id end end app/models/postie.rb @@ -19,5 +19,21 @@ describe Postie do Postie.find_by_pretty_id('mypostie').should == postie end + it "should raise an error when no name matches" do + postie = Postie.new(:name => 'My Postie') + postie.save! + + lambda{Postie.find_by_pretty_id('dontexist')}.should raise_error(ActiveRecord::RecordNotFound) + end + end + + it "should know its pretty_id" do + postie = Postie.new(:name => 'My Postie') + postie.pretty_id.should == 'mypostie' + end + + it "should use its pretty_id for params" do + postie = Postie.new(:name => 'My Postie') + postie.to_param.should == postie.pretty_id end end spec/models/postie_spec.rb 4dc282e558887f38b0337af1fcd22ba65a80db40 Paul Cortens paul@thoughtless.ca http://github.com/thoughtless/cuc_demo/commit/679529424b0edce220e1f0c14d482bac5dcad59e 679529424b0edce220e1f0c14d482bac5dcad59e 2008-11-24T13:17:26-08:00 2008-11-24T13:17:26-08:00 have Postie#to_param use pretty_id 06c40f6a6e4d7e17f8cb2691de0f39ae7f37caeb Paul Cortens paul@thoughtless.ca