ghazel-rack-bug 0.3.0.1

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

@@ -0,0 +1,30 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+class Rack::Bug
+  describe ActiveRecordPanel do
+    before do
+      ActiveRecordPanel.reset
+      rack_env "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_via_rack "/"
+        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_via_rack "/"
+        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,167 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+class Rack::Bug
+  describe CachePanel do
+    before do
+      CachePanel.reset
+      rack_env "rack-bug.panel_classes", [CachePanel]
+      unless defined?(Rails)
+        @added_rails = true
+        Object.const_set :Rails, Module.new
+      end
+    end
+
+    after do
+      Object.send :remove_const, :Rails if @added_rails
+    end
+
+    describe "heading" do
+      it "displays the total memcache time" do
+        response = get_via_rack "/"
+        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_via_rack "/"
+
+          # 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_via_rack "/"
+          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_via_rack "/"
+          response.should have_row("#cache_usage", "Hits", "0")
+        end
+
+        it "displays the number of memcache misses" do
+          CachePanel.record(:get, "user:1") { }
+          response = get_via_rack "/"
+          response.should have_row("#cache_usage", "Misses", "1")
+        end
+
+        it "displays the number of memcache gets" do
+          CachePanel.record(:get, "user:1") { }
+          response = get_via_rack "/"
+          response.should have_row("#cache_usage", "gets", "1")
+        end
+
+        it "displays the number of memcache sets" do
+          CachePanel.record(:set, "user:1") { }
+          response = get_via_rack "/"
+          response.should have_row("#cache_usage", "sets", "1")
+        end
+
+        it "displays the number of memcache deletes" do
+          CachePanel.record(:delete, "user:1") { }
+          response = get_via_rack "/"
+          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_via_rack "/"
+          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_via_rack "/"
+          response.should have_row("#cache_breakdown", "get")
+        end
+
+        it "displays the time for each memcache call" do
+          CachePanel.record(:get, "user:1") { }
+          response = get_via_rack "/"
+          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_via_rack "/"
+          response.should have_row("#cache_breakdown", "user:1", "get")
+        end
+      end
+    end
+
+    describe "expire_all" do
+      before do
+        rack_env "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_via_rack "/__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_via_rack "/__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
+        rack_env "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_via_rack "/__rack_bug__/delete_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+      end
+
+      it "returns OK" do
+        Rails.stub!(:cache => mock("cache", :delete => nil))
+        response = get_via_rack "/__rack_bug__/delete_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+        response.should contain("OK")
+      end
+    end
+
+    describe "view_cache" do
+      before do
+        rack_env "rack-bug.secret_key", 'abc'
+      end
+
+      it "renders the cache key" do
+        Rails.stub!(:cache => mock("cache", :read => "cache body"))
+        response = get_via_rack "/__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_via_rack "/__rack_bug__/view_cache", :key => "user:1",
+          :hash => "f87215442d312d8e42cf51e6b66fc3eb3d59ac74"
+        response.should contain("[1, 2]")
+      end
+    end
+
+  end
+end

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

@@ -0,0 +1,43 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+class Rack::Bug
+  describe LogPanel do
+    before do
+      LogPanel.reset
+      rack_env "rack-bug.panel_classes", [LogPanel]
+    end
+
+    describe "heading" do
+      it "displays 'Log'" do
+        response = get_via_rack "/"
+        response.should have_heading("Log")
+      end
+    end
+
+    describe "content" do
+      it "displays recorded log lines" do
+        LogPanel.record("This is a logged message", 0)
+        response = get_via_rack "/"
+        response.should contain("This is a logged message")
+        response.should contain("DEBUG")
+      end
+    end
+    
+    describe "Extended Logger" do
+      it "does still return true/false for #add if class Logger" do
+        #AS::BufferedLogger returns the added string, Logger returns true/false
+        LOGGER.add(0, "foo").should  == true
+      end
+    end
+    
+    
+    describe "With no logger defined" do
+      it "does not err out" do
+        logger = LOGGER
+        Object.send :remove_const, :LOGGER
+        lambda{ load("rack/bug/panels/log_panel/logger_extension.rb") }.should_not raise_error
+        ::LOGGER = logger
+      end
+    end
+  end
+end

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

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

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

@@ -0,0 +1,40 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+class Rack::Bug
+  describe RailsInfoPanel do
+    before do
+      rack_env "rack-bug.panel_classes", [RailsInfoPanel]
+
+      unless defined?(Rails)
+        @added_rails = true
+        Object.const_set :Rails, Module.new
+        Rails::Info = Class.new do
+          def self.properties
+            []
+          end
+        end
+      end
+    end
+
+    after do
+      Object.send :remove_const, :Rails if @added_rails
+    end
+
+    describe "heading" do
+      it "displays the Rails version" do
+        Rails.stub!(:version => "v2.3.0")
+        response = get_via_rack "/"
+        response.should have_heading("Rails v2.3.0")
+      end
+    end
+
+    describe "content" do
+      it "displays the Rails::Info properties" do
+        Rails.stub!(:version => "v2.3.0")
+        Rails::Info.stub!(:properties => [["Name", "Value"]])
+        response = get_via_rack "/"
+        response.should have_row("#rails_info", "Name", "Value")
+      end
+    end
+  end
+end

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

