coverband 5.2.4 → 5.2.5.rc.2

data/test/forked/rails_route_tracker_stack_test.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+require File.expand_path("../rails_test_helper", File.dirname(__FILE__))
+
+class RailsRoutesWithoutConfigStackTest < Minitest::Test
+  def setup
+    super
+    setup_server
+  end
+
+  def teardown
+    super
+    shutdown_server
+  end
+
+  test "check route tracker" do
+    output = `sleep 7 && curl http://localhost:9999/dummy_view/show`
+    assert output.match(/rendered view/)
+    assert output.match(/I am no dummy view tracker text/)
+    output = `sleep 2 && curl http://localhost:9999/coverage/route_tracker`
+    assert output.match(/Used Routes: \(1\)/)
+    assert output.match(/dummy_view\/show/)
+    assert output.match(/GET/)
+  end
+
+  private
+
+  # NOTE: We aren't leveraging Capybara because it loads all of our other test helpers and such,
+  # which in turn Configures coverband making it impossible to test the configuration error
+  def setup_server
+    ENV["RAILS_ENV"] = "test"
+    require "rails"
+    fork do
+      exec "cd test/rails#{Rails::VERSION::MAJOR}_dummy && COVERBAND_TEST=test bundle exec rackup config.ru -p 9999 --pid /tmp/testrack.pid"
+    end
+  end
+
+  def shutdown_server
+    if File.exist?("/tmp/testrack.pid")
+      pid = `cat /tmp/testrack.pid`&.strip&.to_i
+      Process.kill("HUP", pid)
+      sleep 1
+    end
+  end
+end

data/test/forked/rails_view_tracker_stack_test.rb

@@ -17,7 +17,7 @@ class RailsWithoutConfigStackTest < Minitest::Test
     output = `sleep 7 && curl http://localhost:9999/dummy_view/show`
     assert output.match(/rendered view/)
     assert output.match(/I am no dummy view tracker text/)
-    output = `sleep 1 && curl http://localhost:9999/coverage/view_tracker`
+    output = `sleep 2 && curl http://localhost:9999/coverage/view_tracker`
     assert output.match(/Used Views: \(1\)/)
     assert output.match(/dummy_view\/show/)
   end

data/test/rails5_dummy/config/coverband.rb

@@ -10,8 +10,7 @@ Coverband.configure do |config|
   config.logger = Rails.logger
   config.verbose = true
   config.background_reporting_enabled = true
-  config.track_gems = true
-  config.gem_details = true
+  config.track_routes = true
   config.use_oneshot_lines_coverage = true if ENV["ONESHOT"]
   config.simulate_oneshot_lines_coverage = true if ENV["SIMULATE_ONESHOT"]
 end

data/test/rails7_dummy/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require_relative "config/application"
+
+Rails.application.load_tasks

data/test/rails7_dummy/app/controllers/dummy_controller.rb

@@ -0,0 +1,5 @@
+class DummyController < ActionController::Base
+  def show
+    render plain: "I am no dummy"
+  end
+end

data/test/rails7_dummy/app/controllers/dummy_view_controller.rb

@@ -0,0 +1,16 @@
+class DummyViewController < ActionController::Base
+  def show
+    @text = "I am no dummy view tracker text"
+    render layout: false
+  end
+
+  def show_haml
+    @text = "I am haml text"
+    render layout: false
+  end
+
+  def show_slim
+    @text = "I am slim text"
+    render layout: false
+  end
+end

data/test/rails7_dummy/app/views/dummy_view/show.html.erb

@@ -0,0 +1,5 @@
+rendered view
+
+<div>
+  <%= @text %>
+</div>

data/test/rails7_dummy/app/views/dummy_view/show_haml.html.haml

@@ -0,0 +1,4 @@
+rendered haml view
+
+#foo
+  = @text

data/test/rails7_dummy/app/views/dummy_view/show_slim.html.slim

@@ -0,0 +1,4 @@
+rendered slim view
+
+#content
+  = @text

data/test/rails7_dummy/config/application.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+require File.expand_path("boot", __dir__)
+
+require "rails"
+require "action_controller/railtie"
+require "coverband"
+Bundler.require(*Rails.groups)
+
+module Rails6Dummy
+  class Application < Rails::Application
+    config.eager_load = true
+    config.consider_all_requests_local = true
+  end
+end

