routes_coverage 0.0.3 → 0.1.0

data/lib/routes_coverage/formatters/html_views/route_group.erb

@@ -9,6 +9,7 @@
        </span>
     </span> hits/route)
   </h2>
+
   <a name="<%= title_id %>"></a>
   <div>
     <b><%= result.total_count %></b> routes in total.
@@ -16,9 +17,11 @@
     <span class="green"><b><%= result.hit_routes_count %></b> covered</span> and
     <span class="red"><b><%= result.pending_routes.size %></b> missed </span>
   </div>
+
   <table class="route_list">
     <thead>
       <tr>
+        <%# NB: columns are also referenced from datatables config in js %>
         <th>Name</th>
         <th>Verb</th>
         <th>Path</th>

data/lib/routes_coverage/version.rb

@@ -1,3 +1,3 @@
 module RoutesCoverage
-  VERSION = "0.0.3"
+  VERSION = "0.1.0"
 end

data/lib/routes_coverage.rb

@@ -5,7 +5,7 @@ require "routes_coverage/middleware"
 require "routes_coverage/formatters/base"
 require "routes_coverage/formatters/summary_text"
 require "routes_coverage/formatters/full_text"
-require "routes_coverage/formatters/simplecov_html"
+require "routes_coverage/formatters/html"
 
 module RoutesCoverage
   class Railtie < ::Rails::Railtie
@@ -33,7 +33,7 @@ module RoutesCoverage
       @exclude_namespaces = []
       @minimum_coverage = 1
       @round_precision = 1
-      @format = :summary_text
+      @format = :html
       @groups = {}
     end
 
@@ -43,8 +43,8 @@ module RoutesCoverage
         Formatters::FullText
       when :summary_text
         Formatters::SummaryText
-      when :simplecov_html
-        Formatters::SimpleCovHtml
+      when :html, :simplecov_html
+        Formatters::Html
       when Formatters::Base
         format
       else
@@ -97,7 +97,21 @@ module RoutesCoverage
       ]
     }]
 
-    #TODO: group 'ungroupped'
+    if groups.size > 1
+      ungroupped_routes = all_routes.reject{|r|
+        groups.values.any?{|group_routes|
+          group_routes.all_routes.include? r
+        }
+      }
+
+      if ungroupped_routes.any?
+        groups["Ungroupped"] = Result.new(
+          ungroupped_routes,
+          Hash[@@route_hit_count.select{|r,_hits| ungroupped_routes.include? r}],
+          settings
+        )
+      end
+    end
 
     puts
     puts settings.formatter_class.new(all_result, groups, settings).format

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: routes_coverage
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - Vasily Fedoseyev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-14 00:00:00.000000000 Z
+date: 2017-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -83,6 +83,8 @@ files:
 - Rakefile
 - bin/console
 - bin/setup
+- compiled_assets/routes.css
+- compiled_assets/routes.js
 - gemfiles/.bundle/config
 - gemfiles/rails_40.gemfile
 - gemfiles/rails_40.gemfile.lock
@@ -102,9 +104,9 @@ files:
 - lib/routes_coverage/adapters/simplecov.rb
 - lib/routes_coverage/formatters/base.rb
 - lib/routes_coverage/formatters/full_text.rb
+- lib/routes_coverage/formatters/html.rb
 - lib/routes_coverage/formatters/html_views/layout.erb
 - lib/routes_coverage/formatters/html_views/route_group.erb
-- lib/routes_coverage/formatters/simplecov_html.rb
 - lib/routes_coverage/formatters/summary_text.rb
 - lib/routes_coverage/middleware.rb
 - lib/routes_coverage/result.rb

data/lib/routes_coverage/formatters/simplecov_html.rb

@@ -1,97 +0,0 @@
-require "erb"
-require "cgi"
-require "fileutils"
-require "digest/sha1"
-require "time"
-
-module RoutesCoverage
-  module Formatters
-    class SimpleCovHtml < Base
-      def format
-        #TODO: copy assets, if simplecov does not generate a report along us
-
-        # Dir[File.join(File.dirname(__FILE__), "../public/*")].each do |path|
-        #   FileUtils.cp_r(path, asset_output_path)
-        # end
-
-        routes_filename = "routes.html"
-
-        File.open(File.join(output_path, routes_filename), "wb") do |file|
-          file.puts template("layout").result(binding)
-        end
-
-        "Routes coverage is #{result.coverage}% Report generated to #{output_path}/#{routes_filename}"
-      end
-
-      private
-
-      def template(name)
-        ERB.new(File.read(File.join(File.dirname(__FILE__), "html_views", "#{name}.erb")))
-      end
-
-      def output_path
-        SimpleCov.coverage_path
-      end
-
-      def asset_output_path
-        return @asset_output_path if defined?(@asset_output_path) && @asset_output_path
-        @asset_output_path = File.join(output_path, "assets", SimpleCov::Formatter::HTMLFormatter::VERSION)
-        FileUtils.mkdir_p(@asset_output_path)
-        @asset_output_path
-      end
-
-      def assets_path(name)
-        File.join("./assets", SimpleCov::Formatter::HTMLFormatter::VERSION, name)
-      end
-
-      # Returns the html for the given source_file
-      def formatted_source_file(source_file)
-        template("source_file").result(binding)
-      end
-
-      # Returns a table containing the given source files
-      def route_group_result(title, result)
-        title_id = title.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-\_]/, "")
-        # Silence a warning by using the following variable to assign to itself:
-        # "warning: possibly useless use of a variable in void context"
-        # The variable is used by ERB via binding.
-        title_id = title_id
-        template("route_group").result(binding)
-      end
-
-      def coverage_css_class(covered_percent)
-        if covered_percent > 90
-          "green"
-        elsif covered_percent > 80
-          "yellow"
-        else
-          "red"
-        end
-      end
-
-      def strength_css_class(covered_strength)
-        if covered_strength > 1
-          "green"
-        elsif covered_strength == 1
-          "yellow"
-        else
-          "red"
-        end
-      end
-
-      def hits_css_class hits
-        hits > 0 ? 'green' : 'red'
-      end
-
-      # Return a (kind of) unique id for the source file given. Uses SHA1 on path for the id
-      def id(source_file)
-        Digest::SHA1.hexdigest(source_file.filename)
-      end
-
-      def timeago(time)
-        "<abbr class=\"timeago\" title=\"#{time.iso8601}\">#{time.iso8601}</abbr>"
-      end
-
-    end
-  end
-end