@@ -0,0 +1,69 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+$LOADED_FEATURES << "redis.rb" #avoid dependency on redis
+
+class Rack::Bug
+  describe RedisPanel do
+    before do
+      RedisPanel.reset
+      rack_env "rack-bug.panel_classes", [RedisPanel]
+    end
+
+    describe "heading" do
+      it "displays the total redis time" do
+        response = get_via_rack "/"
+        response.should have_heading("Redis: 0.00ms")
+      end
+    end
+
+    describe "content" do
+      describe "usage table" do
+        it "displays the total number of redis calls" do
+          RedisPanel.record(["get, user:1"], Kernel.caller) { }
+          response = get_via_rack "/"
+
+          # This causes a bus error:
+          # response.should have_selector("th:content('Total Calls') + td", :content => "1")
+
+          response.should have_row("#redis_usage", "Total Calls", "1")
+        end
+
+        it "displays the total redis time" do
+          response = get_via_rack "/"
+          response.should have_row("#redis_usage", "Total Time", "0.00ms")
+        end
+      end
+
+      describe "breakdown" do
+        it "displays each redis operation" do
+          RedisPanel.record(["get, user:1"], Kernel.caller) { }
+          response = get_via_rack "/"
+          response.should have_row("#redis_breakdown", "get")
+        end
+
+        it "displays the time for redis call" do
+          RedisPanel.record(["get, user:1"], Kernel.caller) { }
+          response = get_via_rack "/"
+          response.should have_row("#redis_breakdown", "user:1", TIME_MS_REGEXP)
+        end
+
+        it "displays the arguments for each redis call" do
+          RedisPanel.record(["get, user:1"], Kernel.caller) { }
+          response = get_via_rack "/"
+          response.should have_row("#redis_breakdown", "user:1", "get")
+        end
+
+        it "displays a link to show the backtrace when it's available" do
+          RedisPanel.record(["get, user:1"], Kernel.caller) { }
+          response = get_via_rack "/"
+          response.should have_row("#redis_breakdown", "user:1", "Show Backtrace")
+        end
+
+        it "does not display a link to show the backtrace when it's not available" do
+          RedisPanel.record(["get, user:1"], []) { }
+          response = get_via_rack "/"
+          response.should_not contain("Show Backtrace")
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,146 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
+
+class Rack::Bug
+  describe SQLPanel do
+    before do
+      SQLPanel.reset
+      rack_env "rack-bug.panel_classes", [SQLPanel]
+
+      unless defined?(ActiveRecord)
+        @added_rails = true
+        Object.const_set :ActiveRecord, Module.new
+        ActiveRecord.const_set :Base, Class.new
+      end
+    end
+    
+    after do
+      Object.send :remove_const, :ActiveRecord if @added_active_record
+    end
+
+    describe "heading" do
+      it "displays the total SQL query count" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get_via_rack "/"
+        response.should have_heading("1 Queries")
+      end
+
+      it "displays the total SQL time" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get_via_rack "/"
+        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_via_rack "/"
+        response.should have_row("#sql", "SELECT NOW();")
+      end
+
+      it "displays the time of each executed SQL query" do
+        SQLPanel.record("SELECT NOW();") { }
+        response = get_via_rack "/"
+        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
+        rack_env "rack-bug.secret_key", "abc"
+        expect_query "SELECT username FROM users",
+          [["username"],
+           ["bryan"]]
+
+        response = get_via_rack "/__rack_bug__/execute_sql", {:query => "SELECT username FROM users",
+          :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"}, {:xhr => true}
+        response.should contain("SELECT username FROM users")
+        response.should be_ok
+      end
+
+      it "is forbidden when the hash is missing or wrong" do
+        rack_env "rack-bug.secret_key", 'abc'
+
+        lambda {
+          get_via_rack "/__rack_bug__/execute_sql", {:query => "SELECT username FROM users",
+            :hash => "foobar"}, {:xhr => true}
+        }.should raise_error(SecurityError)
+      end
+
+      it "is not available when the rack-bug.secret_key is nil" do
+        rack_env "rack-bug.secret_key", nil
+
+        lambda {
+          get_via_rack "/__rack_bug__/execute_sql", {:query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"}, {:xhr => true}
+        }.should raise_error(SecurityError)
+      end
+
+      it "is not available when the rack-bug.secret_key is an empty string" do
+        rack_env "rack-bug.secret_key", ""
+
+        lambda {
+          get_via_rack "/__rack_bug__/execute_sql", {:query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"}, {:xhr => true}
+        }.should raise_error(SecurityError)
+      end
+    end
+
+    describe "explain_sql" do
+      it "displays the query explain plan" do
+        rack_env "rack-bug.secret_key", "abc"
+        expect_query "EXPLAIN SELECT username FROM users",
+          [["table"],
+           ["users"]]
+
+        response = get_via_rack "/__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
+        rack_env "rack-bug.secret_key", 'abc'
+
+        lambda {
+          get_via_rack "/__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
+        rack_env "rack-bug.secret_key", nil
+
+        lambda {
+          get_via_rack "/__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
+        rack_env "rack-bug.secret_key", ""
+
+        lambda {
+          get_via_rack "/__rack_bug__/explain_sql", :query => "SELECT username FROM users",
+            :hash => "6f286f55b75716e5c91f16d77d09fa73b353ebc1"
+        }.should raise_error(SecurityError)
+      end
+    end
+  end
+end