data/test/rails7_dummy/config/boot.rb

@@ -0,0 +1,3 @@
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../Gemfile", __FILE__)
+
+require "bundler/setup" # Set up gems listed in the Gemfile.

data/test/rails7_dummy/config/coverband.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require_relative "../../rails5_dummy/config/coverband"

data/test/rails7_dummy/config/coverband_missing_redis.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require_relative "../../rails5_dummy/config/coverband_missing_redis"

data/test/rails7_dummy/config/environment.rb

@@ -0,0 +1,5 @@
+# Load the Rails application.
+require File.expand_path("../application", __FILE__)
+
+# Initialize the Rails application.
+Rails.application.initialize!

data/test/rails7_dummy/config/routes.rb

@@ -0,0 +1,7 @@
+Rails.application.routes.draw do
+  get "dummy/show"
+  get "dummy_view/show", to: "dummy_view#show"
+  get "dummy_view/show_haml", to: "dummy_view#show_haml"
+  get "dummy_view/show_slim", to: "dummy_view#show_slim"
+  mount Coverband::Reporters::Web.new, at: "/coverage"
+end

data/test/rails7_dummy/config/secrets.yml

@@ -0,0 +1,3 @@
+test:
+  secret_key_base: 8080f2894307a3dcf72127c0a279a729c58c7c10c11a15de761c8e16017e0e478647d1a7ac11bf143730cac7b6901fa000428c6b4873d9298250f8ca4657b5c6
+

data/test/rails7_dummy/config.ru

@@ -0,0 +1,4 @@
+# This file is used by Rack-based servers to start the application.
+
+require ::File.expand_path("../config/environment", __FILE__)
+run Rails.application

data/test/test_helper.rb

@@ -1,11 +1,21 @@
 # frozen_string_literal: true
 
+require "bigdecimal"
+
 original_verbosity = $VERBOSE
 $VERBOSE = nil
+
+if ENV["SKIP_SIMPLECOV"] || BigDecimal(RUBY_VERSION[0, 3]) >= BigDecimal("3.1")
+  $SKIP_SIMPLECOV = true
+end
+
 require "rubygems"
-require "pry-byebug" unless ENV["CI"]
-require "simplecov"
-require "coveralls"
+
+unless $SKIP_SIMPLECOV
+  require "simplecov"
+  require "coveralls"
+end
+
 require "minitest/autorun"
 require "minitest/stub_const"
 require "mocha/minitest"
@@ -25,11 +35,10 @@ require "coverband/utils/results"
 require "coverband/reporters/html_report"
 require "webmock/minitest"
 
-# require 'pry-byebug' unless ENV['CI'] # Ruby 2.3 on CI crashes on pry & JRuby doesn't support it
 require_relative "unique_files"
 $VERBOSE = original_verbosity
 
-unless ENV["ONESHOT"] || ENV["SIMULATE_ONESHOT"]
+unless ENV["ONESHOT"] || ENV["SIMULATE_ONESHOT"] || $SKIP_SIMPLECOV
   SimpleCov.formatter = Coveralls::SimpleCov::Formatter
   SimpleCov.start do
     add_filter "test/forked"

data/views/nav.erb

@@ -25,12 +25,17 @@
 </ul>
 
 <ul class="extra_tabs">
-  <% if nav_options[:show_coverage_link] %>
+  <% if nav_options[:active_link] %>
     <li><a href='<%= base_path %>'>Coverage</a></li>
   <% end %>
   <% if Coverband.configuration.track_views %>
-    <li class='<%= nav_options[:show_coverage_link] ? 'active' : '' %>'>
+    <li class='<%= nav_options[:active_link] == 'view_tracker' ? 'active' : '' %>'>
       <a href='<%= base_path %>view_tracker'>Views Tracker</a>
     </li>
   <% end %>
+    <% if Coverband.configuration.track_routes %>
+    <li class='<%= nav_options[:active_link] == 'route_tracker' ? 'active' : '' %>'>
+      <a href='<%= base_path %>route_tracker'>Route Tracker</a>
+    </li>
+  <% end %>
 </ul>

data/views/route_tracker.erb

