deps/v1/ for Rails cache only, versions for fastly only

pt-stripe · Oct 8, 2016 · 6e51006 · 6e51006
1 parent 13ee266
commit 6e51006
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/gem_cache_purger.rb b/lib/gem_cache_purger.rb
@@ -1,9 +1,12 @@
 class GemCachePurger
   def self.call(gem_name)
     # We need to purge from Fastly and from Memcached
-    ["deps/v1/#{gem_name}", "info/#{gem_name}", "names"].each do |path|
+    ["info/#{gem_name}", "names"].each do |path|
       Rails.cache.delete(path)
       Fastly.delay.purge(path)
     end
+
+    Rails.cache.delete("deps/v1/#{gem_name}")
+    Fastly.delay.purge("versions")
   end
 end
diff --git a/test/unit/gem_cache_purger_test.rb b/test/unit/gem_cache_purger_test.rb
@@ -12,14 +12,15 @@ class GemCachePurgerTest < ActiveSupport::TestCase
 
     should "expire API memcached" do
       assert_received(Rails.cache, :delete) { |cache| cache.with("info/#{@gem_name}") }
-      assert_received(Rails.cache, :delete) { |cache| cache.with("deps/v1/#{@gem_name}") }
       assert_received(Rails.cache, :delete) { |cache| cache.with("names") }
+      assert_received(Rails.cache, :delete) { |cache| cache.with("deps/v1/#{@gem_name}") }
     end
 
     should "purge cdn cache" do
       Delayed::Worker.new.work_off
       assert_received(Fastly, :purge) { |path| path.with("info/#{@gem_name}") }
       assert_received(Fastly, :purge) { |path| path.with("names") }
+      assert_received(Fastly, :purge) { |path| path.with("versions") }
     end
   end
 end