cuke_sniffer 0.0.6 → 0.0.7
Sign up to get free protection for your applications and to get access to all the features.
- data/bin/cuke_sniffer +6 -1
- data/lib/cuke_sniffer/cli.rb +35 -14
- data/lib/cuke_sniffer/cuke_sniffer_helper.rb +165 -164
- data/lib/cuke_sniffer/formatter.rb +2 -19
- data/lib/cuke_sniffer/report/css.html.erb +133 -0
- data/lib/cuke_sniffer/report/dead_steps.html.erb +62 -35
- data/lib/cuke_sniffer/report/features.html.erb +104 -50
- data/lib/cuke_sniffer/report/hooks.html.erb +78 -48
- data/lib/cuke_sniffer/report/improvement_list.html.erb +12 -6
- data/lib/cuke_sniffer/report/js.html.erb +65 -0
- data/lib/cuke_sniffer/report/min_template.html.erb +16 -125
- data/lib/cuke_sniffer/report/rules.html.erb +75 -8
- data/lib/cuke_sniffer/report/standard_template.html.erb +14 -127
- data/lib/cuke_sniffer/report/step_definitions.html.erb +68 -49
- data/lib/cuke_sniffer/report/summary.html.erb +40 -15
- data/lib/cuke_sniffer/report/title.html.erb +9 -0
- data/lib/cuke_sniffer/rule_config.rb +508 -483
- metadata +5 -3
- data/lib/cuke_sniffer/report/sub_rules.html.erb +0 -24
|
@@ -32,8 +32,8 @@ module CukeSniffer
|
|
|
32
32
|
# Formats the section data for a summary object
|
|
33
33
|
def self.console_summary(name, summary)
|
|
34
34
|
" #{name}\n" +
|
|
35
|
-
" Min: #{summary[:min]}
|
|
36
|
-
" Max: #{summary[:max]}
|
|
35
|
+
" Min: #{summary[:min]}\n" +
|
|
36
|
+
" Max: #{summary[:max]}\n" +
|
|
37
37
|
" Average: #{summary[:average]}\n"
|
|
38
38
|
end
|
|
39
39
|
|
|
@@ -84,16 +84,9 @@ module CukeSniffer
|
|
|
84
84
|
|
|
85
85
|
# Returns the Rules erb page that utilizes sub page sections of enabled and disabled rules
|
|
86
86
|
def self.rules_template(cuke_sniffer)
|
|
87
|
-
enabled_rules = sub_rules_template(cuke_sniffer, true, "Enabled Rules")
|
|
88
|
-
disabled_rules = sub_rules_template(cuke_sniffer, false, "Disabled Rules")
|
|
89
87
|
ERB.new(extract_markup("rules.html.erb")).result(binding)
|
|
90
88
|
end
|
|
91
89
|
|
|
92
|
-
# Returns the sub rules erb page for enabled status of a rule
|
|
93
|
-
def self.sub_rules_template(cuke_sniffer, state, heading)
|
|
94
|
-
ERB.new(extract_markup("sub_rules.html.erb")).result(binding)
|
|
95
|
-
end
|
|
96
|
-
|
|
97
90
|
# Creates a xml file with the collected project details
|
|
98
91
|
# file_name defaults to "cuke_sniffer.xml" unless specified
|
|
99
92
|
# cuke_sniffer.output_xml
|
|
@@ -109,16 +102,6 @@ module CukeSniffer
|
|
|
109
102
|
end
|
|
110
103
|
end
|
|
111
104
|
|
|
112
|
-
# Formats a list of rule conditions into a compact string of 5 or less elements per row.
|
|
113
|
-
def self.convert_array_condition_into_list_of_strings(condition_list)
|
|
114
|
-
result = []
|
|
115
|
-
while condition_list.size > 0
|
|
116
|
-
five_words = condition_list.slice!(0, 5)
|
|
117
|
-
result << five_words.join(", ")
|
|
118
|
-
end
|
|
119
|
-
result
|
|
120
|
-
end
|
|
121
|
-
|
|
122
105
|
# Sorts all of the lists on a cuke_sniffer object to be in descending order for each objects score.
|
|
123
106
|
def self.sort_cuke_sniffer_lists(cuke_sniffer)
|
|
124
107
|
cuke_sniffer.features = cuke_sniffer.features.sort_by { |feature| feature.total_score }.reverse
|
|
@@ -0,0 +1,133 @@
|
|
|
1
|
+
<link rel="stylesheet" type="text/css">
|
|
2
|
+
|
|
3
|
+
<style>
|
|
4
|
+
.new_sub_section {
|
|
5
|
+
border: 2px solid #509f50;
|
|
6
|
+
padding: 2px;
|
|
7
|
+
display: none;
|
|
8
|
+
overflow:auto;
|
|
9
|
+
}
|
|
10
|
+
|
|
11
|
+
.title {
|
|
12
|
+
padding-left: 1%;
|
|
13
|
+
width: 25%;
|
|
14
|
+
background-color: #509f50;
|
|
15
|
+
color: white;
|
|
16
|
+
border-top: 0;
|
|
17
|
+
border-bottom: 0;
|
|
18
|
+
font-weight: bold;
|
|
19
|
+
font-size: 21pt;
|
|
20
|
+
cursor: pointer;
|
|
21
|
+
font-family: Arial;
|
|
22
|
+
border-top-left-radius: 15px;
|
|
23
|
+
border-top-right-radius: 15px;
|
|
24
|
+
}
|
|
25
|
+
|
|
26
|
+
.function_bar {
|
|
27
|
+
background-color: #509f50;
|
|
28
|
+
display: none;
|
|
29
|
+
}
|
|
30
|
+
|
|
31
|
+
.help {
|
|
32
|
+
cursor: hand;
|
|
33
|
+
}
|
|
34
|
+
|
|
35
|
+
.disabled_title_row {
|
|
36
|
+
background-color:#aca8a5;
|
|
37
|
+
cursor: hand;
|
|
38
|
+
padding-left: 2px;
|
|
39
|
+
}
|
|
40
|
+
|
|
41
|
+
.white_title_row {
|
|
42
|
+
cursor: hand;
|
|
43
|
+
padding-left: 2px;
|
|
44
|
+
}
|
|
45
|
+
|
|
46
|
+
.blue_title_row {
|
|
47
|
+
background-color:#ccffff;
|
|
48
|
+
cursor: hand;
|
|
49
|
+
padding-left: 2px;
|
|
50
|
+
}
|
|
51
|
+
|
|
52
|
+
.white_row {
|
|
53
|
+
padding-left: 2px;
|
|
54
|
+
}
|
|
55
|
+
|
|
56
|
+
.blue_row {
|
|
57
|
+
background-color:#ccffff;
|
|
58
|
+
padding-left: 2px;
|
|
59
|
+
}
|
|
60
|
+
|
|
61
|
+
.colored_subsection {
|
|
62
|
+
width:100%;
|
|
63
|
+
padding-left: 8px;
|
|
64
|
+
}
|
|
65
|
+
|
|
66
|
+
.shrink_section {
|
|
67
|
+
border: 2px solid #509f50;
|
|
68
|
+
display: none;
|
|
69
|
+
}
|
|
70
|
+
|
|
71
|
+
.table_top {
|
|
72
|
+
width: 20%;
|
|
73
|
+
font-size: 14pt;
|
|
74
|
+
padding: 0;
|
|
75
|
+
}
|
|
76
|
+
|
|
77
|
+
.divide_row {
|
|
78
|
+
border-bottom: 1px solid black;
|
|
79
|
+
}
|
|
80
|
+
|
|
81
|
+
.red_number {
|
|
82
|
+
color: red;
|
|
83
|
+
font-weight: bold;
|
|
84
|
+
display: inline;
|
|
85
|
+
}
|
|
86
|
+
|
|
87
|
+
.notes {
|
|
88
|
+
font-weight: bold;
|
|
89
|
+
}
|
|
90
|
+
|
|
91
|
+
.sub_section {
|
|
92
|
+
display: none;
|
|
93
|
+
margin-left: 1%;
|
|
94
|
+
margin-right: 1%;
|
|
95
|
+
border-top: 2px solid #509f50;
|
|
96
|
+
}
|
|
97
|
+
|
|
98
|
+
.empty_set_message {
|
|
99
|
+
font-weight: bold;
|
|
100
|
+
margin-left: 1%;
|
|
101
|
+
}
|
|
102
|
+
|
|
103
|
+
.rule_notes {
|
|
104
|
+
margin-left: 1%;
|
|
105
|
+
border-left: 2px solid #90ee90;
|
|
106
|
+
border-top: 2px solid #90ee90;
|
|
107
|
+
display: block;
|
|
108
|
+
clear: left;
|
|
109
|
+
padding-bottom: 0%;
|
|
110
|
+
text-indent: 0%;
|
|
111
|
+
padding-left: 0.5%;
|
|
112
|
+
margin-bottom: 0.5%;
|
|
113
|
+
}
|
|
114
|
+
|
|
115
|
+
.rule_style {
|
|
116
|
+
margin: 0;
|
|
117
|
+
text_indent: 3%;
|
|
118
|
+
|
|
119
|
+
}
|
|
120
|
+
|
|
121
|
+
.rule_heading {
|
|
122
|
+
font-weight: bold;
|
|
123
|
+
margin: 0.25%;
|
|
124
|
+
}
|
|
125
|
+
|
|
126
|
+
ol {
|
|
127
|
+
list-style-type: none;
|
|
128
|
+
display: block;
|
|
129
|
+
padding-left: 1%
|
|
130
|
+
|
|
131
|
+
}
|
|
132
|
+
|
|
133
|
+
</style>
|
|
@@ -1,36 +1,63 @@
|
|
|
1
|
-
<div class="title" onclick="toggleById('dead_steps_data', this)">
|
|
2
|
-
Dead Steps +
|
|
3
|
-
</div>
|
|
4
|
-
<div
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
<%
|
|
11
|
-
|
|
12
|
-
<%
|
|
13
|
-
<div class="
|
|
14
|
-
<%
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
|
|
21
|
-
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
1
|
+
<div class="title" onclick="toggleById('dead_steps_data', this); updateDisplayStatus(document.getElementById('dead_steps_function_bar'));updateDivScroll('dead_steps_data', 'dead_steps_table');">
|
|
2
|
+
Dead Steps +
|
|
3
|
+
</div>
|
|
4
|
+
<div id="dead_steps_function_bar" class="function_bar">
|
|
5
|
+
<input type="button" class="function_button" value="Collapse All" onclick="collapseAll('dead_step_detail')" />
|
|
6
|
+
<input type="button" class="function_button" value="Expand All" onclick="expandAllShiftingRows('dead_step_detail')" />
|
|
7
|
+
</div>
|
|
8
|
+
<div class="new_sub_section" id="dead_steps_data">
|
|
9
|
+
<% dead_steps = cuke_sniffer.get_dead_steps %>
|
|
10
|
+
<% dead_steps.delete(:total) %>
|
|
11
|
+
<% dead_step_count = dead_steps.values.flatten.count %>
|
|
12
|
+
<% if !cuke_sniffer.cataloged? %>
|
|
13
|
+
<div class="notes">Steps were not cataloged.</div>
|
|
14
|
+
<% elsif dead_step_count == 0 %>
|
|
15
|
+
<div class="notes">No dead steps found!</div>
|
|
16
|
+
<% elsif dead_step_count == 1 %>
|
|
17
|
+
<div class="notes"><%= dead_step_count %> Dead Step found.</div>
|
|
18
|
+
<% else %>
|
|
19
|
+
<div class="notes"><%= dead_step_count %> Dead Steps found.</div>
|
|
20
|
+
<% end %>
|
|
21
|
+
<% unless dead_steps.empty? or !cuke_sniffer.cataloged? %>
|
|
22
|
+
<table id="dead_steps_table" style="width:100%; border-top: 2px solid #509f50" border="0" cellspacing="0">
|
|
23
|
+
<% index = 0 %>
|
|
24
|
+
<% dead_steps.each_key do |file_name| %>
|
|
25
|
+
<% if index.odd? %>
|
|
26
|
+
<tr class="notes blue_title_row" onclick="updateDisplayStatus(document.getElementById('dead_step_file_detail_<%=index%>')); updateDivScroll('dead_steps_data', 'dead_steps_table');">
|
|
27
|
+
<% else %>
|
|
28
|
+
<tr class="notes white_title_row" onclick="updateDisplayStatus(document.getElementById('dead_step_file_detail_<%=index%>')); updateDivScroll('dead_steps_data', 'dead_steps_table');">
|
|
29
|
+
<% end %>
|
|
30
|
+
<td class="red_number" style="width: 1%"><%= dead_steps[file_name].size %></td>
|
|
31
|
+
<td id="dead_step_file_<%= index %>">
|
|
32
|
+
<%= file_name.gsub(cuke_sniffer.step_definitions_location, "") %>
|
|
33
|
+
</td>
|
|
34
|
+
<td style="text-align: right; width:5%" valign="top">
|
|
35
|
+
<a href="file:///<%= file_name.gsub(/:\d+$/, "") %>.rb" title="Note: Links to file on system this report was generated.">
|
|
36
|
+
(open file)
|
|
37
|
+
</a>
|
|
38
|
+
</td>
|
|
39
|
+
</tr>
|
|
40
|
+
|
|
41
|
+
<% if index.odd? %>
|
|
42
|
+
<tr class="blue_row dead_step_detail" id="dead_step_file_detail_<%= index %>" style="display: none;">
|
|
43
|
+
<% else %>
|
|
44
|
+
<tr class="white_row dead_step_detail" id="dead_step_file_detail_<%= index %>" style="display: none;">
|
|
45
|
+
<% end %>
|
|
46
|
+
<td colspan="3">
|
|
47
|
+
<table id="sub_detail_table_<%= index %>" style="width:100%;">
|
|
48
|
+
<% dead_steps[file_name].each do |regex| %>
|
|
49
|
+
<tr>
|
|
50
|
+
<td style="text-indent: 8px;"><%= regex.match(/:(?<regex>.*)/)[:regex] %></td>
|
|
51
|
+
<td style="text-align: right;">Line: <%= regex.match(/(?<line>\d+):/)[:line] %></td>
|
|
52
|
+
</tr>
|
|
53
|
+
<% end %>
|
|
54
|
+
</table>
|
|
55
|
+
</td>
|
|
56
|
+
</tr>
|
|
57
|
+
|
|
58
|
+
<% index += 1 %>
|
|
59
|
+
<% end %>
|
|
60
|
+
</table>
|
|
61
|
+
<% end %>
|
|
62
|
+
</div>
|
|
36
63
|
<br style="clear:both">
|
|
@@ -1,60 +1,114 @@
|
|
|
1
|
-
<div class="title" onclick="toggleById('features_data', this)">
|
|
1
|
+
<div class="title" onclick="toggleById('features_data', this);updateDisplayStatus(document.getElementById('feature_function_bar'));updateDivScroll('features_data', 'features_table');">
|
|
2
2
|
Features +
|
|
3
3
|
</div>
|
|
4
|
-
|
|
4
|
+
|
|
5
|
+
<div id="feature_function_bar" class="function_bar">
|
|
6
|
+
<input type="button" class="function_button" value="Collapse All" onclick="collapseAll('feature_detail')" />
|
|
7
|
+
<input type="button" class="function_button" value="Expand All" onclick="expandAllShiftingRows('feature_detail')" />
|
|
8
|
+
</div>
|
|
9
|
+
|
|
10
|
+
<div class="new_sub_section" id="features_data">
|
|
5
11
|
<% if cuke_sniffer.features.count == 0 %>
|
|
6
12
|
<div class="empty_set_message">There were no Features to sniff in '<%= cuke_sniffer.features_location %>'!</div>
|
|
7
13
|
<% elsif cuke_sniffer.features.count >= 1 && cuke_sniffer.scenarios.count >=1 && cuke_sniffer.summary[:features][:total_score] ==0 && cuke_sniffer.summary[:scenarios][:total_score] ==0 %>
|
|
8
14
|
<div class="empty_set_message">Excellent! No smells found for Features and Scenarios!</div>
|
|
9
15
|
<% end %>
|
|
10
|
-
<% index = 0 %>
|
|
11
|
-
<% cuke_sniffer.features.each do |feature| %>
|
|
12
|
-
<% next if feature.total_score <= 0 %>
|
|
13
|
-
<div style="cursor: pointer; padding-top: 3px" onclick="updateDisplayStatus(document.getElementById('feature_<%=index%>'))">
|
|
14
|
-
<div class="red_number" style="float:left"><%= feature.total_score + feature.scenarios_score %></div>
|
|
15
|
-
<div class="notes"><%= feature.name %></div>
|
|
16
|
-
</div>
|
|
17
|
-
<div id="feature_<%= index %>" class="sub_section">
|
|
18
|
-
<div style="text-indent: 4%; float: left; width: 45%; border-right: 2px solid #add8e6; padding-bottom: 4px">
|
|
19
|
-
<% if feature.rules_hash.empty? %>
|
|
20
|
-
<pre style="margin: 0;"> </pre>
|
|
21
|
-
<% end %>
|
|
22
|
-
<% feature.rules_hash.each_key do |rule| %>
|
|
23
|
-
<pre style="margin: 0;"><%= rule %></pre>
|
|
24
|
-
<% end %>
|
|
25
|
-
</div>
|
|
26
|
-
<div style="text-indent: 1%; padding-bottom: 4px">
|
|
27
|
-
<%= feature.location.gsub(cuke_sniffer.features_location, "") %>
|
|
28
|
-
</div>
|
|
29
|
-
<br style="clear:both">
|
|
30
16
|
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
|
|
49
|
-
|
|
50
|
-
|
|
51
|
-
|
|
52
|
-
|
|
53
|
-
|
|
54
|
-
|
|
55
|
-
|
|
56
|
-
|
|
57
|
-
|
|
58
|
-
|
|
17
|
+
<table id="features_table" border="0" cellspacing="0" style="width:100%;padding-left: 2px">
|
|
18
|
+
<% index = 0 %>
|
|
19
|
+
<% cuke_sniffer.features.each do |feature| %>
|
|
20
|
+
<% next if feature.total_score == 0 %>
|
|
21
|
+
<% if index.odd? %>
|
|
22
|
+
<tr class="notes blue_title_row" style="cursor: hand;" onclick="updateDisplayStatus(document.getElementById('feature_detail_<%=index%>'));updateDivScroll('features_data', 'features_table');">
|
|
23
|
+
<% else %>
|
|
24
|
+
<tr class="notes white_title_row" style="cursor: hand;" onclick="updateDisplayStatus(document.getElementById('feature_detail_<%=index%>'));updateDivScroll('features_data', 'features_table');">
|
|
25
|
+
<% end %>
|
|
26
|
+
<td valign="top" class="red_number" style="width:1%">
|
|
27
|
+
<%= feature.total_score %>
|
|
28
|
+
</td>
|
|
29
|
+
<td style="padding-left: 2px; width:100%;">
|
|
30
|
+
<% if feature.name.to_s.empty? %>
|
|
31
|
+
No Feature Description
|
|
32
|
+
<% else %>
|
|
33
|
+
<%= feature.name %>
|
|
34
|
+
<% end %>
|
|
35
|
+
</td>
|
|
36
|
+
</tr>
|
|
37
|
+
<% if index.odd? %>
|
|
38
|
+
<tr id="feature_detail_<%= index %>" class="blue_row feature_detail" style="display:none;">
|
|
39
|
+
<% else %>
|
|
40
|
+
<tr id="feature_detail_<%= index %>" class="white_row feature_detail" style="display:none;">
|
|
41
|
+
<% end %>
|
|
42
|
+
|
|
43
|
+
<td colspan="2">
|
|
44
|
+
<table border="0" cellspacing="0" class="colored_subsection">
|
|
45
|
+
<tr>
|
|
46
|
+
<td>
|
|
47
|
+
<% SummaryHelper::sort_improvement_list(feature.rules_hash).each do |phrase, count| %>
|
|
48
|
+
<div>
|
|
49
|
+
<div style="color:red; display:inline-block;">
|
|
50
|
+
<%= count %>
|
|
51
|
+
</div>
|
|
52
|
+
<div style="display:inline-block;">
|
|
53
|
+
<%= phrase %>
|
|
54
|
+
</div>
|
|
55
|
+
</div>
|
|
56
|
+
<% end %>
|
|
57
|
+
</td>
|
|
58
|
+
<td style="text-align: right;">
|
|
59
|
+
<a href="file:///<%= feature.location.gsub(/:\d+$/, "") %>" title="Note: Links to file on system this report was generated.">
|
|
60
|
+
<%= feature.location.gsub(cuke_sniffer.features_location, '') %>
|
|
61
|
+
</a>
|
|
62
|
+
</td>
|
|
63
|
+
</tr>
|
|
64
|
+
<% unless feature.background.nil? %>
|
|
65
|
+
<tr>
|
|
66
|
+
<td valign="top" style="border-top: 2px solid #509f50">
|
|
67
|
+
<div style="display:inline;" class="red_number"><%= feature.background.score %></div>
|
|
68
|
+
<div style="display:inline; padding-left: 2px; width:100%;"><%= feature.background.type %>
|
|
69
|
+
: <%= feature.background.name %></div>
|
|
70
|
+
</td>
|
|
71
|
+
<td valign="top" style="padding-left:2px;border-top: 2px solid #509f50;">
|
|
72
|
+
<% SummaryHelper::sort_improvement_list(feature.background.rules_hash).each do |phrase, count| %>
|
|
73
|
+
<div>
|
|
74
|
+
<div style="color:red; display:inline-block;">
|
|
75
|
+
<%= count %>
|
|
76
|
+
</div>
|
|
77
|
+
<div style="display:inline-block; padding-left:2px;">
|
|
78
|
+
<%= phrase %>
|
|
79
|
+
</div>
|
|
80
|
+
</div>
|
|
81
|
+
<% end %>
|
|
82
|
+
</td>
|
|
83
|
+
</tr>
|
|
84
|
+
<% end %>
|
|
85
|
+
<% feature.scenarios.each do |scenario| %>
|
|
86
|
+
<% next if scenario.score == 0 %>
|
|
87
|
+
<tr>
|
|
88
|
+
<td valign="top" style="border-top: 2px solid #509f50">
|
|
89
|
+
<div style="display:inline;" class="red_number"><%= scenario.score %></div>
|
|
90
|
+
<div style="display:inline; padding-left: 2px; width:100%;"><%= scenario.type %>
|
|
91
|
+
: <%= scenario.name %></div>
|
|
92
|
+
</td>
|
|
93
|
+
<td valign="top" style="padding-left:2px;border-top: 2px solid #509f50">
|
|
94
|
+
<% SummaryHelper::sort_improvement_list(scenario.rules_hash).each do |phrase, count| %>
|
|
95
|
+
<div>
|
|
96
|
+
<div style="color:red; display:inline-block;">
|
|
97
|
+
<%= count %>
|
|
98
|
+
</div>
|
|
99
|
+
<div style="display:inline-block;">
|
|
100
|
+
<%= phrase %>
|
|
101
|
+
</div>
|
|
102
|
+
</div>
|
|
103
|
+
<% end %>
|
|
104
|
+
</td>
|
|
105
|
+
</tr>
|
|
106
|
+
<% end %>
|
|
107
|
+
</table>
|
|
108
|
+
</td>
|
|
109
|
+
</tr>
|
|
110
|
+
<% index += 1 %>
|
|
111
|
+
<% end %>
|
|
112
|
+
</table>
|
|
59
113
|
</div>
|
|
60
114
|
<br style="clear:both">
|
|
@@ -1,49 +1,79 @@
|
|
|
1
|
-
<div class="title" onclick="toggleById('hooks_data', this)">
|
|
2
|
-
Hooks +
|
|
3
|
-
</div>
|
|
4
|
-
|
|
5
|
-
|
|
6
|
-
|
|
7
|
-
|
|
8
|
-
|
|
9
|
-
|
|
10
|
-
|
|
11
|
-
<% cuke_sniffer.hooks.
|
|
12
|
-
|
|
13
|
-
|
|
14
|
-
|
|
15
|
-
|
|
16
|
-
|
|
17
|
-
|
|
18
|
-
|
|
19
|
-
<%
|
|
20
|
-
<%
|
|
21
|
-
|
|
22
|
-
<%
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
|
|
27
|
-
|
|
28
|
-
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
|
|
33
|
-
|
|
34
|
-
|
|
35
|
-
|
|
36
|
-
<% hook.
|
|
37
|
-
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
|
|
46
|
-
|
|
47
|
-
|
|
48
|
-
</
|
|
1
|
+
<div class="title" onclick="toggleById('hooks_data', this);updateDisplayStatus(document.getElementById('hooks_function_bar'));updateDivScroll('hooks_data', 'hooks_table');">
|
|
2
|
+
Hooks +
|
|
3
|
+
</div>
|
|
4
|
+
|
|
5
|
+
<div id="hooks_function_bar" class="function_bar">
|
|
6
|
+
<input type="button" class="function_button" value="Collapse All" onclick="collapseAll('hook_detail')" />
|
|
7
|
+
<input type="button" class="function_button" value="Expand All" onclick="expandAllShiftingRows('hook_detail')" />
|
|
8
|
+
</div>
|
|
9
|
+
|
|
10
|
+
<div class="new_sub_section" id="hooks_data">
|
|
11
|
+
<% if cuke_sniffer.hooks.count == 0 %>
|
|
12
|
+
<div class="empty_set_message">There were no Hooks to sniff in '<%= cuke_sniffer.hooks_location %>'!</div>
|
|
13
|
+
<% elsif cuke_sniffer.hooks.count >= 1 && cuke_sniffer.summary[:hooks][:total_score] ==0 %>
|
|
14
|
+
<div class="empty_set_message">Excellent! No smells found for Hooks!</div>
|
|
15
|
+
<% end %>
|
|
16
|
+
<table id="hooks_table" style="width:100%;" border="0" cellspacing="0">
|
|
17
|
+
<% index = 0 %>
|
|
18
|
+
<% cuke_sniffer.hooks.each do |hook| %>
|
|
19
|
+
<% next if hook.score <= 0 %>
|
|
20
|
+
<% if index.odd? %>
|
|
21
|
+
<tr class="notes blue_title_row" onclick="updateDisplayStatus(document.getElementById('hook_detail_<%=index%>'));updateDivScroll('hooks_data', 'hooks_table');">
|
|
22
|
+
<% else %>
|
|
23
|
+
<tr class="notes white_title_row" onclick="updateDisplayStatus(document.getElementById('hook_detail_<%=index%>'));updateDivScroll('hooks_data', 'hooks_table');">
|
|
24
|
+
<% end %>
|
|
25
|
+
<td id="hook_score_<%= index %>" valign="top" style="width:1%; color: red">
|
|
26
|
+
<%= hook.score %>
|
|
27
|
+
</td>
|
|
28
|
+
<td id="hook_name_<%= index %>" style="text-indent: 2px;">
|
|
29
|
+
<% tags = "(" %>
|
|
30
|
+
<% hook.tags.each do |tag| %>
|
|
31
|
+
<% tags << tag %>
|
|
32
|
+
<% tags << ", " unless tag == hook.tags.last %>
|
|
33
|
+
<% end %>
|
|
34
|
+
<% tags += ")" %>
|
|
35
|
+
<% parameters = "| " %>
|
|
36
|
+
<% hook.parameters.each do |parameter| %>
|
|
37
|
+
<% parameters << parameter %>
|
|
38
|
+
<% parameters << ", " unless parameter == hook.parameters.last %>
|
|
39
|
+
<% end %>
|
|
40
|
+
<% parameters += " |" %>
|
|
41
|
+
<% name = hook.type %>
|
|
42
|
+
<% name += tags unless tags == "()" %>
|
|
43
|
+
<% name += " do " %>
|
|
44
|
+
<% name += parameters unless parameters == "| |" %>
|
|
45
|
+
<%= name %>
|
|
46
|
+
</td>
|
|
47
|
+
</td>
|
|
48
|
+
</tr>
|
|
49
|
+
|
|
50
|
+
<% if index.odd? %>
|
|
51
|
+
<tr id="hook_detail_<%= index %>" class="blue_row hook_detail" style="display: none;">
|
|
52
|
+
<% else %>
|
|
53
|
+
<tr id="hook_detail_<%= index %>" class="white_row hook_detail" style="display: none;">
|
|
54
|
+
<% end %>
|
|
55
|
+
|
|
56
|
+
<td colspan="2">
|
|
57
|
+
<% SummaryHelper::sort_improvement_list(hook.rules_hash).each do |phrase, count| %>
|
|
58
|
+
<div style="text-indent: 10px;">
|
|
59
|
+
<div style="color:red; display:inline-block;">
|
|
60
|
+
<%= count %>
|
|
61
|
+
</div>
|
|
62
|
+
<div style="display:inline-block;">
|
|
63
|
+
<%= phrase %>
|
|
64
|
+
</div>
|
|
65
|
+
</div>
|
|
66
|
+
<% end %>
|
|
67
|
+
</td>
|
|
68
|
+
<td valign="top" style="text-align: right">
|
|
69
|
+
<a href="file:///<%= hook.location.gsub(/:\d+$/, "") %>" title="Note: Links to file on system this report was generated.">
|
|
70
|
+
<%= hook.location.gsub(cuke_sniffer.hooks_location, '') %>
|
|
71
|
+
</a>
|
|
72
|
+
</td>
|
|
73
|
+
</tr>
|
|
74
|
+
|
|
75
|
+
<% index += 1 %>
|
|
76
|
+
<% end %>
|
|
77
|
+
</table>
|
|
78
|
+
</div>
|
|
49
79
|
<br style="clear:both">
|