danger-undercover 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb04717e10cd473267cb307eb0d4b4d0fcd4301fc9e5aa49addf8167cc7b8dbe
-  data.tar.gz: be57b82ff15f6808c4cb4c7ec835c347e1907d4f152a0e2ba8ffc56a83f71cbb
+  metadata.gz: b5ba1eda2ac42f3e902619f81a4c6eda0371c5add9c8f6e4a63dbe69e54ec3d6
+  data.tar.gz: 3336e15c47eb09127809cabc2ea4b6ee7a0e646bf1f2be7ad7f631a4052cdecb
 SHA512:
-  metadata.gz: 826db9606d9cdb7bd8baf9b2090654b03412a00bdae6921bb662aee6267730c4c6c5fabc85b04f3380e9ce6b38d5487c1b81e74d4eefbeaeab93740d9d6bdfdf
-  data.tar.gz: bbed20bc346d9118d268d042c18427c1e856f973f9afbabb58dcbccb8ebe66a08fe8d66152fc479e4e602ead77647465db25c685dcfe56f84a16f39d4bfbcb16
+  metadata.gz: cf71b015300f51aa2d57172a03032a4534ce9fad0e83f97e840e4c951c2d4e042253fcc4581fcc725e4fc2343eb3d357aef557904d8235204e232a1dfda77fed
+  data.tar.gz: a4323eb9d539a83fe0041ad528023d7f175d075f4f4091126fb5676c466096dacb614f43302d6c122b99a56cb919fc834a98d6c845788b08a9caf0c63ac4741b

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-undercover (1.0.0)
+    danger-undercover (1.1.0)
       danger-plugin-api (~> 1.0)
 
 GEM
@@ -19,7 +19,7 @@ GEM
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (8.0.6)
+    danger (8.1.0)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
@@ -35,8 +35,9 @@ GEM
     danger-plugin-api (1.0.0)
       danger (> 2.0)
     diff-lcs (1.4.4)
-    faraday (1.0.1)
+    faraday (1.1.0)
       multipart-post (>= 1.2, < 3)
+      ruby2_keywords
     faraday-http-cache (2.2.0)
       faraday (>= 0.8)
     ffi (1.13.1)
@@ -117,6 +118,7 @@ GEM
     rubocop-ast (0.7.1)
       parser (>= 2.7.1.5)
     ruby-progressbar (1.10.1)
+    ruby2_keywords (0.0.2)
     sawyer (0.8.2)
       addressable (>= 2.3.5)
       faraday (> 0.8, < 2.0)

data/README.md

