generators/tartarus/templates/spec/controllers/exceptions_controller_spec.rb
@@ -11,7 +11,8 @@ class TartarusGenerator < Rails::Generator::NamedBase
puts "\nGenerated files:\n"
# Directories
m.directory "app/views/exceptions"
- m.directory 'spec/models'
+ m.directory "spec/models"
+ m.directory "spec/controllers"
# Configuration
m.template 'config/exceptions.yml', 'config/exceptions.yml'
@@ -33,6 +34,7 @@ class TartarusGenerator < Rails::Generator::NamedBase
# Specs
m.template 'spec/models/logged_exception_spec.rb', "spec/models/#{file_name}_spec.rb"
+ m.template 'spec/controllers/exceptions_controller_spec.rb', 'spec/controllers/exceptions_controller_spec.rb'
# Public
m.file 'public/javascripts/tartarus.jquery.js', 'public/javascripts/tartarus.jquery.js'
generators/tartarus/tartarus_generator.rb
@@ -1,10 +1,31 @@
class ExceptionsController < ApplicationController
def index
- @exceptions = <%= class_name %>.all(:select => '*, COUNT(*) as count', :group => 'hash_id', :order => 'created_at DESC')
+ @exceptions = <%= class_name %>.all(:select => '*, COUNT(*) as count', :group => 'group_id', :order => 'created_at DESC')
end
def details
- @exceptions = <%= class_name %>.paginate(:all, :conditions => { :hash_id => params[:id] }, :order => 'created_at DESC', :page => params[:page], :per_page => 1)
+ @exceptions = <%= class_name %>.paginate(:all, :conditions => { :group_id => params[:id] }, :order => 'created_at DESC', :page => params[:page], :per_page => 1)
@exception = @exceptions.first
end
+
+ def remove_all
+ <%= class_name %>.delete_all
+ redirect_to :action => :index
+ end
+
+ def remove_group
+ <%= class_name %>.delete_all(:group_id => params[:id])
+ redirect_to :action => :index
+ end
+
+ def remove_individual
+ exception = <%= class_name %>.find_by_id(params[:id])
+ exception.destroy
+
+ if <%= class_name %>.count(:conditions => { :group_id => exception.group_id }).zero?
+ redirect_to :action => :index
+ else
+ redirect_to :action => :details, :id => exception.group_id
+ end
+ end
end
generators/tartarus/templates/app/controllers/exceptions_controller.rb
@@ -1,6 +1,6 @@
-<tr id="exception_<%= exception.hash_id %>">
+<tr id="exception_<%= exception.group_id %>">
<td>
- <strong><%= link_to "#{exception.exception_class}", :action => 'details', :id => exception.hash_id %></strong>
+ <strong><%= link_to "#{exception.exception_class}", :action => 'details', :id => exception.group_id %></strong>
<br />
<span><%= truncate(exception.message, :length => 115) %></span>
</td>
generators/tartarus/templates/app/views/exceptions/_exception.html.erb
@@ -1,17 +1,24 @@
-<div id="exceptional">
+<div id="tartarus">
<div id="exception_group_details">
- <div id="exceptional_cap">
- <div id="exceptional_nav">
+ <div id="tartarus_cap">
+ <div id="tartarus_nav">
<%= link_to 'Return to overview', :controller => 'exceptions', :action => 'index' %>
- <%= will_paginate @exceptions, :page_links => false, :class => 'exceptional_pagination' %>
+ <%= will_paginate @exceptions, :page_links => false, :class => 'tartarus_pagination' %>
</div>
<h2><%= @exception.exception_class %> in "<%= "#{@exception.controller_path}##{@exception.action_name}" %>"</h2>
<div>Viewing Exception <%= params[:page] || 1 %> of <%= @exceptions.total_pages %></div>
</div>
</div>
+ <h3><%= @exception.created_at.strftime("%B %e %Y at %I:%M:%S%p") %></h3>
<pre><%= @exception.message %></pre>
+ <div id="exception_actions">
+ <%= link_to 'Remove all', :action => 'remove_group', :id => @exception.group_id %>
+ ·
+ <%= link_to 'Remove', :action => 'remove_individual', :id => @exception.id %>
+ </div>
+
<div id="exception">
<div id="request_information" class="togglable">
<h3><a class="toggle_link" href="javascript:void(0);">Request</a></h3>
generators/tartarus/templates/app/views/exceptions/details.html.erb
@@ -1,13 +1,13 @@
-<div id="exceptional">
- <div id="exceptional_cap">
- <div id="exceptional_nav">
+<div id="tartarus">
+ <div id="tartarus_cap">
+ <div id="tartarus_nav">
<a href="#">Remove all</a>
</div>
<h2>Exception Overview</h2>
<div><%= @exceptions.length %> Exceptions</div>
</div>
- <table id="exceptional_overview">
+ <table id="tartarus_overview">
<thead>
<tr>
<th id="exception_summary">Summary</th>
generators/tartarus/templates/app/views/exceptions/index.html.erb
@@ -1,7 +1,7 @@
class Add<%= class_name %>Table < ActiveRecord::Migration
def self.up
create_table :<%= plural_name %>, :force => true do |t|
- t.string :hash_id
+ t.string :group_id
t.string :exception_class
t.string :controller_path
t.string :action_name
generators/tartarus/templates/db/migrate/add_logged_exceptions.rb
@@ -1,23 +1,26 @@
-#exceptional { margin: 0px 20px }
+#tartarus { margin: 0px 20px }
-#exceptional_cap { padding-bottom: 10px; border-bottom: 1px solid #777; margin: 20px 0px; }
-#exceptional h2 { margin: 10px 0px; }
+#tartarus_cap { padding-bottom: 10px; border-bottom: 1px solid #777; margin: 20px 0px; }
+#tartarus h2 { margin: 10px 0px; }
-#exceptional_nav { float: right }
-.exceptional_pagination { margin-top: 20px; }
+#tartarus_nav { float: right }
+.tartarus_pagination { margin-top: 20px; }
-#exceptional table { border-collapse: collapse; width: 100%; }
-#exceptional table tr { border-bottom: 1px solid #E0DFE0; }
-#exceptional table th, #exceptional table td { padding: 5px; }
-#exceptional table th { text-align: left; }
+#tartarus table { border-collapse: collapse; width: 100%; }
+#tartarus table tr { border-bottom: 1px solid #E0DFE0; }
+#tartarus table th, #tartarus table td { padding: 5px; }
+#tartarus table th { text-align: left; }
-#exceptional th#exception_location { width: 150px; }
-#exceptional th#exception_date { width: 150px; }
-#exceptional th#exception_count { text-align: center; }
+#tartarus th#exception_location { width: 150px; }
+#tartarus th#exception_date { width: 150px; }
+#tartarus th#exception_count { text-align: center; }
-#exceptional pre { -webkit-border-radius: 5px; -moz-border-radius: 5px; font-size: 10px; color: #fff; background-color: #222; padding: 10px; font-family: Consolas, Monaco, 'Courier New', Courier,monospace; }
-#exceptional a, #exceptional a:visited { text-decoration: underline; #bdf; }
-#exceptional a:hover { #09f }
+#tartarus pre { -webkit-border-radius: 5px; -moz-border-radius: 5px; font-size: 10px; color: #fff; background-color: #222; padding: 10px; font-family: Consolas, Monaco, 'Courier New', Courier,monospace; }
+#tartarus a, #tartarus a:visited { text-decoration: underline; #bdf; }
+#tartarus a:hover { #09f }
+
+#tartarus #exception { clear: both; }
+#tartarus #exception_actions { float: right; }
.toggle_data { display: none }
#backtrace_information .toggle_data { display: block; }
generators/tartarus/templates/public/stylesheets/tartarus.css
@@ -1,6 +1,7 @@
require File.dirname(__FILE__) + '/../spec_helper'
describe <%= class_name %> do
- before( :each ) do
+ it "should include the Tartarus::Logger module" do
+ <%= class_name %>.included_modules.include?(Tartarus::Logger)
end
end
generators/tartarus/templates/spec/models/logged_exception_spec.rb
@@ -7,14 +7,15 @@ module Tartarus::Logger
module ClassMethods
def log(controller, exception)
create do |logged_exception|
- hash_id = "#{exception.class.name}#{exception.message}#{controller.controller_path}#{controller.action_name}"
+ group_id = "#{exception.class.name}#{exception.message}#{controller.controller_path}#{controller.action_name}"
+
logged_exception.exception_class = exception.class.name
logged_exception.controller_path = controller.controller_path
logged_exception.action_name = controller.action_name
logged_exception.message = exception.message
logged_exception.backtrace = exception.backtrace * "\n"
logged_exception.request = normalize_request_data(controller.request)
- logged_exception.hash_id = Digest::SHA1.hexdigest(hash_id)
+ logged_exception.group_id = Digest::SHA1.hexdigest(group_id)
end
end
lib/tartarus/logger.rb
spec/rails/db/test.sqlite3
@@ -13,9 +13,9 @@ describe Tartarus::Logger do
@exception.stub!(:backtrace).and_return(['one', 'two', 'three'])
end
- it "should create a hash_id for grouping of exceptions that are the same" do
+ it "should create a group_id for grouping of exceptions that are the same" do
@logged_exception = LoggedException.log(@controller, @exception)
- @logged_exception.hash_id.should == 'ea61658eacfe0930ae2b318297ab51a3c0b5668c'
+ @logged_exception.group_id.should == 'ea61658eacfe0930ae2b318297ab51a3c0b5668c'
end
it "should convert the backtrace from an array to a string seperated by newlines" do
spec/tartarus/logger_spec.rb
f4b67b38d4a907c025c59cf46dd45f2f68a019b9
Daniel Insley
dan@viviti.com
http://github.com/dinsley/tartarus/commit/c356c6461f2e991ca36761e2a11fa4a6eb0392ec
c356c6461f2e991ca36761e2a11fa4a6eb0392ec
2009-10-21T14:31:54-07:00
2009-10-21T14:31:54-07:00
generator base completed
2f22b86449dc805c5cb92e1d7731fc0f88fb335e
Daniel Insley
dan@viviti.com