@@ -0,0 +1,49 @@
+<!DOCTYPE html>
+<html xmlns='http://www.w3.org/1999/xhtml'>
+  <head>
+    <title>Coverband Info: <%= Coverband::VERSION %></title>
+    <meta http-equiv="content-type" content="text/html; charset=utf-8" />
+        <script src='<%= assets_path('dependencies.js') %>' type='text/javascript'></script>
+    <script src='<%= assets_path('application.js') %>' type='text/javascript'></script>
+    <link href='<%= assets_path('application.css') %>' media='screen, projection, print' rel='stylesheet' type='text/css'>
+    <link rel="icon" type="image/png" href="<%= assets_path('favicon.png') %>" />
+  </head>
+
+  <body>
+    <div id="wrapper" style="">
+      <%= display_nav(active_link: 'route_tracker') %>
+      <div id="content">
+        <% tracker = Coverband::Collectors::RouteTracker.new(store: Coverband.configuration.store) %>
+        <h4>
+          <% if Coverband.configuration.web_enable_clear %>
+            <%= button("#{base_path}clear_route_tracking", 'reset route tracker', delete: true) %>
+          <% end %>
+        </h4>
+        <h2>Unused Routes: (<%= tracker.unused_routes.length %>)</h2>
+        <p>These routes have not been rendered since recording started at <%= tracker.tracking_since %></p>
+        <ul>
+        <% tracker.unused_routes.each do |route| %>
+          <li class="unused-routes"><%= route %></li>
+        <% end %>
+        </ul>
+
+        <h2>Used Routes: (<%= tracker.used_routes.length %>)</h2>
+        <p>These routes have been rendered at least once</p>
+        <ul>
+        <% tracker.used_routes.each_pair do |route, time_at| %>
+          <li class="used-routes">
+            <%= route %>
+            <span class="last_seen_at">last activity recorded <%= Time.at(time_at.to_i)%></span>
+            <% if Coverband.configuration.web_enable_clear %>
+              <%= button("#{base_path}clear_route_tracking_route?route=#{route}", 'reset tracked route', delete: true) %>
+            <% end %>
+          </li>
+        <% end %>
+        </ul>
+      </div>
+      <div id="footer">
+        Generated by <a href="http://github.com/danmayer/coverband">Coverband</a> v<%= Coverband::VERSION %>
+      </div>
+    </div>
+  </body>
+</html>

data/views/view_tracker.erb

@@ -11,7 +11,7 @@
 
   <body>
     <div id="wrapper" style="">
-      <%= display_nav(show_coverage_link: true) %>
+      <%= display_nav(active_link: 'view_tracker') %>
       <div id="content">
         <% tracker = Coverband::Collectors::ViewTracker.new(store: Coverband.configuration.store) %>
         <h4>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 5.2.4
+  version: 5.2.5.rc.2
 platform: ruby
 authors:
 - Dan Mayer
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-07-08 00:00:00.000000000 Z
+date: 2022-11-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -280,8 +280,7 @@ files:
 - CODE_OF_CONDUCT.md
 - CONTRIBUTING.md
 - Gemfile
-- Gemfile.rails4
-- Gemfile.rails6
+- Gemfile.rails7
 - LICENSE
 - LICENSE.txt
 - README.md
@@ -302,6 +301,7 @@ files:
 - lib/coverband/at_exit.rb
 - lib/coverband/collectors/coverage.rb
 - lib/coverband/collectors/delta.rb
+- lib/coverband/collectors/route_tracker.rb
 - lib/coverband/collectors/view_tracker.rb
 - lib/coverband/collectors/view_tracker_service.rb
 - lib/coverband/configuration.rb
@@ -323,6 +323,7 @@ files:
 - lib/coverband/utils/jruby_ext.rb
 - lib/coverband/utils/lines_classifier.rb
 - lib/coverband/utils/method_definition_scanner.rb
+- lib/coverband/utils/rails6_ext.rb
 - lib/coverband/utils/railtie.rb
 - lib/coverband/utils/relative_file_converter.rb
 - lib/coverband/utils/result.rb
@@ -379,6 +380,7 @@ files:
 - test/coverband/at_exit_test.rb
 - test/coverband/collectors/coverage_test.rb
 - test/coverband/collectors/delta_test.rb
+- test/coverband/collectors/route_tracker_test.rb
 - test/coverband/collectors/view_tracker_test.rb
 - test/coverband/configuration_test.rb
 - test/coverband/coverband_test.rb