@@ -15,10 +15,16 @@ To know more about running undercover [visit here](https://github.com/grodowski/
 > Use the `-c --compare ref` flag to specify a git ref (commit hash, branch name, tag) to compare against. 
 **This is a recommended usage for CI/CD build environments**, as `undercover` will `exit 1` if there are any warnings.
 
-Run the below command to output undercover report to a `txt` file which this plugin will use to geneate PR comments.
-To use it on a CI server, run this command before running `Danger` so that the file is created beforehand.
+This plugin provides a command `undercover-report` that uses `undercover` command from 
+[Undercover](https://github.com/grodowski/undercover) gem. It takes all the options that `undercover` command takes.
 
-    $ undercover -c $compare_git_ref > coverage/undercover.txt
+They both works in the same way but what `undercover-report` extra does is it prints `undercover` report to a default
+ file in `coverage/undercover.txt`. This makes using `undercover` in CI server much easier.
+
+To use it on a CI server, run the below command before running `Danger` so that the report file is created beforehand
+ which `Danger` will use..
+
+    $ undercover-report -c $compare_git_ref
 
 >Here $compare_git_ref as per undercover documentation, can be a commit hash, branch name, or tag. i.e. origin/master
 , origin/development
@@ -27,7 +33,7 @@ To use it on a CI server, run this command before running `Danger` so that the f
 Then in your `Dangerfile` add the following line with the output file 
 
 ```ruby
-undercover.report 'coverage/undercover.txt'
+undercover.report
 ```
 
 ## Development

data/bin/undercover-report

@@ -0,0 +1,10 @@
+# bin/undercover_report
+
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+$LOAD_PATH.unshift("#{__dir__}/../lib")
+
+require 'undercover/cli'
+
+puts DangerUndercover::CLI.run(ARGV)

data/lib/undercover/cli.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+require 'fileutils'
+
+module DangerUndercover
+  # module for undercover-report
+  module CLI
+    class << self
+      # Runs the undercover command with provided arguments
+      # and writes the output to a file
+      # @return  [String]
+      #
+      def run(args = nil)
+        undercover_output = `undercover #{args&.join(' ')}`
+
+        File.open(output_file, 'w') do |f|
+          f.write(undercover_output)
+        end
+
+        undercover_output
+      end
+
+      private
+
+      # Returns the file to write report to
+      # @return  [String]
+      #
+      def output_file
+        create_directory!
+
+        File.join(output_directory, 'undercover.txt')
+      end
+
+      # Creates directory if doesn't exists
+      # @return  [String]
+      #
+      def create_directory!
+        return if Dir.exist?(output_directory)
+
+        FileUtils.mkdir_p(output_directory)
+      end
+
+      # Output directory
+      # @return  [String]
+      #
+      def output_directory
+        File.join(Dir.getwd, 'coverage')
+      end
+    end
+  end
+end

data/lib/undercover/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Undercover
-  VERSION = '1.0.0'
+  VERSION = '1.1.0'
 end

data/lib/undercover/plugin.rb

@@ -16,6 +16,7 @@ module Danger
   #
   class DangerUndercover < Plugin
     VALID_FILE_FORMAT = '.txt'
+    DEFAULT_PATH = 'coverage/undercover.txt'
 
     # Checks the file validity and warns if no file is found
     # if a valid file is found then if there are no changes,
@@ -23,15 +24,15 @@ module Danger
     # If there are reports then it shows the report as a warning in danger.
     # @return  [void]
     #
-    def report(undercover_path, sticky: true)
+    def report(undercover_path = DEFAULT_PATH, sticky: true)
       return fail('Undercover: coverage report cannot be found.') unless valid_file? undercover_path
 
-      report = File.open(undercover_path).read
+      report = File.open(undercover_path).read.force_encoding('UTF-8')
 
-      if report.match(/No coverage is missing in latest changes/)
-        message(report, sticky: sticky)
-      else
+      if report.match(/some methods have no test coverage/)
         warn(report, sticky: sticky)
+      else
+        message(report, sticky: sticky)
       end
     end
 

data/spec/cli_spec.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+require File.expand_path('spec_helper', __dir__)
+require 'undercover/cli'
+
+# rubocop:disable Metrics/BlockLength
+module DangerUndercover
+  describe DangerUndercover::CLI do
+    let!(:mock_message) { 'Test Passed' }
+    let!(:directory) { File.join(Dir.getwd, 'coverage') } # default directory
+    let!(:file) { File.join(Dir.getwd, 'coverage/undercover.txt') } # default file
+
+    before(:each) do
+      # mocks the undercover #{args&.join(' ')} CLI command output.
+      allow(described_class).to receive(:`).and_return(mock_message)
+    end
+
+    after(:all) do
+      # removes the folder after tests pass.
+      FileUtils.rm_rf(File.join(Dir.getwd, 'coverage'))
+    end
+
+    it 'prints the undercover output' do
+      expect(described_class.run).to eql(mock_message)
+    end
+
+    it "creates a default folder if doesn't exists" do
+      FileUtils.rm_rf(directory)
+      described_class.run
+
+      expect(Dir.exist?(directory)).to be true
+    end
+
+    it 'creates default file undercover.txt' do
+      described_class.run
+
+      expect(File.exist?(file)).to be true
+    end
+
+    it 'writes undercover report to default file' do
+      described_class.run
+      report = File.open(file).read
+
+      expect(report).to eql(mock_message)
+    end
+  end
+end
+# rubocop:enable Metrics/BlockLength

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger-undercover
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Nimble
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-10-15 00:00:00.000000000 Z
+date: 2020-10-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -155,7 +155,8 @@ description: Show undercover report to PRs
 email:
 - dev@nimblehq.co
 - rafayet@nimblehq.co
-executables: []
+executables:
+- undercover-report
 extensions: []
 extra_rdoc_files: []
 files:
@@ -171,11 +172,14 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- bin/undercover-report
 - danger-undercover.gemspec
 - lib/danger_plugin.rb
 - lib/danger_undercover.rb
+- lib/undercover/cli.rb
 - lib/undercover/gem_version.rb
 - lib/undercover/plugin.rb
+- spec/cli_spec.rb
 - spec/fixtures/undercover_failed.txt
 - spec/fixtures/undercover_passed.txt
 - spec/spec_helper.rb
@@ -204,6 +208,7 @@ signing_key:
 specification_version: 4
 summary: A Danger plugin for Undercover gem
 test_files:
+- spec/cli_spec.rb
 - spec/fixtures/undercover_failed.txt
 - spec/fixtures/undercover_passed.txt
 - spec/spec_helper.rb