@@ -64,6 +64,8 @@ def run_reek(file, configuration) (0..2).to_a.each do |index| notifier_class.new("REEK WARNING", "#{analyzer.code_smells[index]}", REEK_ICON).notify end + analyzer.to_html + Utils.run_command("firefox #{Utils::WORK_DIR}/reek.html") end end lib/continuous4r.watchr.rb @@ -1,4 +1,5 @@ require 'utils.rb' +require 'reek_builder.rb' # Analyseur de qualite de code Ruby # Author:: Vincent Dubois # Date : 05 octobre 2009 - version 0.0.5 @@ -18,15 +19,30 @@ class ReekAnalyzer reek_extracted_lines.each do |line| extract_content(line) unless line.strip.empty? end - File.delete("reek.log") + #File.delete("reek.log") + end + + # extracting HTML for reek + def to_html + File.open("#{Utils::WORK_DIR}/reek.html", "wb") do |file| + file.write("<div id='bodyColumn'><div class='contentBox'>") + file.write("<div class='section'><a name='reek'></a><h2>Reek Results for #{@file}</h2>") + file.write("<p><a href='http://reek.rubyforge.org/' target='_blank'>Reek</a> detects common code smells in ruby code.</p>") + #file.write(Utils.heading_for_builder("Common code smells freeness : #{"%0.2f" % ReekBuilder.percent}%", ReekBuilder.percent) %> + file.write("<table class='bodyTable'><tr><th>Code Smell</th></tr>") + @code_smells.each_with_index do |code_smell, index| + file.write("<tr class='#{index % 2 == 0 ? 'a' : 'b'}'><td>#{code_smell}</td></tr>") + end + file.write("</table><p>Generated on #{Time.now.localtime}</p></div></div></div>") + end end private # extracting content logic from flog log file def extract_content(str) - str_splitted = str.split(/ /) - @code_smells << str_splitted[0] + @code_smells << str.strip if str.match(/^ /) end + end lib/reek_analyzer.rb @@ -6,19 +6,23 @@ class ReekBuilder # Implementation de la construction de la tache - def build(project_name, auto_install, proxy_option) + def build(project_name, auto_install, proxy_option, file = nil) # On verifie la presence de reek Utils.verify_gem_presence("reek", auto_install, proxy_option) # On lance la generation puts " Building reek report..." files = Array.new - files << Dir.glob("app/controllers/*.rb") - files << Dir.glob("app/helpers/*.rb") - files << Dir.glob("app/models/*.rb") - files << Dir.glob("lib/**/*.rb") - files << Dir.glob("test/**/*.rb") - files << Dir.glob("spec/**/*.rb") - files.flatten! + if file.nil? + files << Dir.glob("app/controllers/*.rb") + files << Dir.glob("app/helpers/*.rb") + files << Dir.glob("app/models/*.rb") + files << Dir.glob("lib/**/*.rb") + files << Dir.glob("test/**/*.rb") + files << Dir.glob("spec/**/*.rb") + files.flatten! + else + files << file + end reek_command = "reek" files.each do |file| reek_command += " '#{file}'" lib/reek_builder.rb @@ -1,6 +1,8 @@ # Module de fonctions utilitaires necessaire a la construction module Utils + WORK_DIR = "tmp/continuous4r" + # Methode qui permet de construire une page avec eruby, et de lever une exception au besoin def self.erb_run page, is_a_task = true page_file = File.open("#{Continuous4r::WORK_DIR}/#{page}.html", "w") lib/utils.rb dd07c0066c1d73b6855aa12f46f5b4deb1838e48 Vincent Dubois vdubois@sqli-1610-2.(none) http://github.com/vdubois/continuous4r/commit/6c33a8cc2bb7fe8ae2e18c74adade1350404b067 6c33a8cc2bb7fe8ae2e18c74adade1350404b067 2009-10-06T04:19:43-07:00 2009-10-06T04:19:43-07:00 reek analysis with html report 0a22ed6e1401e9d68c9be7d56486df7269a99ba3 Vincent Dubois vdubois@sqli-1610-2.(none)