kjeldahl-metric_fu 1.1.5
Sign up to get free protection for your applications and to get access to all the features.
- data/HISTORY +159 -0
- data/MIT-LICENSE +22 -0
- data/Manifest.txt +25 -0
- data/README +1 -0
- data/Rakefile +18 -0
- data/TODO +9 -0
- data/lib/base/base_template.rb +146 -0
- data/lib/base/configuration.rb +187 -0
- data/lib/base/generator.rb +160 -0
- data/lib/base/graph.rb +38 -0
- data/lib/base/md5_tracker.rb +52 -0
- data/lib/base/report.rb +100 -0
- data/lib/generators/churn.rb +90 -0
- data/lib/generators/flay.rb +33 -0
- data/lib/generators/flog.rb +140 -0
- data/lib/generators/rcov.rb +103 -0
- data/lib/generators/reek.rb +37 -0
- data/lib/generators/roodi.rb +31 -0
- data/lib/generators/saikuro.rb +208 -0
- data/lib/generators/stats.rb +43 -0
- data/lib/graphs/flay_grapher.rb +35 -0
- data/lib/graphs/flog_grapher.rb +51 -0
- data/lib/graphs/grapher.rb +19 -0
- data/lib/graphs/rcov_grapher.rb +34 -0
- data/lib/graphs/reek_grapher.rb +44 -0
- data/lib/graphs/roodi_grapher.rb +34 -0
- data/lib/metric_fu.rb +30 -0
- data/lib/templates/awesome/awesome_template.rb +30 -0
- data/lib/templates/awesome/churn.html.erb +19 -0
- data/lib/templates/awesome/css/buttons.css +82 -0
- data/lib/templates/awesome/css/default.css +75 -0
- data/lib/templates/awesome/css/integrity.css +335 -0
- data/lib/templates/awesome/css/reset.css +7 -0
- data/lib/templates/awesome/flay.html.erb +27 -0
- data/lib/templates/awesome/flog.html.erb +47 -0
- data/lib/templates/awesome/index.html.erb +28 -0
- data/lib/templates/awesome/layout.html.erb +27 -0
- data/lib/templates/awesome/rcov.html.erb +36 -0
- data/lib/templates/awesome/reek.html.erb +34 -0
- data/lib/templates/awesome/roodi.html.erb +21 -0
- data/lib/templates/awesome/saikuro.html.erb +71 -0
- data/lib/templates/awesome/stats.html.erb +41 -0
- data/lib/templates/standard/churn.html.erb +31 -0
- data/lib/templates/standard/default.css +64 -0
- data/lib/templates/standard/flay.html.erb +34 -0
- data/lib/templates/standard/flog.html.erb +53 -0
- data/lib/templates/standard/index.html.erb +38 -0
- data/lib/templates/standard/rcov.html.erb +43 -0
- data/lib/templates/standard/reek.html.erb +42 -0
- data/lib/templates/standard/roodi.html.erb +29 -0
- data/lib/templates/standard/saikuro.html.erb +84 -0
- data/lib/templates/standard/standard_template.rb +26 -0
- data/lib/templates/standard/stats.html.erb +55 -0
- data/spec/base/base_template_spec.rb +161 -0
- data/spec/base/configuration_spec.rb +300 -0
- data/spec/base/generator_spec.rb +181 -0
- data/spec/base/graph_spec.rb +24 -0
- data/spec/base/md5_tracker_spec.rb +57 -0
- data/spec/base/report_spec.rb +139 -0
- data/spec/generators/churn_spec.rb +152 -0
- data/spec/generators/flay_spec.rb +104 -0
- data/spec/generators/flog_spec.rb +219 -0
- data/spec/generators/reek_spec.rb +60 -0
- data/spec/generators/saikuro_spec.rb +58 -0
- data/spec/generators/stats_spec.rb +74 -0
- data/spec/graphs/flay_grapher_spec.rb +44 -0
- data/spec/graphs/flog_grapher_spec.rb +61 -0
- data/spec/graphs/grapher_spec.rb +9 -0
- data/spec/graphs/rcov_grapher_spec.rb +44 -0
- data/spec/graphs/reek_grapher_spec.rb +53 -0
- data/spec/graphs/roodi_grapher_spec.rb +44 -0
- data/spec/resources/saikuro/app/controllers/sessions_controller.rb_cyclo.html +10 -0
- data/spec/resources/saikuro/app/controllers/users_controller.rb_cyclo.html +16 -0
- data/spec/resources/saikuro/index_cyclo.html +155 -0
- data/spec/resources/saikuro_sfiles/thing.rb_cyclo.html +11 -0
- data/spec/resources/yml/20090630.yml +7844 -0
- data/spec/spec.opts +8 -0
- data/spec/spec_helper.rb +6 -0
- data/tasks/metric_fu.rake +23 -0
- data/vendor/_fonts/monaco.ttf +0 -0
- data/vendor/saikuro/saikuro.rb +1219 -0
- metadata +181 -0
@@ -0,0 +1,60 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe Reek do
|
4
|
+
describe "analyze method" do
|
5
|
+
before :each do
|
6
|
+
MetricFu::Reek.stub!(:verify_dependencies!).and_return(true)
|
7
|
+
@lines = <<-HERE
|
8
|
+
"app/controllers/activity_reports_controller.rb" -- 4 warnings:
|
9
|
+
ActivityReportsController#authorize_user calls current_user.primary_site_ids multiple times (Duplication)
|
10
|
+
ActivityReportsController#authorize_user calls params[id] multiple times (Duplication)
|
11
|
+
ActivityReportsController#authorize_user calls params[primary_site_id] multiple times (Duplication)
|
12
|
+
ActivityReportsController#authorize_user has approx 6 statements (Long Method)
|
13
|
+
|
14
|
+
"app/controllers/application.rb" -- 1 warnings:
|
15
|
+
ApplicationController#start_background_task/block/block is nested (Nested Iterators)
|
16
|
+
|
17
|
+
"app/controllers/link_targets_controller.rb" -- 1 warnings:
|
18
|
+
LinkTargetsController#authorize_user calls current_user.role multiple times (Duplication)
|
19
|
+
|
20
|
+
"app/controllers/newline_controller.rb" -- 1 warnings:
|
21
|
+
NewlineController#some_method calls current_user.<< "new line\n" multiple times (Duplication)
|
22
|
+
HERE
|
23
|
+
MetricFu::Configuration.run {}
|
24
|
+
File.stub!(:directory?).and_return(true)
|
25
|
+
reek = MetricFu::Reek.new
|
26
|
+
reek.instance_variable_set(:@output, @lines)
|
27
|
+
@matches = reek.analyze
|
28
|
+
end
|
29
|
+
|
30
|
+
it "should find the code smell's method name" do
|
31
|
+
smell = @matches.first[:code_smells].first
|
32
|
+
smell[:method].should == "ActivityReportsController#authorize_user"
|
33
|
+
end
|
34
|
+
|
35
|
+
it "should find the code smell's type" do
|
36
|
+
smell = @matches[1][:code_smells].first
|
37
|
+
smell[:type].should == "Nested Iterators"
|
38
|
+
end
|
39
|
+
|
40
|
+
it "should find the code smell's message" do
|
41
|
+
smell = @matches[1][:code_smells].first
|
42
|
+
smell[:message].should == "is nested"
|
43
|
+
end
|
44
|
+
|
45
|
+
it "should find the code smell's type" do
|
46
|
+
smell = @matches.first
|
47
|
+
smell[:file_path].should == "app/controllers/activity_reports_controller.rb"
|
48
|
+
end
|
49
|
+
|
50
|
+
it "should NOT insert nil smells into the array when there's a newline in the method call" do
|
51
|
+
@matches.last[:code_smells].should == @matches.last[:code_smells].compact
|
52
|
+
@matches.last.should == {:file_path=>"app/controllers/newline_controller.rb",
|
53
|
+
:code_smells=>[{:type=>"Duplication",
|
54
|
+
:method=>"\"",
|
55
|
+
:message=>"multiple times"}]}
|
56
|
+
# Note: hopefully a temporary solution until I figure out how to deal with newlines in the method call more effectively -Jake 5/11/2009
|
57
|
+
end
|
58
|
+
end
|
59
|
+
|
60
|
+
end
|
@@ -0,0 +1,58 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe Saikuro do
|
4
|
+
describe "to_h method" do
|
5
|
+
before :all do
|
6
|
+
MetricFu::Configuration.run {}
|
7
|
+
File.stub!(:directory?).and_return(true)
|
8
|
+
saikuro = MetricFu::Saikuro.new
|
9
|
+
saikuro.stub!(:metric_directory).and_return(File.join(File.dirname(__FILE__), "..", "resources", "saikuro"))
|
10
|
+
saikuro.analyze
|
11
|
+
@output = saikuro.to_h
|
12
|
+
end
|
13
|
+
|
14
|
+
it "should find the filename of a file" do
|
15
|
+
@output[:saikuro][:files].first[:filename].should == 'users_controller.rb'
|
16
|
+
end
|
17
|
+
|
18
|
+
it "should find the name of the classes" do
|
19
|
+
@output[:saikuro][:classes].first[:name].should == "UsersController"
|
20
|
+
@output[:saikuro][:classes][1][:name].should == "SessionsController"
|
21
|
+
end
|
22
|
+
|
23
|
+
it "should put the most complex method first" do
|
24
|
+
@output[:saikuro][:methods].first[:name].should == "UsersController#create"
|
25
|
+
@output[:saikuro][:methods].first[:complexity].should == 4
|
26
|
+
end
|
27
|
+
|
28
|
+
it "should find the complexity of a method" do
|
29
|
+
@output[:saikuro][:methods].first[:complexity].should == 4
|
30
|
+
end
|
31
|
+
|
32
|
+
it "should find the lines of a method" do
|
33
|
+
@output[:saikuro][:methods].first[:lines].should == 15
|
34
|
+
end
|
35
|
+
end
|
36
|
+
|
37
|
+
describe "format_directories method" do
|
38
|
+
it "should format the directories" do
|
39
|
+
MetricFu::Configuration.run {}
|
40
|
+
File.stub!(:directory?).and_return(true)
|
41
|
+
saikuro = MetricFu::Saikuro.new
|
42
|
+
|
43
|
+
MetricFu.saikuro[:input_directory] = ["app", "lib"]
|
44
|
+
|
45
|
+
saikuro.format_directories.should == "\"app | lib\""
|
46
|
+
end
|
47
|
+
end
|
48
|
+
|
49
|
+
describe Saikuro::SFile do
|
50
|
+
describe "getting elements from a Saikuro result file" do
|
51
|
+
it "should parse nested START/END sections" do
|
52
|
+
path = File.join(File.dirname(__FILE__), "..", "resources", "saikuro_sfiles", "thing.rb_cyclo.html")
|
53
|
+
sfile = Saikuro::SFile.new path
|
54
|
+
sfile.elements.map { |e| e.complexity }.sort.should eql(["0","0","2"])
|
55
|
+
end
|
56
|
+
end
|
57
|
+
end
|
58
|
+
end
|
@@ -0,0 +1,74 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe Stats do
|
4
|
+
describe "emit method" do
|
5
|
+
it "should gather the raw data" do
|
6
|
+
MetricFu::Configuration.run {}
|
7
|
+
File.stub!(:directory?).and_return(true)
|
8
|
+
stats = MetricFu::Stats.new
|
9
|
+
stats.should_receive(:`).with("rake stats > tmp/metric_fu/scratch/stats/stats.txt")
|
10
|
+
stats.emit
|
11
|
+
end
|
12
|
+
end
|
13
|
+
|
14
|
+
describe "analyze method" do
|
15
|
+
before :each do
|
16
|
+
@lines = <<-HERE.gsub(/^\s*/, "")
|
17
|
+
+----------------------+-------+-------+---------+---------+-----+-------+
|
18
|
+
| Name | Lines | LOC | Classes | Methods | M/C | LOC/M |
|
19
|
+
+----------------------+-------+-------+---------+---------+-----+-------+
|
20
|
+
| Controllers | 470 | 382 | 7 | 53 | 7 | 5 |
|
21
|
+
| Helpers | 128 | 65 | 0 | 6 | 0 | 8 |
|
22
|
+
| Models | 351 | 285 | 9 | 31 | 3 | 7 |
|
23
|
+
| Libraries | 305 | 183 | 2 | 30 | 15 | 4 |
|
24
|
+
| Model specs | 860 | 719 | 0 | 2 | 0 | 357 |
|
25
|
+
| View specs | 0 | 0 | 0 | 0 | 0 | 0 |
|
26
|
+
| Controller specs | 1570 | 1308 | 1 | 10 | 10 | 128 |
|
27
|
+
| Helper specs | 191 | 172 | 0 | 0 | 0 | 0 |
|
28
|
+
| Library specs | 31 | 27 | 0 | 0 | 0 | 0 |
|
29
|
+
+----------------------+-------+-------+---------+---------+-----+-------+
|
30
|
+
| Total | 3906 | 3141 | 19 | 132 | 6 | 21 |
|
31
|
+
+----------------------+-------+-------+---------+---------+-----+-------+
|
32
|
+
Code LOC: 915 Test LOC: 2226 Code to Test Ratio: 1:2.4
|
33
|
+
|
34
|
+
HERE
|
35
|
+
MetricFu::Configuration.run {}
|
36
|
+
File.stub!(:directory?).and_return(true)
|
37
|
+
stats = MetricFu::Stats.new
|
38
|
+
File.should_receive(:open).and_return(mock("file", :read => @lines))
|
39
|
+
@results = stats.analyze
|
40
|
+
end
|
41
|
+
|
42
|
+
it "should get code Lines Of Code" do
|
43
|
+
@results[:codeLOC].should == 915
|
44
|
+
end
|
45
|
+
|
46
|
+
it "should get test Lines Of Code" do
|
47
|
+
@results[:testLOC].should == 2226
|
48
|
+
end
|
49
|
+
|
50
|
+
it "should get code to test ratio" do
|
51
|
+
@results[:code_to_test_ratio].should == 2.4
|
52
|
+
end
|
53
|
+
|
54
|
+
it "should get data on models" do
|
55
|
+
model_data = @results[:lines].find {|line| line[:name] == "Models"}
|
56
|
+
model_data[:classes].should == 9
|
57
|
+
model_data[:methods].should == 31
|
58
|
+
model_data[:loc].should == 285
|
59
|
+
model_data[:lines].should == 351
|
60
|
+
model_data[:methods_per_class].should == 3
|
61
|
+
model_data[:loc_per_method].should == 7
|
62
|
+
end
|
63
|
+
end
|
64
|
+
|
65
|
+
describe "to_h method" do
|
66
|
+
it "should put things into a hash" do
|
67
|
+
MetricFu::Configuration.run {}
|
68
|
+
File.stub!(:directory?).and_return(true)
|
69
|
+
stats = MetricFu::Stats.new
|
70
|
+
stats.instance_variable_set(:@stats, "the_stats")
|
71
|
+
stats.to_h[:stats].should == "the_stats"
|
72
|
+
end
|
73
|
+
end
|
74
|
+
end
|
@@ -0,0 +1,44 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe FlayGrapher do
|
4
|
+
before :each do
|
5
|
+
@flay_grapher = MetricFu::FlayGrapher.new
|
6
|
+
MetricFu.configuration
|
7
|
+
end
|
8
|
+
|
9
|
+
it "should respond to flay_score and labels" do
|
10
|
+
@flay_grapher.should respond_to(:flay_score)
|
11
|
+
@flay_grapher.should respond_to(:labels)
|
12
|
+
end
|
13
|
+
|
14
|
+
describe "responding to #initialize" do
|
15
|
+
it "should initialise flay_score and labels" do
|
16
|
+
@flay_grapher.flay_score.should == []
|
17
|
+
@flay_grapher.labels.should == {}
|
18
|
+
end
|
19
|
+
end
|
20
|
+
|
21
|
+
describe "responding to #get_metrics" do
|
22
|
+
before(:each) do
|
23
|
+
@metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
|
24
|
+
@date = "01022003"
|
25
|
+
end
|
26
|
+
|
27
|
+
it "should push 476 to flay_score" do
|
28
|
+
@flay_grapher.flay_score.should_receive(:push).with(476)
|
29
|
+
@flay_grapher.get_metrics(@metrics, @date)
|
30
|
+
end
|
31
|
+
|
32
|
+
it "should update labels with the date" do
|
33
|
+
@flay_grapher.labels.should_receive(:update).with({ 0 => "01022003" })
|
34
|
+
@flay_grapher.get_metrics(@metrics, @date)
|
35
|
+
end
|
36
|
+
end
|
37
|
+
|
38
|
+
describe "responding to #graph!" do
|
39
|
+
it "should write flay.png" do
|
40
|
+
@flay_grapher.graph!
|
41
|
+
lambda{ File.open(File.join(MetricFu.output_directory, 'flay.png')) }.should_not raise_error
|
42
|
+
end
|
43
|
+
end
|
44
|
+
end
|
@@ -0,0 +1,61 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe MetricFu::FlogGrapher do
|
4
|
+
before :each do
|
5
|
+
@flog_grapher = MetricFu::FlogGrapher.new
|
6
|
+
MetricFu.configuration
|
7
|
+
end
|
8
|
+
|
9
|
+
it "should respond to flog_total, flog_average and labels" do
|
10
|
+
@flog_grapher.should respond_to(:flog_average)
|
11
|
+
@flog_grapher.should respond_to(:labels)
|
12
|
+
@flog_grapher.should respond_to(:top_five_percent_average)
|
13
|
+
end
|
14
|
+
|
15
|
+
describe "responding to #initialize" do
|
16
|
+
it "should initialize top_five_percent_average, flog_average and labels" do
|
17
|
+
@flog_grapher.flog_average.should == []
|
18
|
+
@flog_grapher.labels.should == {}
|
19
|
+
@flog_grapher.top_five_percent_average.should == []
|
20
|
+
end
|
21
|
+
end
|
22
|
+
|
23
|
+
describe "responding to #get_metrics" do
|
24
|
+
before(:each) do
|
25
|
+
@metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
|
26
|
+
@date = "01022003"
|
27
|
+
end
|
28
|
+
|
29
|
+
it "should push to top_five_percent_average" do
|
30
|
+
average = (73.6 + 68.5 + 66.1 + 46.6 + 44.8 + 44.1 + 41.2 + 36.0) / 8.0
|
31
|
+
@flog_grapher.top_five_percent_average.should_receive(:push).with(average)
|
32
|
+
@flog_grapher.get_metrics(@metrics, @date)
|
33
|
+
end
|
34
|
+
|
35
|
+
it "should push 9.9 to flog_average" do
|
36
|
+
@flog_grapher.flog_average.should_receive(:push).with(9.9)
|
37
|
+
@flog_grapher.get_metrics(@metrics, @date)
|
38
|
+
end
|
39
|
+
|
40
|
+
it "should update labels with the date" do
|
41
|
+
@flog_grapher.labels.should_receive(:update).with({ 0 => "01022003" })
|
42
|
+
@flog_grapher.get_metrics(@metrics, @date)
|
43
|
+
end
|
44
|
+
end
|
45
|
+
|
46
|
+
describe "responding to #graph!" do
|
47
|
+
it "should write flog.png" do
|
48
|
+
@flog_grapher.graph!
|
49
|
+
lambda{ File.open(File.join(MetricFu.output_directory, 'flog.png')) }.should_not raise_error
|
50
|
+
end
|
51
|
+
|
52
|
+
it "should not fall over when given empty results" do
|
53
|
+
gruff_line = Gruff::Line.new(MetricFu.graph_size)
|
54
|
+
gruff_line.stub!(:write)
|
55
|
+
Gruff::Line.stub!(:new).and_return(gruff_line)
|
56
|
+
metrics_hash = { :flog => {:average => 0, :pages => [], :total => 0} }
|
57
|
+
@flog_grapher.get_metrics(metrics_hash ,"20090629")
|
58
|
+
@flog_grapher.graph!
|
59
|
+
end
|
60
|
+
end
|
61
|
+
end
|
@@ -0,0 +1,9 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
|
2
|
+
|
3
|
+
describe Grapher do
|
4
|
+
it "should blow up if gruff is not availible" do
|
5
|
+
Grapher.should_receive(:require).and_raise(LoadError)
|
6
|
+
Grapher.should_receive(:puts).with(/config\.graphs/)
|
7
|
+
lambda {Grapher.new}.should raise_error(LoadError)
|
8
|
+
end
|
9
|
+
end
|
@@ -0,0 +1,44 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe RcovGrapher do
|
4
|
+
before :each do
|
5
|
+
@rcov_grapher = MetricFu::RcovGrapher.new
|
6
|
+
MetricFu.configuration
|
7
|
+
end
|
8
|
+
|
9
|
+
it "should respond to rcov_percent and labels" do
|
10
|
+
@rcov_grapher.should respond_to(:rcov_percent)
|
11
|
+
@rcov_grapher.should respond_to(:labels)
|
12
|
+
end
|
13
|
+
|
14
|
+
describe "responding to #initialize" do
|
15
|
+
it "should initialise rcov_percent and labels" do
|
16
|
+
@rcov_grapher.rcov_percent.should == []
|
17
|
+
@rcov_grapher.labels.should == {}
|
18
|
+
end
|
19
|
+
end
|
20
|
+
|
21
|
+
describe "responding to #get_metrics" do
|
22
|
+
before(:each) do
|
23
|
+
@metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
|
24
|
+
@date = "01022003"
|
25
|
+
end
|
26
|
+
|
27
|
+
it "should push 49.6 to rcov_percent" do
|
28
|
+
@rcov_grapher.rcov_percent.should_receive(:push).with(49.6)
|
29
|
+
@rcov_grapher.get_metrics(@metrics, @date)
|
30
|
+
end
|
31
|
+
|
32
|
+
it "should update labels with the date" do
|
33
|
+
@rcov_grapher.labels.should_receive(:update).with({ 0 => "01022003" })
|
34
|
+
@rcov_grapher.get_metrics(@metrics, @date)
|
35
|
+
end
|
36
|
+
end
|
37
|
+
|
38
|
+
describe "responding to #graph!" do
|
39
|
+
it "should write rcov.png" do
|
40
|
+
@rcov_grapher.graph!
|
41
|
+
lambda{ File.open(File.join(MetricFu.output_directory, 'rcov.png')) }.should_not raise_error
|
42
|
+
end
|
43
|
+
end
|
44
|
+
end
|
@@ -0,0 +1,53 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe ReekGrapher do
|
4
|
+
before :each do
|
5
|
+
@reek_grapher = MetricFu::ReekGrapher.new
|
6
|
+
MetricFu.configuration
|
7
|
+
end
|
8
|
+
|
9
|
+
it "should respond to reek_count and labels" do
|
10
|
+
@reek_grapher.should respond_to(:reek_count)
|
11
|
+
@reek_grapher.should respond_to(:labels)
|
12
|
+
end
|
13
|
+
|
14
|
+
describe "responding to #initialize" do
|
15
|
+
it "should initialise reek_count and labels" do
|
16
|
+
@reek_grapher.reek_count.should == {}
|
17
|
+
@reek_grapher.labels.should == {}
|
18
|
+
end
|
19
|
+
end
|
20
|
+
|
21
|
+
describe "responding to #get_metrics" do
|
22
|
+
before(:each) do
|
23
|
+
@metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
|
24
|
+
@date = "01022003"
|
25
|
+
end
|
26
|
+
|
27
|
+
it "should set a hash of code smells to reek_count" do
|
28
|
+
@reek_grapher.get_metrics(@metrics, @date)
|
29
|
+
@reek_grapher.reek_count.should == {
|
30
|
+
"Uncommunicative Name" => [27],
|
31
|
+
"Feature Envy" => [20],
|
32
|
+
"Utility Function" => [15],
|
33
|
+
"Long Method" => [26],
|
34
|
+
"Nested Iterators" => [12],
|
35
|
+
"Control Couple" => [4],
|
36
|
+
"Duplication" => [48],
|
37
|
+
"Large Class" => [1]
|
38
|
+
}
|
39
|
+
end
|
40
|
+
|
41
|
+
it "should update labels with the date" do
|
42
|
+
@reek_grapher.labels.should_receive(:update).with({ 0 => "01022003" })
|
43
|
+
@reek_grapher.get_metrics(@metrics, @date)
|
44
|
+
end
|
45
|
+
end
|
46
|
+
|
47
|
+
describe "responding to #graph!" do
|
48
|
+
it "should write reek.png" do
|
49
|
+
@reek_grapher.graph!
|
50
|
+
lambda{ File.open(File.join(MetricFu.output_directory, 'reek.png')) }.should_not raise_error
|
51
|
+
end
|
52
|
+
end
|
53
|
+
end
|
@@ -0,0 +1,44 @@
|
|
1
|
+
require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
|
2
|
+
|
3
|
+
describe RoodiGrapher do
|
4
|
+
before :each do
|
5
|
+
@roodi_grapher = MetricFu::RoodiGrapher.new
|
6
|
+
MetricFu.configuration
|
7
|
+
end
|
8
|
+
|
9
|
+
it "should respond to roodi_count and labels" do
|
10
|
+
@roodi_grapher.should respond_to(:roodi_count)
|
11
|
+
@roodi_grapher.should respond_to(:labels)
|
12
|
+
end
|
13
|
+
|
14
|
+
describe "responding to #initialize" do
|
15
|
+
it "should initialise roodi_count and labels" do
|
16
|
+
@roodi_grapher.roodi_count.should == []
|
17
|
+
@roodi_grapher.labels.should == {}
|
18
|
+
end
|
19
|
+
end
|
20
|
+
|
21
|
+
describe "responding to #get_metrics" do
|
22
|
+
before(:each) do
|
23
|
+
@metrics = YAML::load(File.open(File.join(File.dirname(__FILE__), "..", "resources", "yml", "20090630.yml")))
|
24
|
+
@date = "01022003"
|
25
|
+
end
|
26
|
+
|
27
|
+
it "should push 13 to roodi_count" do
|
28
|
+
@roodi_grapher.roodi_count.should_receive(:push).with(13)
|
29
|
+
@roodi_grapher.get_metrics(@metrics, @date)
|
30
|
+
end
|
31
|
+
|
32
|
+
it "should update labels with the date" do
|
33
|
+
@roodi_grapher.labels.should_receive(:update).with({ 0 => "01022003" })
|
34
|
+
@roodi_grapher.get_metrics(@metrics, @date)
|
35
|
+
end
|
36
|
+
end
|
37
|
+
|
38
|
+
describe "responding to #graph!" do
|
39
|
+
it "should write rcov.png" do
|
40
|
+
@roodi_grapher.graph!
|
41
|
+
lambda{ File.open(File.join(MetricFu.output_directory, 'roodi.png')) }.should_not raise_error
|
42
|
+
end
|
43
|
+
end
|
44
|
+
end
|
@@ -0,0 +1,10 @@
|
|
1
|
+
-- START --
|
2
|
+
Type:Global Name: Complexity:0 Lines:1
|
3
|
+
-- END --
|
4
|
+
-- START SessionsController --
|
5
|
+
Type:Class Name:SessionsController Complexity:6 Lines:40
|
6
|
+
Type:Def Name:new Complexity:2 Lines:2
|
7
|
+
Type:Def Name:create Complexity:2 Lines:19
|
8
|
+
Type:Def Name:destroy Complexity:1 Lines:4
|
9
|
+
Type:Def Name:note_failed_signin Complexity:1 Lines:3
|
10
|
+
-- END SessionsController --
|
@@ -0,0 +1,16 @@
|
|
1
|
+
-- START UsersController --
|
2
|
+
Type:Class Name:UsersController Complexity:25 Lines:111
|
3
|
+
Type:Def Name:signup Complexity:1 Lines:3
|
4
|
+
Type:Def Name:new Complexity:1 Lines:2
|
5
|
+
Type:Def Name:edit Complexity:1 Lines:3
|
6
|
+
Type:Def Name:update Complexity:2 Lines:8
|
7
|
+
Type:Def Name:index Complexity:1 Lines:2
|
8
|
+
Type:Def Name:create Complexity:4 Lines:15
|
9
|
+
Type:Def Name:thank_you Complexity:2 Lines:10
|
10
|
+
Type:Def Name:destroy Complexity:1 Lines:3
|
11
|
+
Type:Def Name:add_primary_site Complexity:1 Lines:4
|
12
|
+
Type:Def Name:users_have_changed Complexity:3 Lines:8
|
13
|
+
Type:Def Name:after_create_page Complexity:2 Lines:6
|
14
|
+
Type:Def Name:sanitize_params Complexity:3 Lines:3
|
15
|
+
Type:Def Name:authorize_user Complexity:3 Lines:5
|
16
|
+
-- END UsersController --
|
@@ -0,0 +1,155 @@
|
|
1
|
+
<html><head><title>Index for cyclomatic complexity</title></head>
|
2
|
+
<style>
|
3
|
+
body {
|
4
|
+
margin: 20px;
|
5
|
+
padding: 0;
|
6
|
+
font-size: 12px;
|
7
|
+
font-family: bitstream vera sans, verdana, arial, sans serif;
|
8
|
+
background-color: #efefef;
|
9
|
+
}
|
10
|
+
|
11
|
+
table {
|
12
|
+
border-collapse: collapse;
|
13
|
+
/*border-spacing: 0;*/
|
14
|
+
border: 1px solid #666;
|
15
|
+
background-color: #fff;
|
16
|
+
margin-bottom: 20px;
|
17
|
+
}
|
18
|
+
|
19
|
+
table, th, th+th, td, td+td {
|
20
|
+
border: 1px solid #ccc;
|
21
|
+
}
|
22
|
+
|
23
|
+
table th {
|
24
|
+
font-size: 12px;
|
25
|
+
color: #fc0;
|
26
|
+
padding: 4px 0;
|
27
|
+
background-color: #336;
|
28
|
+
}
|
29
|
+
|
30
|
+
th, td {
|
31
|
+
padding: 4px 10px;
|
32
|
+
}
|
33
|
+
|
34
|
+
td {
|
35
|
+
font-size: 13px;
|
36
|
+
}
|
37
|
+
|
38
|
+
.class_name {
|
39
|
+
font-size: 17px;
|
40
|
+
margin: 20px 0 0;
|
41
|
+
}
|
42
|
+
|
43
|
+
.class_complexity {
|
44
|
+
margin: 0 auto;
|
45
|
+
}
|
46
|
+
|
47
|
+
.class_complexity>.class_complexity {
|
48
|
+
margin: 0;
|
49
|
+
}
|
50
|
+
|
51
|
+
.class_total_complexity, .class_total_lines, .start_token_count, .file_count {
|
52
|
+
font-size: 13px;
|
53
|
+
font-weight: bold;
|
54
|
+
}
|
55
|
+
|
56
|
+
.class_total_complexity, .class_total_lines {
|
57
|
+
color: #c00;
|
58
|
+
}
|
59
|
+
|
60
|
+
.start_token_count, .file_count {
|
61
|
+
color: #333;
|
62
|
+
}
|
63
|
+
|
64
|
+
.warning {
|
65
|
+
background-color: yellow;
|
66
|
+
}
|
67
|
+
|
68
|
+
.error {
|
69
|
+
background-color: #f00;
|
70
|
+
}
|
71
|
+
</style>
|
72
|
+
|
73
|
+
<body>
|
74
|
+
<h1>Index for cyclomatic complexity</h1>
|
75
|
+
<h2 class="class_name">Errors and Warnings</h2>
|
76
|
+
<table width="100%" border="1">
|
77
|
+
<tr><th>Class</th><th>Method</th><th>Complexity</th></tr>
|
78
|
+
|
79
|
+
<tr><td><a href="./app/controllers/activity_reports_controller.rb_cyclo.html">ActivityReportsController</a></td><td>authorize_user</td>
|
80
|
+
<td class="warning">6</td></tr>
|
81
|
+
<tr><td><a href="./app/controllers/primary_sites_controller.rb_cyclo.html">PrimarySitesController</a></td><td>create</td>
|
82
|
+
<td class="warning">5</td></tr>
|
83
|
+
<tr><td><a href="./app/controllers/primary_sites_controller.rb_cyclo.html">PrimarySitesController</a></td><td>save_link_targets</td>
|
84
|
+
<td class="warning">5</td></tr>
|
85
|
+
<tr><td><a href="./app/models/link_target.rb_cyclo.html">LinkTarget</a></td><td>update_backlink_status</td>
|
86
|
+
<td class="warning">5</td></tr>
|
87
|
+
</table>
|
88
|
+
<hr/>
|
89
|
+
<h2 class="class_name">Analyzed Files</h2>
|
90
|
+
<ul>
|
91
|
+
<li>
|
92
|
+
<p class="file_name"><a href="./app/models/user.rb_cyclo.html">app/models/user.rb</a>
|
93
|
+
</li>
|
94
|
+
<li>
|
95
|
+
<p class="file_name"><a href="./app/models/subscriber.rb_cyclo.html">app/models/subscriber.rb</a>
|
96
|
+
</li>
|
97
|
+
<li>
|
98
|
+
<p class="file_name"><a href="./app/models/primary_site.rb_cyclo.html">app/models/primary_site.rb</a>
|
99
|
+
</li>
|
100
|
+
<li>
|
101
|
+
<p class="file_name"><a href="./app/models/link_target.rb_cyclo.html">app/models/link_target.rb</a>
|
102
|
+
</li>
|
103
|
+
<li>
|
104
|
+
<p class="file_name"><a href="./app/models/comparison_site.rb_cyclo.html">app/models/comparison_site.rb</a>
|
105
|
+
</li>
|
106
|
+
<li>
|
107
|
+
<p class="file_name"><a href="./app/models/backlink_crawl_status.rb_cyclo.html">app/models/backlink_crawl_status.rb</a>
|
108
|
+
</li>
|
109
|
+
<li>
|
110
|
+
<p class="file_name"><a href="./app/models/backlink_crawl_notifier.rb_cyclo.html">app/models/backlink_crawl_notifier.rb</a>
|
111
|
+
</li>
|
112
|
+
<li>
|
113
|
+
<p class="file_name"><a href="./app/models/activity_report_measure.rb_cyclo.html">app/models/activity_report_measure.rb</a>
|
114
|
+
</li>
|
115
|
+
<li>
|
116
|
+
<p class="file_name"><a href="./app/models/activity_report.rb_cyclo.html">app/models/activity_report.rb</a>
|
117
|
+
</li>
|
118
|
+
<li>
|
119
|
+
<p class="file_name"><a href="./app/helpers/users_helper.rb_cyclo.html">app/helpers/users_helper.rb</a>
|
120
|
+
</li>
|
121
|
+
<li>
|
122
|
+
<p class="file_name"><a href="./app/helpers/sessions_helper.rb_cyclo.html">app/helpers/sessions_helper.rb</a>
|
123
|
+
</li>
|
124
|
+
<li>
|
125
|
+
<p class="file_name"><a href="./app/helpers/primary_sites_helper.rb_cyclo.html">app/helpers/primary_sites_helper.rb</a>
|
126
|
+
</li>
|
127
|
+
<li>
|
128
|
+
<p class="file_name"><a href="./app/helpers/link_targets_helper.rb_cyclo.html">app/helpers/link_targets_helper.rb</a>
|
129
|
+
</li>
|
130
|
+
<li>
|
131
|
+
<p class="file_name"><a href="./app/helpers/application_helper.rb_cyclo.html">app/helpers/application_helper.rb</a>
|
132
|
+
</li>
|
133
|
+
<li>
|
134
|
+
<p class="file_name"><a href="./app/controllers/users_controller.rb_cyclo.html">app/controllers/users_controller.rb</a>
|
135
|
+
</li>
|
136
|
+
<li>
|
137
|
+
<p class="file_name"><a href="./app/controllers/sessions_controller.rb_cyclo.html">app/controllers/sessions_controller.rb</a>
|
138
|
+
</li>
|
139
|
+
<li>
|
140
|
+
<p class="file_name"><a href="./app/controllers/primary_sites_controller.rb_cyclo.html">app/controllers/primary_sites_controller.rb</a>
|
141
|
+
</li>
|
142
|
+
<li>
|
143
|
+
<p class="file_name"><a href="./app/controllers/link_targets_controller.rb_cyclo.html">app/controllers/link_targets_controller.rb</a>
|
144
|
+
</li>
|
145
|
+
<li>
|
146
|
+
<p class="file_name"><a href="./app/controllers/bookmarklet_integration_controller.rb_cyclo.html">app/controllers/bookmarklet_integration_controller.rb</a>
|
147
|
+
</li>
|
148
|
+
<li>
|
149
|
+
<p class="file_name"><a href="./app/controllers/application.rb_cyclo.html">app/controllers/application.rb</a>
|
150
|
+
</li>
|
151
|
+
<li>
|
152
|
+
<p class="file_name"><a href="./app/controllers/activity_reports_controller.rb_cyclo.html">app/controllers/activity_reports_controller.rb</a>
|
153
|
+
</li>
|
154
|
+
</ul>
|
155
|
+
</body></html>
|
@@ -0,0 +1,11 @@
|
|
1
|
+
-- START Mod --
|
2
|
+
Type:Module Name:Mod Complexity:2 Lines:7
|
3
|
+
Type:Def Name:self.included Complexity:2 Lines:5
|
4
|
+
-- START --
|
5
|
+
Type:Class Name: Complexity:0 Lines:1
|
6
|
+
-- END --
|
7
|
+
Type:Def Name:update Complexity:1 Lines:1
|
8
|
+
-- END Mod --
|
9
|
+
-- START Thing --
|
10
|
+
Type:Class Name:Thing Complexity:0 Lines:2
|
11
|
+
-- END Thing --
|