perfectline-rack-bug 0.1.1

data/spec/rack/bug/panels/active_record_panel_spec.rb

@@ -0,0 +1,30 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe ActiveRecordPanel do
+    before do
+      ActiveRecordPanel.reset
+      header "rack-bug.panel_classes", [ActiveRecordPanel]
+    end
+    
+    describe "heading" do
+      it "displays the total number of instantiated AR objects" do
+        ActiveRecordPanel.record("User")
+        ActiveRecordPanel.record("Group")
+        response = get "/"
+        response.should have_heading("2 AR Objects")
+      end
+    end
+    
+    describe "content" do
+      it "displays the count of instantiated objects for each class" do
+        ActiveRecordPanel.record("User")
+        ActiveRecordPanel.record("User")
+        ActiveRecordPanel.record("Group")
+        response = get "/"
+        response.should have_row("#active_record", "User", "2")
+        response.should have_row("#active_record", "Group", "1")
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/cache_panel_spec.rb

@@ -0,0 +1,159 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe CachePanel do
+    before do
+      CachePanel.reset
+      header "rack-bug.panel_classes", [CachePanel]
+    end
+    
+    describe "heading" do
+      it "displays the total memcache time" do
+        response = get "/"
+        response.should have_heading("Cache: 0.00ms")
+      end
+    end
+    
+    describe "content" do
+      describe "usage table" do
+        it "displays the total number of memcache calls" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          
+          # This causes a bus error:
+          # response.should have_selector("th:content('Total Calls') + td", :content => "1")
+
+          response.should have_row("#cache_usage", "Total Calls", "1")
+        end
+        
+        it "displays the total memcache time" do
+          response = get "/"
+          response.should have_row("#cache_usage", "Total Time", "0.00ms")
+        end
+        
+        it "dispays the number of memcache hits" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "Hits", "0")
+        end
+        
+        it "displays the number of memcache misses" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "Misses", "1")
+        end
+        
+        it "displays the number of memcache gets" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "gets", "1")
+        end
+        
+        it "displays the number of memcache sets" do
+          CachePanel.record(:set, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "sets", "1")
+        end
+        
+        it "displays the number of memcache deletes" do
+          CachePanel.record(:delete, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "deletes", "1")
+        end
+        
+        it "displays the number of memcache get_multis" do
+          CachePanel.record(:get_multi, "user:1", "user:2") { }
+          response = get "/"
+          response.should have_row("#cache_usage", "get_multis", "1")
+        end
+      end
+      
+      describe "breakdown" do
+        it "displays each memcache operation" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_breakdown", "get")
+        end
+        
+        it "displays the time for each memcache call" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_breakdown", "user:1", TIME_MS_REGEXP)
+        end
+        
+        it "displays the keys for each memcache call" do
+          CachePanel.record(:get, "user:1") { }
+          response = get "/"
+          response.should have_row("#cache_breakdown", "user:1", "get")
+        end
+      end
+    end
+    
+    describe "expire_all" do
+      before do
+        header "rack-bug.secret_key", 'abc'
+      end
+      
+      it "expires the cache keys" do
+        Rails.stub!(:cache => mock("cache"))
+        Rails.cache.should_receive(:delete).with("user:1")
+        Rails.cache.should_receive(:delete).with("user:2")
+        Rails.cache.should_receive(:delete).with("user:3")
+        Rails.cache.should_receive(:delete).with("user:4")
+        
+        get "/__rack_bug__/delete_cache_list",
+          :keys_1 => "user:1", :keys_2 => "user:2", :keys_3 => "user:3", :keys_4 => "user:4",
+          :hash => "c367b76e0199c308862a3afd8fba32b8715e7976"
+      end
+      
+      it "returns OK" do
+        Rails.stub!(:cache => mock("cache", :delete => nil))
+        response = get "/__rack_bug__/delete_cache_list",
+          :keys_1 => "user:1", :keys_2 => "user:2", :keys_3 => "user:3", :keys_4 => "user:4",
+          :hash => "c367b76e0199c308862a3afd8fba32b8715e7976"
+        response.should contain("OK")
+      end
+    end
+    
+    describe "expire" do
+      before do
+        header "rack-bug.secret_key", 'abc'
+      end
+      
+      it "expires the cache key" do
+        Rails.stub!(:cache => mock("cache"))
+        Rails.cache.should_receive(:delete).with("user:1")
+        get "/__rack_bug__/delete_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+      end
+      
+      it "returns OK" do
+        Rails.stub!(:cache => mock("cache", :delete => nil))
+        response = get "/__rack_bug__/delete_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+        response.should contain("OK")
+      end
+    end
+    
+    describe "view_cache" do
+      before do
+        header "rack-bug.secret_key", 'abc'
+      end
+      
+      it "renders the cache key" do
+        Rails.stub!(:cache => mock("cache", :read => "cache body"))
+        response = get "/__rack_bug__/view_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+        response.should contain("cache body")
+      end
+      
+      it "renders non-String cache values properly" do
+        Rails.stub!(:cache => mock("cache", :read => [1, 2]))
+        response = get "/__rack_bug__/view_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+        response.should contain("[1, 2]")
+      end
+    end
+
+  end
+end