@@ -418,6 +420,7 @@ files:
 - test/forked/rails_full_stack_test.rb
 - test/forked/rails_full_stack_views_test.rb
 - test/forked/rails_rake_full_stack_test.rb
+- test/forked/rails_route_tracker_stack_test.rb
 - test/forked/rails_view_tracker_stack_test.rb
 - test/integration/full_stack_deferred_eager_test.rb
 - test/integration/full_stack_test.rb
@@ -465,6 +468,21 @@ files:
 - test/rails6_dummy/config/routes.rb
 - test/rails6_dummy/config/secrets.yml
 - test/rails6_dummy/tmp/.keep
+- test/rails7_dummy/Rakefile
+- test/rails7_dummy/app/controllers/dummy_controller.rb
+- test/rails7_dummy/app/controllers/dummy_view_controller.rb
+- test/rails7_dummy/app/views/dummy_view/show.html.erb
+- test/rails7_dummy/app/views/dummy_view/show_haml.html.haml
+- test/rails7_dummy/app/views/dummy_view/show_slim.html.slim
+- test/rails7_dummy/config.ru
+- test/rails7_dummy/config/application.rb
+- test/rails7_dummy/config/boot.rb
+- test/rails7_dummy/config/coverband.rb
+- test/rails7_dummy/config/coverband_missing_redis.rb
+- test/rails7_dummy/config/environment.rb
+- test/rails7_dummy/config/routes.rb
+- test/rails7_dummy/config/secrets.yml
+- test/rails7_dummy/tmp/.keep
 - test/rails_test_helper.rb
 - test/test_helper.rb
 - test/unique_files.rb
@@ -472,6 +490,7 @@ files:
 - views/file_list.erb
 - views/layout.erb
 - views/nav.erb
+- views/route_tracker.erb
 - views/settings.erb
 - views/source_file.erb
 - views/source_file_loader.erb
@@ -496,11 +515,11 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.2.32
 signing_key:
 specification_version: 4
 summary: Rack middleware to measure production code usage (LOC runtime usage)
@@ -521,6 +540,7 @@ test_files:
 - test/coverband/at_exit_test.rb
 - test/coverband/collectors/coverage_test.rb
 - test/coverband/collectors/delta_test.rb
+- test/coverband/collectors/route_tracker_test.rb
 - test/coverband/collectors/view_tracker_test.rb
 - test/coverband/configuration_test.rb
 - test/coverband/coverband_test.rb
@@ -560,6 +580,7 @@ test_files:
 - test/forked/rails_full_stack_test.rb
 - test/forked/rails_full_stack_views_test.rb
 - test/forked/rails_rake_full_stack_test.rb
+- test/forked/rails_route_tracker_stack_test.rb
 - test/forked/rails_view_tracker_stack_test.rb
 - test/integration/full_stack_deferred_eager_test.rb
 - test/integration/full_stack_test.rb
@@ -607,6 +628,21 @@ test_files:
 - test/rails6_dummy/config/routes.rb
 - test/rails6_dummy/config/secrets.yml
 - test/rails6_dummy/tmp/.keep
+- test/rails7_dummy/Rakefile
+- test/rails7_dummy/app/controllers/dummy_controller.rb
+- test/rails7_dummy/app/controllers/dummy_view_controller.rb
+- test/rails7_dummy/app/views/dummy_view/show.html.erb
+- test/rails7_dummy/app/views/dummy_view/show_haml.html.haml
+- test/rails7_dummy/app/views/dummy_view/show_slim.html.slim
+- test/rails7_dummy/config.ru
+- test/rails7_dummy/config/application.rb
+- test/rails7_dummy/config/boot.rb
+- test/rails7_dummy/config/coverband.rb
+- test/rails7_dummy/config/coverband_missing_redis.rb
+- test/rails7_dummy/config/environment.rb
+- test/rails7_dummy/config/routes.rb
+- test/rails7_dummy/config/secrets.yml
+- test/rails7_dummy/tmp/.keep
 - test/rails_test_helper.rb
 - test/test_helper.rb
 - test/unique_files.rb

data/Gemfile.rails6

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-source 'https://rubygems.org'
-
-# Specify your gem's dependencies in coverband.gemspec
-gemspec
-gem 'rails', '~>6'
-gem "haml"
-gem "slim"