cache busting that actually works

joshski · Feb 2, 2011 · 6715f6f · 6715f6f
1 parent 616dc5d
commit 6715f6f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/lackie/cache_buster.rb b/lib/lackie/cache_buster.rb
@@ -2,7 +2,7 @@ module Lackie
   class CacheBuster
     def self.unique_string
       @@unique_number ||= 0
-      (@@unique_number += 1).to_s
+      "#{@@unique_number += 1}#{Time.now.to_i}"
     end
   end
 end
diff --git a/spec/lackie/cache_buster_spec.rb b/spec/lackie/cache_buster_spec.rb
@@ -3,8 +3,13 @@
 
 module Lackie
   describe CacheBuster do
-    it "returns unique strings" do
+    it "produces unique strings" do
       (1..99).map { |i| CacheBuster.unique_string }.uniq.size.should == 99
     end
+
+    it "produces strings that include the current time" do
+      Time.should_receive(:now).and_return("332211")
+      CacheBuster.unique_string.should =~ /332211/
+    end
   end
 end