data/spec/rack/bug/panels/env_panel_spec.rb

@@ -0,0 +1,24 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe EnvPanel do
+    before do
+      header "rack-bug.panel_classes", [EnvPanel]
+    end
+    
+    describe "heading" do
+      it "displays 'Rack Env'" do
+        response = get "/"
+        response.should have_heading("Rack Env")
+      end
+    end
+    
+    describe "content" do
+      it "displays the Rack environment" do
+        response = get "/"
+        response.should have_row("#env", "PATH_INFO", "/")
+        response.should have_row("#env", "REMOTE_ADDR", "127.0.0.1")
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/log_panel_spec.rb

@@ -0,0 +1,25 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe LogPanel do
+    before do
+      LogPanel.reset
+      header "rack-bug.panel_classes", [LogPanel]
+    end
+    
+    describe "heading" do
+      it "displays 'Log'" do
+        response = get "/"
+        response.should have_heading("Log")
+      end
+    end
+    
+    describe "content" do
+      it "displays recorded log lines" do
+        LogPanel.record("This is a logged message")
+        response = get "/"
+        response.should contain("This is a logged message")
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/memory_panel_spec.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe MemoryPanel do
+    before do
+      header "rack-bug.panel_classes", [MemoryPanel]
+    end
+    
+    describe "heading" do
+      it "displays the total memory" do
+        response = get "/"
+        response.should have_heading(/\d+ KB total/)
+      end
+      
+      it "displays the memory change during the request" do
+        response = get "/"
+        response.should have_heading(/\d+ KB Δ/)
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/rails_info_panel_spec.rb

@@ -0,0 +1,25 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe RailsInfoPanel do
+    before do
+      header "rack-bug.panel_classes", [RailsInfoPanel]
+    end
+    
+    describe "heading" do
+      it "displays the Rails version" do
+        Rails.stub!(:version => "v2.3.0")
+        response = get "/"
+        response.should have_heading("Rails v2.3.0")
+      end
+    end
+
+    describe "content" do
+      it "displays the Rails::Info properties" do
+        Rails::Info.stub!(:properties => [["Name", "Value"]])
+        response = get "/"
+        response.should have_row("#rails_info", "Name", "Value")
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/sql_panel_spec.rb

