@@ -58,6 +58,7 @@ class FlowMeter < ActiveRecord::Base def cleaned_up_url(att) new_att = self[att].gsub(%r{//+},'/').gsub(%r{\s+},'') + new_att.gsub!(%r{\/$},'') unless new_att == '/' new_att.gsub!(%r{^/},'') unless new_att == '/' new_att end app/models/flow_meter.rb @@ -64,6 +64,12 @@ describe FlowMeter do @flow_meter.redirect_url.should == 'chop' end + it "should remove a trailing slash from the catch_url before saving" do + @flow_meter.catch_url = '/no_trailing_slash/' + @flow_meter.save + @flow_meter.catch_url.should == 'no_trailing_slash' + end + it "should remove all whitespace from catch_url before saving" do @flow_meter.catch_url = "hello there" @flow_meter.save @@ -107,6 +113,7 @@ describe FlowMeter do end it "should load save all flow_meters into FlowMeter.all, a Hash with the catch_url as the key, and an array of redirect_url and status as the value" do + FlowMeter.destroy_all @flow_meter.save FlowMeter.all.should == {'stuff' => ['things', '307 Temporarily Moved']} end spec/models/flow_meter_spec.rb 739c155cb18992209d7e81890f36fdbfe8baf138 Jim Gay jim@saturnflyer.com http://github.com/jomz/radiant-vapor-extension/commit/8305de17c0f7413980f09a65106dd56ac63d812a 8305de17c0f7413980f09a65106dd56ac63d812a 2008-12-30T13:55:01-08:00 2008-12-30T13:55:01-08:00 disallow trailing slash in db d70a791d5a01699b181acd810cf61fb5836ebc71 Jim Gay jim@saturnflyer.com