solargraph_test_coverage 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 87bb6c9da590a67d83bc973958129c9a51048932b10e6f93f5b71f434dbc5c24
-  data.tar.gz: 860848733c64a6ecb211547511e189d6a88375785b0b6de000a32cb9f565a737
+  metadata.gz: 663088457d9de39c37ee21f06b2bd16be5a983025af2909f55acfa1d432be0f4
+  data.tar.gz: adc3da11d72363b2c1d3ca4a8db307609b34090d9bdf24643a411e47e0a8be09
 SHA512:
-  metadata.gz: 05b5cd3c1dd159e537d1832f4a4e95c69225908b9b58a94010de09997264b6262519f32d320e9a43e25b2c24911e59db25e3f89737db934fb4c1653736d14e3e
-  data.tar.gz: be451c86383a71ad4e462bef3b2bb5f807061d392ba5510cb2b0b957fa639192579c191e9fbb034b5066a389662ebd7422f2390e253941a9f6568ec07dcea980
+  metadata.gz: f3407c7390c65e09f14c8f872455317ef28e8dfc30592554d9087d5bf1fb34e30b802077b7098dd1adc7c1a2082ce29c37b999a358654a929c614d6952e376b4
+  data.tar.gz: 7c252dc55695be04bdfed84d654d3d1ab41143c4d7cd606488f5cdd56365bea3eee8968f8674d1aa4e1d38e254ece0170dd3f6567752bf46e6fe4451da783f8e

data/Gemfile

@@ -1,6 +1,8 @@
-source "https://rubygems.org"
+# frozen_string_literal: true
+
+source 'https://rubygems.org'
 
 # Specify your gem's dependencies in solargraph_test_coverage.gemspec
 gemspec
 
-gem "rake", "~> 12.0"
+gem 'rake', '~> 12.0'

data/Gemfile.lock

@@ -1,8 +1,7 @@
 PATH
   remote: .
   specs:
-    solargraph_test_coverage (0.1.1)
-      rspec-core (~> 3.10, >= 3.10.0)
+    solargraph_test_coverage (0.2.0)
       solargraph (~> 0.40, > 0.40)
 
 GEM
@@ -18,7 +17,7 @@ GEM
       rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
-    nokogiri (1.12.3-x86_64-darwin)
+    nokogiri (1.12.4-x86_64-darwin)
       racc (~> 1.4)
     parallel (1.20.1)
     parser (3.0.2.0)
@@ -30,9 +29,6 @@ GEM
     reverse_markdown (2.0.0)
       nokogiri
     rexml (3.2.5)
-    rspec-core (3.10.1)
-      rspec-support (~> 3.10.0)
-    rspec-support (3.10.2)
     rubocop (1.20.0)
       parallel (~> 1.10)
       parser (>= 3.0.0.0)

data/README.md

@@ -2,7 +2,14 @@
 
 Solargraph Plugin that provides a diagnostic reporter for unit-test coverage.
 
