oops forgot to actually check for slug collisions

coreypurcell · coreypurcell · commit 1fb3a551fb50 · 2011-11-01T22:30:39.000-04:00
diff --git a/app/models/program.rb b/app/models/program.rb
@@ -13,7 +13,7 @@ class Program
   def make_slug
     if slug.blank?
       all_slugs = Program.where(:author_username => author_username).map(&:slug)
-      self.slug = Sluggifier.generate(title)
+      self.slug = Sluggifier.generate(title, all_slugs)
     end
   end
 
diff --git a/spec/unit/program_spec.rb b/spec/unit/program_spec.rb
@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Program do
+    let(:prog) { Program.create(author_username: 'tester', title: 'test title')}
+  it "sets the slug" do
+    prog.slug.should == 'test-title'
+  end
+
+  it "makes unique slugs" do
+    first_prog = Program.create(author_username: 'tester', 
+                   title: 'test title #2',
+                   slug: 'test-title')
+    prog.slug.should_not == first_prog.slug
+  end
+end
diff --git a/spec/unit/sluggifier_spec.rb b/spec/unit/sluggifier_spec.rb
@@ -8,7 +8,6 @@
     it "makes a slug from a title" do
       Sluggifier.generate('test a slug').should == 'test-a-slug'
     end
-    
 
     context "doesn't allow multiple slugs to be the same" do
         let(:all_slugs) { ['test'] }