@@ -0,0 +1,136 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe SQLPanel do
+    before do
+      SQLPanel.reset
+      header "rack-bug.panel_classes", [SQLPanel]
+    end
+    
+    describe "heading" do
+      it "displays the total SQL query count" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get "/"
+        response.should have_heading("1 Queries")
+      end
+      
+      it "displays the total SQL time" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get "/"
+        response.should have_heading(/Queries \(\d+\.\d{2}ms\)/)
+      end
+    end
+    
+    describe "content" do
+      it "displays each executed SQL query" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get "/"
+        response.should have_row("#sql", "SELECT NOW();")
+      end
+      
+      it "displays the time of each executed SQL query" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get "/"
+        response.should have_row("#sql", "SELECT NOW();", TIME_MS_REGEXP)
+      end
+    end
+    
+    def stub_result(results = [[]])
+      columns = results.first
+      fields = columns.map { |c| stub("field", :name => c) }
+      rows = results[1..-1]
+      
+      result = stub("result", :fetch_fields => fields)
+      result.stub!(:each).and_yield(*rows)
+      return result
+    end
+    
+    def expect_query(sql, results)
+      conn = stub("connection")
+      ActiveRecord::Base.stub!(:connection => conn)
+      conn.should_receive(:execute).with(sql).and_return(stub_result(results))
+    end
+    
+    describe "execute_sql" do
+      it "displays the query results" do
+        header "rack-bug.secret_key", "abc"
+        expect_query "SELECT username FROM users",
+          [["username"],
+           ["bryan"]]
+        
+        response = get "/__rack_bug__/execute_sql", :query => "SELECT username FROM users",
+          :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        response.should contain("SELECT username FROM users")
+        response.should be_ok
+      end
+      
+      it "is forbidden when the hash is missing or wrong" do
+        header "rack-bug.secret_key", 'abc'
+        
+        lambda {
+          get "/__rack_bug__/execute_sql", :query => "SELECT username FROM users",
+            :hash => "foobar"
+        }.should raise_error(SecurityError)
+      end
+      
+      it "is not available when the rack-bug.secret_key is nil" do
+        header "rack-bug.secret_key", nil
+        
+        lambda {
+          get "/__rack_bug__/execute_sql", :query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        }.should raise_error(SecurityError)
+      end
+      
+      it "is not available when the rack-bug.secret_key is an empty string" do
+        header "rack-bug.secret_key", ""
+        
+        lambda {
+          get "/__rack_bug__/execute_sql", :query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        }.should raise_error(SecurityError)
+      end
+    end
+    
+    describe "explain_sql" do
+      it "displays the query explain plan" do
+        header "rack-bug.secret_key", "abc"
+        expect_query "EXPLAIN SELECT username FROM users",
+          [["table"],
+           ["users"]]
+        
+        response = get "/__rack_bug__/explain_sql", :query => "SELECT username FROM users",
+          :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        response.should contain("SELECT username FROM users")
+        response.should be_ok
+      end
+      
+      it "is forbidden when the hash is missing or wrong" do
+        header "rack-bug.secret_key", 'abc'
+        
+        lambda {
+          get "/__rack_bug__/explain_sql", :query => "SELECT username FROM users",
+            :hash => "foobar"
+        }.should raise_error(SecurityError)
+      end
+      
+      it "is not available when the rack-bug.secret_key is nil" do
+        header "rack-bug.secret_key", nil
+        
+        lambda {
+          get "/__rack_bug__/explain_sql", :query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        }.should raise_error(SecurityError)
+      end
+      
+      it "is not available when the rack-bug.secret_key is an empty string" do
+        header "rack-bug.secret_key", ""
+        
+        lambda {
+          get "/__rack_bug__/explain_sql", :query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        }.should raise_error(SecurityError)
+      end
+    end
+  end
+end

data/spec/rack/bug/panels/templates_panel_spec.rb

@@ -0,0 +1,71 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+module Rack::Bug
+  describe TemplatesPanel do
+    before do
+      TemplatesPanel.reset
+      header "rack-bug.panel_classes", [TemplatesPanel]
+    end
+    
+    describe "heading" do
+      it "displays the total rendering time" do
+        response = get "/"
+        response.should have_heading("Templates: 0.00ms")
+      end
+    end
+    
+    describe "content" do
+      it "displays the template paths" do
+        TemplatesPanel.record("users/show") { }
+        response = get "/"
+        response.should contain("users/show")
+      end
+      
+      it "displays the template children" do
+        TemplatesPanel.record("users/show") do
+          TemplatesPanel.record("users/toolbar") { }
+        end
+        
+        response = get "/"
+        response.should have_selector("li", :content => "users/show") do |li|
+          li.should contain("users/toolbar")
+        end
+      end
+      
+      context "for templates that rendered templates" do
+        it "displays the total time" do
+          TemplatesPanel.record("users/show") do
+            TemplatesPanel.record("users/toolbar") { }
+          end
+          
+          response = get "/"
+          response.should have_selector("li", :content => "users/show") do |li|
+            li.should contain(TIME_MS_REGEXP)
+          end
+        end
+        
+        it "displays the exclusive time" do
+          TemplatesPanel.record("users/show") do
+            TemplatesPanel.record("users/toolbar") { }
+          end
+          
+          response = get "/"
+          response.should have_selector("li", :content => "users/show") do |li|
+            li.should contain(/\d\.\d{2} exclusive/)
+          end
+        end
+      end
+      
+      context "for leaf templates" do
+        it "does not display the exclusive time" do
+          TemplatesPanel.record("users/show") { }
+          
+          response = get "/"
+          response.should contain("users/show") do |li|
+            li.should_not contain("exclusive")
+          end
+        end
+      end
+    end
+  end
+end