-Currently only works with RSpec. 
+Currently there are four different diagnostics:
+- Line is not covered
+- Branch is not covered (With a note if it's the 'THEN' or 'ELSE' branch)
+- Spec is failing (Error message will be on line 1)
+- Spec cannot be found (Error message will be on line 1)
+
+
+Currently expects RSpec/Rails.
 
 ## Installation
 
@@ -13,6 +20,7 @@ gem 'solargraph_test_coverage'
 ```
 
 Then add this to your `.solargraph.yml` config:
+
 ```yaml
 plugins:
   - solargraph_test_coverage
@@ -20,16 +28,22 @@ reporters:
   - test_coverage
 ```
 
-If you are using a `spec/rails_helper.rb` file, you will need to change:
-```ruby
-require "spec_helper"
+Additionally, a `test_coverage` key can be added to `.solargraph.yml`. The default values are shown below:
+
+```yaml
+test_coverage:
+  preload_rails: true
+  test_framework: rspec # or minitest
+  coverage:
+  - line
+  - branch
+  - test_failing
+  - test_missing
+  exclude_paths:
+  - 'app/controller'
 ```
 
-to
 
-```ruby
-require_relative "spec_helper"
-```
 
 And then execute:
 
@@ -41,18 +55,12 @@ Or install it yourself as:
 
 ## Usage
 
-With solargraph running and connected to your text editor, you should see diagnostic messages for test coverage.
-
-Currently there are four different messages:
-- Line is not covered
-- Branch is not covered (With a note if it's the 'THEN' or 'ELSE' branch)
-- Spec is failing (Error message will be on line 1)
-- Spec cannot be found (Error message will be on line 1)
+With solargraph running and connected to your text editor, you should see diagnostic messages for line coverage, branch coverage, test file presence, and test status (if it's failing)
 
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/ckolkey/solargraph_test_coverage.
+Bug reports and pull requests are welcome on GitHub at https://github.com/deepdivr/solargraph_test_coverage.
 
 
 ## License

data/Rakefile

@@ -1,2 +1,4 @@
-require "bundler/gem_tasks"
-task :default => :spec
+# frozen_string_literal: true
+
+require 'bundler/gem_tasks'
+task default: :spec

data/lib/solargraph_test_coverage/config.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+# frozen_string_literal = true
+
+module SolargraphTestCoverage
+  class Config
+    class << self
+      DEFAULTS = {
+        'preload_rails' => true,         # can be true or false - performance optimization
+        'test_framework' => 'rspec',     # can be 'rspec' or 'minitest'
+        'coverage' => [                  # All diagnostics are enabled by default
+          'line',                        # Specifying an array with fewer diagnostics will overwrite this
+          'branch',
+          'test_failing',
+          'test_missing'
+        ],
+        'exclude_paths' => [             # don't attempt to find/run a spec for files that match these paths
+          'app/controller'
+        ]
+      }.freeze
+
+      def preload_rails?
+        plugin_config['preload_rails']
+      end
+
+      def exclude_paths
+        plugin_config['exclude_paths']
+      end
+
+      def line_coverage?
+        plugin_config['coverage'].include? 'line'
+      end
+
+      def branch_coverage?
+        plugin_config['coverage'].include? 'branch'
+      end
+
+      def test_failing_coverage?
+        plugin_config['coverage'].include? 'test_failing'
+      end
+
+      def test_missing_coverage?
+        plugin_config['coverage'].include? 'test_missing'
+      end
+
+      def test_framework
+        plugin_config['test_framework']
+      end
+
+      def test_dir
+        case test_framework
+        when 'rspec'
+          'spec'
+        when 'minitest'
+          'test'
+        end
+      end
+
+      def test_file_suffix
+        case test_framework
+        when 'rspec'
+          '_spec.rb'
+        when 'minitest'
+          '_test.rb'
+        end
+      end
+
+      private
+
+      def plugin_config
+        @plugin_config ||= workspace_config.tap do |config|
+          DEFAULTS.each_key { |key| config[key] = DEFAULTS[key] unless config.key?(key) }
+        end
+      end
+
+      def workspace_config
+        Solargraph::Workspace::Config.new(Dir.pwd).raw_data.fetch('test_coverage', {})
+      end
+    end
+  end
+end

data/lib/solargraph_test_coverage/helpers.rb

@@ -3,14 +3,40 @@
 module SolargraphTestCoverage
   module Helpers
     #
-    # Attempts to find the corrosponding unit test file
-    # TODO: This could be a bit more robust, supporting gems (/lib/) and
-    #       minitest (/test/)
+    # Determines if a file should be excluded from running diagnostics
+    #
+    # @return [Boolean]
+    #
+    def exclude_file?(source_filename)
+      return true if source_filename.start_with? File.join(Dir.pwd, Config.test_dir)
+
+      Config.exclude_paths.each { |path| return true if source_filename.sub(Dir.pwd, '').include? path }
+
+      false
+    end
+
+    #
+    # Attempts to find the corresponding unit test file
     #
     # @return [String]
     #
-    def locate_test_file(source)
-      source.location.filename.sub('/app/', '/spec/').sub('.rb', '_spec.rb')
+    def test_file(source)
+      @test_file ||= begin
+        relative_filepath = source.location.filename.sub(Dir.pwd, '').split('/').reject(&:empty?)
+        relative_filepath[0] = Config.test_dir
+
+        File.join(Dir.pwd, relative_filepath.join('/'))
+            .sub('.rb', Config.test_file_suffix)
+      end
+    end
+
+    #
+    # Memoized wrapper for #run_test
+    #
+    # @return [Hash]
+    #
+    def results(source)
+      @results ||= run_test(source)
     end
 
     #
@@ -20,11 +46,11 @@ module SolargraphTestCoverage
     #
     # @return [Hash]
     #
-    def run_rspec(source, test_file)
+    def run_test(source)
       ForkProcess.run do
         Coverage.start(lines: true, branches: true)
-        exit_code = RSpec::Core::Runner.run([test_file])
-        Coverage.result.fetch(source.location.filename, {}).merge({ test_status: exit_code })
+        runner = TestRunner.with(test_file(source)).run!
+        Coverage.result.fetch(source.location.filename, {}).merge({ test_status: runner.passed? })
       end
     end
 
@@ -55,6 +81,22 @@ module SolargraphTestCoverage
       Branch.build_from(results).reject(&:covered?)
     end
 
+    #
+    # requires the specified testing framework
+    #
+    # @return [Boolean]
+    #
+    def self.require_testing_framework!
+      case Config.test_framework
+      when 'rspec'
+        require 'rspec/core'
+      when 'minitest'
+        require 'minitest/autorun'
+      else
+        raise UnknownTestingGem
+      end
+    end
+
     #
     # Only called once, when gem is loaded
     # Preloads rails via spec/rails_helper if Rails isn't already defined
@@ -63,21 +105,33 @@ module SolargraphTestCoverage
     # We rescue the LoadError since Solargraph would catch it otherwise,
     # and not load the plugin at all.
     #
+    # Adding the spec/ directory to the $LOAD_PATH lets 'require "spec_helper"'
+    # commonly found in rails_helper work.
+    #
     # If Coverage was started in Rails/Spec helper by SimpleCov,
     # calling Coverage.result after requiring stops and resets it.
     #
     # This is a bit experimental - I'm not sure if there will be downstream side-effects
     # from having Rails preloaded, but... we'll see :)
     #
+    # @return [Boolean]
+    #
     def self.preload_rails!
       return if defined?(Rails)
       return unless File.file?('spec/rails_helper.rb')
 
-      require File.expand_path("spec/rails_helper")
+      spec_path = File.join(Dir.pwd, 'spec')
+      $LOAD_PATH.unshift(spec_path) unless $LOAD_PATH.include?(spec_path)
+
+      require File.expand_path('spec/rails_helper')
       Coverage.result(stop: true, clear: true) if Coverage.running?
+
+      true
     rescue LoadError => e
       Solargraph::Logging.logger.warn "LoadError when trying to require 'rails_helper'"
       Solargraph::Logging.logger.warn "[#{e.class}] #{e.message}"
+
+      false
     end
   end
 end

data/lib/solargraph_test_coverage/test_coverage_reporter.rb

@@ -11,29 +11,46 @@ module SolargraphTestCoverage
     # @return [Array]
     #
     def diagnose(source, _api_map)
-      return [] if source.code.empty? || !source.location.filename.include?('/app/')
+      return [] if source.code.empty? || exclude_file?(source.location.filename)
+      return [test_missing_error(source)] unless File.file?(test_file(source))
 
-      test_file = locate_test_file(source)
-      return [no_test_file_error(source, test_file)] unless File.file?(test_file)
-
-      results  = run_rspec(source, test_file)
-      lines    = uncovered_lines(results).map { |line| line_coverage_warning(source, line) }
-      branches = uncovered_branches(results).map { |branch| branch_coverage_warning(source, branch.report) }
-      status   = results[:test_status].zero? ? [] : [test_failing_error(source)]
-
-      lines + branches + status
+      messages(source)
     rescue ChildFailedError
       []
     end
 
     private
 
+    def messages(source)
+      messages = [
+        line_warnings(source),
+        branch_warnings(source),
+        test_passing_error(source)
+      ]
+
+      messages.flatten.compact
+    end
+
+    def line_warnings(source)
+      uncovered_lines(results(source)).map { |line| line_coverage_warning(source, line) }
+    end
+
+    def branch_warnings(source)
+      uncovered_branches(results(source)).map { |branch| branch_coverage_warning(source, branch.report) }
+    end
+
+    def test_passing_error(source)
+      results(source)[:test_status] ? [] : [test_failing_error(source)]
+    end
+
     #
     # Creates LSP warning message for missing line coverage
     #
     # @return [Hash]
     #
     def line_coverage_warning(source, line)
+      return unless Config.line_coverage?
+
       {
         range: Solargraph::Range.from_to(line, 0, line, source.code.lines[line].length).to_hash,
         severity: Solargraph::Diagnostics::Severities::WARNING,
@@ -51,8 +68,11 @@ module SolargraphTestCoverage
     # @return [Hash]
     #
     def branch_coverage_warning(source, report)
+      return unless Config.branch_coverage?
+
       {
-        range: Solargraph::Range.from_to(report[:line] - 1, 0, report[:line] - 1, source.code.lines[report[:line] - 1].length).to_hash,
+        range: Solargraph::Range.from_to(report[:line] - 1, 0, report[:line] - 1,
+                                         source.code.lines[report[:line] - 1].length).to_hash,
         severity: Solargraph::Diagnostics::Severities::WARNING,
         source: 'TestCoverage',
         message: "'#{report[:type].upcase}' branch is missing test coverage"
@@ -65,11 +85,13 @@ module SolargraphTestCoverage
     # @return [Hash]
     #
     def test_failing_error(source)
+      return unless Config.test_failing_coverage?
+
       {
         range: Solargraph::Range.from_to(0, 0, 0, source.code.lines[0].length).to_hash,
         severity: Solargraph::Diagnostics::Severities::ERROR,
         source: 'TestCoverage',
-        message: "Unit Test is currently failing."
+        message: 'Unit Test is currently failing.'
       }
     end
 
@@ -78,12 +100,14 @@ module SolargraphTestCoverage
     #
     # @return [Hash]
     #
-    def no_test_file_error(source, test_file_location)
+    def test_missing_error(source)
+      return unless Config.test_missing_coverage?
+
       {
         range: Solargraph::Range.from_to(0, 0, 0, source.code.lines[0].length).to_hash,
         severity: Solargraph::Diagnostics::Severities::ERROR,
         source: 'TestCoverage',
-        message: "No unit test file found at #{test_file_location}"
+        message: "No unit test file found at #{test_file(source)}"
       }
     end
   end

data/lib/solargraph_test_coverage/test_runner.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module SolargraphTestCoverage
+  # Parent Class for different testing frameworks
+  class TestRunner
+    def self.with(test_file)
+      case Config.test_framework
+      when 'rspec'
+        RSpecRunner.new(test_file)
+      when 'minitest'
+        MinitestRunner.new(test_file)
+      end
+    end
+
+    def initialize(test_file)
+      @test_file = test_file
+      @result    = nil
+    end
+
+    def run!
+      @result = test_framework_runner.run([@test_file])
+      self
+    end
+
+    def passed?
+      raise NotImplementedError
+    end
+
+    def test_framework_runner
+      raise NotImplementedError
+    end
+  end
+
+  # Test Runner Subclass for RSpec
+  class RSpecRunner < TestRunner
+    def passed?
+      @result&.zero?
+    end
+
+    def test_framework_runner
+      RSpec::Core::Runner
+    end
+  end
+
+  # Test Runner Subclass for Minitest
+  class MinitestRunner < TestRunner
+    def passed?
+      @result
+    end
+
+    def test_framework_runner
+      Minitest
+    end
+  end
+end

data/lib/solargraph_test_coverage/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SolargraphTestCoverage
-  VERSION = "0.1.2"
+  VERSION = '0.2.0'
 end

data/lib/solargraph_test_coverage.rb

@@ -1,24 +1,23 @@
+# frozen_string_literal: true
+
 require 'solargraph_test_coverage/version'
 require 'solargraph_test_coverage/branch'
 require 'solargraph_test_coverage/fork_process'
 require 'solargraph_test_coverage/helpers'
+require 'solargraph_test_coverage/config'
+require 'solargraph_test_coverage/test_runner'
 require 'solargraph_test_coverage/test_coverage_reporter'
 
 require 'solargraph'
-require 'rspec/core'
 require 'coverage'
 
-# TODO
-# - Finding the right file could use some work
-# - Config Options for different errors (line, branch, test missing, test failing)
-# - Minitest/Cucumber Support
-# - app/lib support
-# - filter out stuff like controllers that wouldn't have tests
-# - Publish Gem to Rubygems
-
 module SolargraphTestCoverage
   class ChildFailedError < StandardError; end
-  Helpers.preload_rails!
+
+  class UnknownTestingGem < StandardError; end
+
+  Helpers.require_testing_framework!
+  Helpers.preload_rails! if Config.preload_rails?
 
   Solargraph::Diagnostics.register 'test_coverage', TestCoverageReporter
 end

data/solargraph_test_coverage.gemspec

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'lib/solargraph_test_coverage/version'
 
 Gem::Specification.new do |spec|
@@ -8,11 +10,11 @@ Gem::Specification.new do |spec|
 
   spec.summary       = 'Solargraph Plugin'
   spec.description   = 'Solargraph Plugin that reports line/branch coverage from unit tests'
-  spec.homepage      = 'https://github.com/ckolkey/solargraph_test_coverage'
+  spec.homepage      = 'https://github.com/deepdivr/solargraph_test_coverage'
   spec.license       = 'MIT'
-  spec.required_ruby_version = Gem::Requirement.new('>= 2.3.0')
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.6.0')
 
-  spec.metadata['allowed_push_host'] = "https://rubygems.org"
+  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
 
   spec.metadata['homepage_uri'] = spec.homepage
   spec.metadata['source_code_uri'] = spec.homepage
@@ -29,5 +31,4 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'solargraph', '~> 0.40', '> 0.40'
-  spec.add_runtime_dependency 'rspec-core', '~> 3.10', '>= 3.10.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solargraph_test_coverage
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.2.0
 platform: ruby
 authors:
 - Cameron Kolkey
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-08-31 00:00:00.000000000 Z
+date: 2021-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solargraph
@@ -30,26 +30,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.40'
-- !ruby/object:Gem::Dependency
-  name: rspec-core
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.10.0
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.10'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 3.10.0
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.10'
 description: Solargraph Plugin that reports line/branch coverage from unit tests
 email:
 - Cameron.Kolkey@gmail.com
@@ -66,19 +46,21 @@ files:
 - Rakefile
 - lib/solargraph_test_coverage.rb
 - lib/solargraph_test_coverage/branch.rb
+- lib/solargraph_test_coverage/config.rb
 - lib/solargraph_test_coverage/fork_process.rb
 - lib/solargraph_test_coverage/helpers.rb
 - lib/solargraph_test_coverage/test_coverage_reporter.rb
+- lib/solargraph_test_coverage/test_runner.rb
 - lib/solargraph_test_coverage/version.rb
 - solargraph_test_coverage.gemspec
-homepage: https://github.com/ckolkey/solargraph_test_coverage
+homepage: https://github.com/deepdivr/solargraph_test_coverage
 licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-  homepage_uri: https://github.com/ckolkey/solargraph_test_coverage
-  source_code_uri: https://github.com/ckolkey/solargraph_test_coverage
-  changelog_uri: https://github.com/ckolkey/solargraph_test_coverage
+  homepage_uri: https://github.com/deepdivr/solargraph_test_coverage
+  source_code_uri: https://github.com/deepdivr/solargraph_test_coverage
+  changelog_uri: https://github.com/deepdivr/solargraph_test_coverage
 post_install_message: 
 rdoc_options: []
 require_paths:
@@ -87,7 +69,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.0
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="