danger-cobertura 1.0.0

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: 82b28e2ecb8f0383f89f977bc3a1b073b0be37076a3e5f97e66679abd8790102
+  data.tar.gz: b44ee503e7b9588a4d385f008cd26a293f32075f34a4ee726053d2dee659b518
+SHA512:
+  metadata.gz: 4616f102f79dbe73922f317efc478cadb10fa5c8584f5d0f214d1b1ae3d869eb2e89163b198d05f883b3f36c885e8f3cb8ac9c6f0410acd4b32f7b8f057fee1b
+  data.tar.gz: 047a27612a80b13e93410a422574c2b32e39c186eec49b095710cbfaa8d9b8884fec0ef7d0b8e73383d3b8e77bef1a393ab493e35710aa90f3a341bcbfb4cf13

data/.gitignore

@@ -0,0 +1,6 @@
+.DS_Store
+pkg
+.idea/
+.yardoc
+Gemfile.lock
+.vscode

data/.rubocop.yml

@@ -0,0 +1,152 @@
+# Defaults can be found here: https://github.com/bbatsov/rubocop/blob/master/config/default.yml
+
+# If you don't like these settings, just delete this file :)
+
+AllCops:
+  TargetRubyVersion: 2.5
+
+Style/StringLiterals:
+  EnforcedStyle: double_quotes
+  Enabled: true
+
+# kind_of? is a good way to check a type
+Style/ClassCheck:
+  EnforcedStyle: kind_of?
+
+# It's better to be more explicit about the type
+Style/BracesAroundHashParameters:
+  Enabled: false
+
+# specs sometimes have useless assignments, which is fine
+Lint/UselessAssignment:
+  Exclude:
+    - '**/spec/**/*'
+
+# We could potentially enable the 2 below:
+Layout/IndentHash:
+  Enabled: false
+
+Layout/AlignHash:
+  Enabled: false
+
+# HoundCI doesn't like this rule
+Layout/DotPosition:
+  Enabled: false
+
+# We allow !! as it's an easy way to convert ot boolean
+Style/DoubleNegation:
+  Enabled: false
+
+# Cop supports --auto-correct.
+Lint/UnusedBlockArgument:
+  Enabled: false
+
+# We want to allow class Fastlane::Class
+Style/ClassAndModuleChildren:
+  Enabled: false
+
+Metrics/AbcSize:
+  Max: 60
+
+# The %w might be confusing for new users
+Style/WordArray:
+  MinSize: 19
+
+# raise and fail are both okay
+Style/SignalException:
+  Enabled: false
+
+# Better too much 'return' than one missing
+Style/RedundantReturn:
+  Enabled: false
+
+# Having if in the same line might not always be good
+Style/IfUnlessModifier:
+  Enabled: false
+
+# and and or is okay
+Style/AndOr:
+  Enabled: false
+
+# Configuration parameters: CountComments.
+Metrics/ClassLength:
+  Max: 350
+
+Metrics/CyclomaticComplexity:
+  Max: 17
+
+# Configuration parameters: AllowURI, URISchemes.
+Metrics/LineLength:
+  Max: 370
+
+# Configuration parameters: CountKeywordArgs.
+Metrics/ParameterLists:
+  Max: 10
+
+Metrics/PerceivedComplexity:
+  Max: 18
+
+# Sometimes it's easier to read without guards
+Style/GuardClause:
+  Enabled: false
+
+# something = if something_else
+# that's confusing
+Style/ConditionalAssignment:
+  Enabled: false
+
+# Better to have too much self than missing a self
+Style/RedundantSelf:
+  Enabled: false
+
+Metrics/MethodLength:
+  Max: 60
+
+# We're not there yet
+Style/Documentation:
+  Enabled: false
+
+# Adds complexity
+Style/IfInsideElse:
+  Enabled: false
+
+# danger specific
+
+Style/BlockComments:
+  Enabled: false
+
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
+# FIXME: 25
+Metrics/BlockLength:
+  Max: 345
+  Exclude:
+    - "**/*_spec.rb"
+
+Style/MixinGrouping:
+  Enabled: false
+
+Style/FileName:
+  Enabled: false
+
+Layout/IndentHeredoc:
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  Enabled: false
+
+PercentLiteralDelimiters:
+  PreferredDelimiters:
+    "%":  ()
+    "%i": ()
+    "%q": ()
+    "%Q": ()
+    "%r": "{}"
+    "%s": ()
+    "%w": ()
+    "%W": ()
+    "%x": ()
+
+Security/YAMLLoad:
+  Enabled: false

data/.travis.yml

@@ -0,0 +1,19 @@
+language: ruby
+
+cache:
+  directories:
+  - bundle
+
+rvm:
+  - 2.5.2
+
+script:
+  - bundle exec rake spec
+
+deploy:
+  provider: rubygems
+  api_key:
+    secure: N+QJ0e6CW/m5zVxKYBZNSShZYghGH7TVgsoDbNndawGe0/A5Q8iSURliqO1Sykbc7xPwnewfNpcVSB+6LjQPOfUn62ZtxuDQQx5orVwcn0ul3wHG7MD9CoJqUD/xbsJNFjBMSI+zYlfg9uI9BrB55ctdra77HcvOsayS3ByL6pB+GoB1HgXwNZQjLL+0Hpy0sO3Vvr7LiAE4nv4CikSQy169hQ7Q70K0bxfhdGYxWX2FW2NxfvxatkBAZrHlN5Vg4NV4ZAKQBMgI3kFRupZOoQkTy08CZCueuac9jl8I/FaIsugKVjgba+JJ79WnqC/9pPSWedcy9kgvl8/mKu8KUEhBFjF0/ib40+KG9ioaHUDr+ddWD1FEm7qFpipamH7mcznmcfSaEqvILz8V36q8bRCjgHksY1TR9rJBa4RFzGvzyUclbtkvqm1KNN9VqEREKFKjw1JtBfPeSqcm+ciYmREl0r/RgI14Z7XyghJi9CxRdggcQonCfQSDcNfuvasJAd+prRmAUue/MRqbtgyS5rfUiEX5U4H5Futi9ri4vhVUc6seNYGPuxcf42/KGUFspkDJwW7nKSNRkf+diV7OfGLLR75YU4r5T6jFx/Jl+cTN++xCGhBGYa97FwI8+NMQCnACERC1hVXADuVGCQGvh68xpuPgFdOwtdLEONol94g=
+  gem: danger-cobertura
+  on:
+    branch: master

data/CHANGELOG.md

@@ -0,0 +1,13 @@
+# Changelog
+All notable changes to this project will be documented in this file.
+
+The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
+and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
+## [Unreleased]
+
+## [1.0.0] - 2018-11-25
+### Added
+- Initial release.
+- Add `show_coverage` method to display a markdown table with coverage information of all modified and added files.
+- Add `warn_if_file_less_than(percentage:)` method to display a warning for each file which does not achieve the minimum coverage given.

data/Gemfile

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+source "https://rubygems.org"
+
+# Specify your gem's dependencies in danger-cobertura.gemspec
+gemspec

data/Guardfile

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+# A guardfile for making Danger Plugins
+# For more info see https://github.com/guard/guard#readme
+
+# To run, use `bundle exec guard`.
+
+guard :rspec, cmd: "bundle exec rspec" do
+  require "guard/rspec/dsl"
+  dsl = Guard::RSpec::Dsl.new(self)
+
+  # RSpec files
+  rspec = dsl.rspec
+  watch(rspec.spec_helper) { rspec.spec_dir }
+  watch(rspec.spec_support) { rspec.spec_dir }
+  watch(rspec.spec_files)
+
+  # Ruby files
+  ruby = dsl.ruby
+  dsl.watch_spec_files_for(ruby.lib_files)
+end

data/LICENSE.txt

@@ -0,0 +1,22 @@
+Copyright (c) 2018 Kyaak <kyaak.dev@gmail.com>
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.md

@@ -0,0 +1,84 @@
+# danger-cobertura
+
+A [cobertura](http://cobertura.github.io/cobertura/) report plugin for [danger](https://danger.systems/ruby).
+
+This plugin allows parsing of an xml coverage report generated by __cobertura__ and displays desired metrics.
+
+_It inspects only modified and added files, deleted ones are not considered useful for this report._
+
+## How it looks like
+### Warnings about file coverage
+<table>
+  <thead>
+    <tr>
+      <th width="50"></th>
+      <th width="100%">
+          1 Warnings
+      </th>
+    </tr>
+  </thead>
+  <tbody>
+    <tr>
+      <td><g-emoji alias="warning" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/26a0.png">⚠️</g-emoji></td>
+      <td>example.py has less than 50% coverage</td>
+    </tr>
+  </tbody>
+</table>
+
+### Coverage report
+
+File | Coverage
+-----|-----
+example_one.py | 50.00
+example_two.py | 35.60
+example_three.py | 10.48
+
+
+## Installation
+
+    $ gem install danger-cobertura
+
+## Usage
+
+    Methods and attributes from this plugin are available in
+    your `Dangerfile` under the `cobertura` namespace.
+
+<blockquote>Warn files with a coverage less than 50%
+<pre>
+cobertura.report = build/reports/coverage.xml
+cobertura.warn_if_file_less_than(percentage: 50.0)
+</pre>
+</blockquote>    
+
+<blockquote>Show coverage of all files 
+<pre>
+cobertura.report = build/reports/coverage.xml
+cobertura.show_coverage
+</pre>
+</blockquote>   
+
+<blockquote>Combine all reports 
+<pre>
+cobertura.report = build/reports/coverage.xml
+cobertura.warn_if_file_less_than(percentage: 50.0)
+cobertura.show_coverage
+</pre>
+</blockquote>  
+
+## Attributes
+
+__`report`__ - Path to the cobertura xml report, e.g. `build/reports/coverage.xml`
+
+## Methods
+
+__`warn_if_file_less_than(percentage:)`__ - Add a danger warning for each file with a lower total coverage as given.
+
+__`show_coverage`__ - Show a markdown table including the coverage for all (modified / added) files.
+
+## Development
+
+1. Clone this repo
+2. Run `bundle install` to setup dependencies.
+3. Run `bundle exec rake spec` to run the tests.
+4. Use `bundle exec guard` to automatically have tests run as you make changes.
+5. Make your changes.

data/Rakefile

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require "bundler/gem_tasks"
+require "rspec/core/rake_task"
+require "rubocop/rake_task"
+
+RSpec::Core::RakeTask.new(:specs)
+
+task default: :specs
+
+task :spec do
+  Rake::Task["specs"].invoke
+  Rake::Task["rubocop"].invoke
+  Rake::Task["spec_docs"].invoke
+end
+
+desc "Run RuboCop on the lib/specs directory"
+RuboCop::RakeTask.new(:rubocop) do |task|
+  task.patterns = %w(lib/**/*.rb spec/**/*.rb)
+end
+
+desc "Ensure that the plugin passes `danger plugins lint`"
+task :spec_docs do
+  sh "bundle exec danger plugins lint"
+end

data/danger-cobertura.gemspec

@@ -0,0 +1,52 @@
+# frozen_string_literal: true
+
+lib = File.expand_path("lib", __dir__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require "cobertura/gem_version.rb"
+
+Gem::Specification.new do |spec|
+  spec.name          = "danger-cobertura"
+  spec.version       = Cobertura::VERSION
+  spec.authors       = ["Kyaak"]
+  spec.email         = ["kyaak.dev@gmail.com"]
+  spec.description   = "A cobertura report plugin for danger."
+  spec.summary       = "This plugin allows parsing of an xml coverage report generated by cobertura and displays desired metrics."
+  spec.homepage      = "https://github.com/Kyaak/danger-cobertura"
+  spec.license       = "MIT"
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.add_runtime_dependency "danger-plugin-api", "~> 1.0"
+  spec.add_runtime_dependency "oga", "~> 2.15"
+
+  # General ruby development
+  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "rake", "~> 10.0"
+
+  # Testing support
+  spec.add_development_dependency "mocha", "~> 1.2"
+  spec.add_development_dependency "rspec", "~> 3.4"
+
+  # Linting code and docs
+  spec.add_development_dependency "rubocop", "~> 0.6"
+  spec.add_development_dependency "yard", "~> 0.9"
+
+  # Makes testing easy via `bundle exec guard`
+  spec.add_development_dependency "guard", "~> 2.14"
+  spec.add_development_dependency "guard-rspec", "~> 4.7"
+
+  # If you want to work on older builds of ruby
+  spec.add_development_dependency "listen", "3.0.7"
+
+  # This gives you the chance to run a REPL inside your tests
+  # via:
+  #
+  #    require 'pry'
+  #    binding.pry
+  #
+  # This will stop test execution and let you inspect the results
+  spec.add_development_dependency "pry"
+end

data/lib/cobertura/coverage_item.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+class CoverageItem
+  def initialize(node)
+    @node = node
+  end
+
+  def total_percentage
+    @total_percentage ||= ((branch_rate + line_rate) / 2) * 100
+  end
+
+  def branch_rate
+    @branch_rate ||= @node.attribute("branch-rate").value.to_f
+  end
+
+  def line_rate
+    @line_rate ||= @node.attribute("line-rate").value.to_f
+  end
+
+  def file_name
+    @file_name ||= @node.attribute("filename").value.to_s
+  end
+
+  def name
+    @name ||= @node.attribute("name").value.to_s
+  end
+end

data/lib/cobertura/gem_version.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+module Cobertura
+  VERSION = "1.0.0"
+end

data/lib/cobertura/plugin.rb

@@ -0,0 +1,97 @@
+# frozen_string_literal: true
+
+module Danger
+  # Show code coverage of modified and added files.
+  # Add warnings if minimum file coverage is not achieved.
+  #
+  # @example Warn on minimum file coverage of 30% and show all modified files coverage.
+  #       cobertura.report = "path/to/my/report.xml"
+  #       cobertura.warn_if_file_less_than(percentage: 30)
+  #       cobertura.show_coverage
+  #
+  # @see  Kyaak/danger-cobertura
+  # @tags cobertura, coverage
+  #
+  class DangerCobertura < Plugin
+    require_relative "./coverage_item"
+    ERROR_FILE_NOT_SET = "Cobertura file not set. Use 'cobertura.file = \"path/to/my/report.xml\"'."
+    ERROR_FILE_NOT_FOUND = "No file found at %s"
+
+    # Path to the xml formatted cobertura report.
+    #
+    # @return   [String]
+    attr_accessor :report
+
+    # Warn if a modified file has a lower total coverage than defined.
+    #
+    # @param percentage [Float] The minimum code coverage required for a file.
+    # @return [Array<String>]
+    def warn_if_file_less_than(percentage:)
+      filtered_items.each do |item|
+        next unless item.total_percentage < percentage
+
+        warn "#{item.name} has less than #{percentage}% coverage"
+      end
+    end
+
+    # Show markdown table of modified and added files.
+    #
+    # @return [Array<String>]
+    def show_coverage
+      return if filtered_items.empty?
+
+      line = +"## Code coverage\n"
+      line << "File | Coverage\n"
+      line << "-----|-----\n"
+      filtered_items.each do |item|
+        line << "#{item.name} | #{format('%.2f', item.total_percentage)}\n"
+      end
+      markdown line
+    end
+
+    private
+
+    # Getter for coverage items of targeted files.
+    #
+    # @return [Array<CoverageItem>]
+    def filtered_items
+      @filtered_items ||= coverage_items.select do |item|
+        target_files.include? item.file_name
+      end
+    end
+
+    # A getter for current updated files.
+    #
+    # @return [Array<String>]
+    def target_files
+      @target_files ||= git.modified_files + git.added_files
+    end
+
+    # Parse the defined coverage report file.
+    #
+    # @return [Oga::XML::Document]
+    def parse
+      require "oga"
+      raise ERROR_FILE_NOT_SET if report.nil? || report.empty?
+      raise format(ERROR_FILE_NOT_FOUND, report) unless File.exist?(report)
+
+      Oga.parse_xml(File.read(report))
+    end
+
+    # Get the xml cobertura report.
+    #
+    # @return [Oga::XML::Document]
+    def xml_report
+      @xml_report ||= parse
+    end
+
+    # Getter for all coverage data.
+    #
+    # @return [Array<CoverageItem>]
+    def coverage_items
+      @coverage_items ||= xml_report.xpath("//class").map do |node|
+        CoverageItem.new(node)
+      end
+    end
+  end
+end

data/lib/danger_cobertura.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require "cobertura/gem_version"

data/lib/danger_plugin.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+require "cobertura/plugin"

data/spec/assets/coverage.xml

@@ -0,0 +1,283 @@
+<?xml version="1.0" ?>
+<coverage branch-rate="0.6257" branches-covered="423" branches-valid="676" complexity="0" line-rate="0.8255" lines-covered="3279" lines-valid="3972" timestamp="1543096490494" version="4.5.1">
+	<!-- Generated by coverage.py: https://coverage.readthedocs.io -->
+	<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
+	<sources>
+		<source>/path/to/my/project/sourcs</source>
+	</sources>
+	<packages>
+		<package branch-rate="1" complexity="0" line-rate="0.9903" name=".">
+			<classes>
+				<class branch-rate="1" complexity="0" filename="top_level_one.py" line-rate="1" name="top_level_one.py">
+					<methods/>
+					<lines>
+						<line hits="1" number="16"/>
+						<line hits="1" number="20"/>
+						<line hits="1" number="21"/>
+						<line hits="1" number="22"/>
+						<line hits="1" number="23"/>
+						<line hits="1" number="24"/>
+						<line hits="1" number="25"/>
+						<line hits="1" number="26"/>
+						<line hits="1" number="27"/>
+						<line hits="1" number="28"/>
+						<line hits="1" number="30"/>
+						<line hits="1" number="31"/>
+						<line hits="1" number="32"/>
+						<line hits="1" number="33"/>
+						<line hits="1" number="34"/>
+						<line hits="1" number="35"/>
+						<line hits="1" number="36"/>
+						<line hits="1" number="37"/>
+						<line hits="1" number="38"/>
+						<line hits="1" number="39"/>
+						<line hits="1" number="40"/>
+						<line hits="1" number="41"/>
+						<line hits="1" number="42"/>
+						<line hits="1" number="43"/>
+						<line hits="1" number="44"/>
+						<line hits="1" number="45"/>
+						<line hits="1" number="46"/>
+					</lines>
+				</class>
+				<class branch-rate="1" complexity="0" filename="top_level_two.py" line-rate="0.9868" name="top_level_two.py">
+					<methods/>
+					<lines>
+						<line hits="1" number="14"/>
+						<line hits="1" number="18"/>
+						<line hits="1" number="19"/>
+						<line hits="1" number="20"/>
+						<line hits="1" number="21"/>
+						<line hits="1" number="22"/>
+						<line hits="1" number="23"/>
+						<line hits="1" number="24"/>
+						<line hits="1" number="25"/>
+						<line hits="1" number="26"/>
+						<line hits="1" number="27"/>
+						<line hits="1" number="28"/>
+						<line hits="1" number="29"/>
+						<line hits="1" number="30"/>
+						<line hits="1" number="31"/>
+						<line hits="1" number="32"/>
+						<line hits="1" number="34"/>
+						<line hits="1" number="35"/>
+						<line hits="1" number="36"/>
+						<line hits="1" number="37"/>
+						<line hits="1" number="39"/>
+						<line hits="1" number="40"/>
+						<line hits="1" number="45"/>
+						<line hits="1" number="48"/>
+						<line hits="1" number="54"/>
+						<line hits="1" number="55"/>
+						<line hits="1" number="56"/>
+						<line hits="1" number="57"/>
+						<line hits="1" number="58"/>
+						<line hits="1" number="59"/>
+						<line hits="1" number="60"/>
+						<line hits="1" number="61"/>
+						<line hits="1" number="62"/>
+						<line hits="1" number="63"/>
+						<line hits="1" number="64"/>
+						<line hits="1" number="66"/>
+						<line hits="1" number="68"/>
+						<line hits="1" number="69"/>
+						<line hits="1" number="71"/>
+						<line hits="1" number="72"/>
+						<line hits="1" number="74"/>
+						<line hits="1" number="75"/>
+						<line hits="1" number="76"/>
+						<line hits="1" number="77"/>
+						<line hits="1" number="78"/>
+						<line hits="1" number="82"/>
+						<line hits="1" number="83"/>
+						<line hits="1" number="84"/>
+						<line hits="1" number="85"/>
+						<line hits="1" number="86"/>
+						<line hits="1" number="87"/>
+						<line hits="1" number="88"/>
+						<line hits="1" number="89"/>
+						<line hits="1" number="90"/>
+						<line hits="1" number="92"/>
+						<line hits="1" number="93"/>
+						<line hits="1" number="95"/>
+						<line hits="1" number="96"/>
+						<line hits="1" number="98"/>
+						<line hits="1" number="99"/>
+						<line hits="1" number="100"/>
+						<line hits="1" number="102"/>
+						<line hits="1" number="103"/>
+						<line hits="1" number="110"/>
+						<line hits="1" number="112"/>
+						<line hits="1" number="113"/>
+						<line hits="1" number="120"/>
+						<line hits="1" number="123"/>
+						<line hits="1" number="127"/>
+						<line hits="1" number="128"/>
+						<line hits="1" number="129"/>
+						<line hits="1" number="130"/>
+						<line hits="1" number="131"/>
+						<line hits="1" number="133"/>
+						<line hits="1" number="134"/>
+						<line hits="0" number="140"/>
+					</lines>
+				</class>
+			</classes>
+		</package>
+		<package branch-rate="0.6719" complexity="0" line-rate="0.8438" name="sub_folder">
+			<classes>
+				<class branch-rate="0.6552" complexity="0" filename="sub_folder/sub_one.py" line-rate="0.783" name="sub_one.py">
+					<methods/>
+					<lines>
+						<line hits="1" number="12"/>
+						<line hits="1" number="13"/>
+						<line hits="1" number="14"/>
+						<line hits="1" number="17"/>
+						<line hits="1" number="22"/>
+						<line hits="1" number="23"/>
+						<line hits="1" number="24"/>
+						<line hits="1" number="26"/>
+						<line hits="1" number="27"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="29" number="28"/>
+						<line hits="0" number="29"/>
+						<line hits="1" number="30"/>
+						<line hits="1" number="32"/>
+						<line hits="1" number="38"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="39"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="40"/>
+						<line hits="1" number="41"/>
+						<line hits="1" number="42"/>
+						<line hits="1" number="44"/>
+						<line hits="1" number="49"/>
+						<line hits="1" number="51"/>
+						<line hits="1" number="56"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="57"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="58"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="59"/>
+						<line hits="1" number="60"/>
+						<line hits="1" number="61"/>
+						<line hits="1" number="63"/>
+						<line hits="1" number="69"/>
+						<line hits="1" number="71"/>
+						<line hits="1" number="76"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="77"/>
+						<line hits="1" number="78"/>
+						<line hits="1" number="79"/>
+						<line hits="1" number="81"/>
+						<line hits="1" number="87"/>
+						<line hits="1" number="89"/>
+						<line hits="0" number="96"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="98,102" number="97"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="97,99" number="98"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="98,100" number="99"/>
+						<line hits="0" number="100"/>
+						<line hits="0" number="101"/>
+						<line hits="0" number="102"/>
+						<line hits="1" number="104"/>
+						<line hits="1" number="109"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="110"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="112" number="111"/>
+						<line hits="0" number="112"/>
+						<line hits="0" number="113"/>
+						<line hits="1" number="114"/>
+						<line hits="1" number="116"/>
+						<line hits="1" number="121"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="122"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="125" number="123"/>
+						<line hits="0" number="125"/>
+						<line hits="0" number="126"/>
+						<line hits="1" number="127"/>
+						<line hits="1" number="129"/>
+						<line hits="1" number="134"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="135"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="136"/>
+						<line hits="1" number="137"/>
+						<line hits="1" number="138"/>
+						<line hits="1" number="139"/>
+						<line hits="1" number="141"/>
+						<line hits="0" number="146"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="148,151" number="147"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="147,149" number="148"/>
+						<line hits="0" number="149"/>
+						<line hits="0" number="150"/>
+						<line hits="0" number="151"/>
+						<line hits="1" number="153"/>
+						<line hits="1" number="158"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="163" number="159"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="159" number="160"/>
+						<line hits="1" number="161"/>
+						<line hits="1" number="162"/>
+						<line hits="1" number="163"/>
+						<line hits="1" number="165"/>
+						<line hits="1" number="170"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="171"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="172"/>
+						<line hits="1" number="173"/>
+						<line hits="1" number="174"/>
+						<line hits="1" number="175"/>
+						<line hits="1" number="177"/>
+						<line hits="1" number="183"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="184"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="185"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="187" number="186"/>
+						<line hits="0" number="187"/>
+						<line hits="1" number="188"/>
+						<line hits="1" number="190"/>
+						<line hits="1" number="191"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="193" number="192"/>
+						<line hits="0" number="193"/>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" missing-branches="195,196" number="194"/>
+						<line hits="0" number="195"/>
+						<line hits="1" number="196"/>
+						<line hits="1" number="198"/>
+						<line hits="1" number="203"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="204"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="206" number="205"/>
+						<line hits="0" number="206"/>
+						<line hits="1" number="207"/>
+					</lines>
+				</class>
+				<class branch-rate="0.75" complexity="0" filename="sub_folder/sub_two.py" line-rate="0.9333" name="sub_two.py">
+					<methods/>
+					<lines>
+						<line hits="1" number="13"/>
+						<line hits="1" number="14"/>
+						<line hits="1" number="17"/>
+						<line hits="1" number="22"/>
+						<line hits="1" number="23"/>
+						<line hits="1" number="24"/>
+						<line hits="1" number="25"/>
+						<line hits="1" number="26"/>
+						<line hits="1" number="27"/>
+						<line hits="1" number="28"/>
+						<line hits="1" number="30"/>
+						<line hits="1" number="35"/>
+						<line hits="1" number="37"/>
+						<line hits="1" number="42"/>
+						<line hits="1" number="44"/>
+						<line hits="0" number="49"/>
+						<line hits="1" number="51"/>
+						<line hits="1" number="56"/>
+						<line branch="true" condition-coverage="50% (1/2)" hits="1" missing-branches="58" number="57"/>
+						<line hits="0" number="58"/>
+						<line hits="1" number="59"/>
+						<line hits="1" number="61"/>
+						<line hits="1" number="66"/>
+						<line hits="1" number="68"/>
+						<line hits="1" number="73"/>
+						<line branch="true" condition-coverage="100% (2/2)" hits="1" number="74"/>
+						<line hits="1" number="75"/>
+						<line hits="1" number="76"/>
+						<line hits="1" number="78"/>
+						<line hits="1" number="83"/>
+					</lines>
+				</class>
+				<class branch-rate="0" complexity="0" filename="sub_folder/sub_three.py" line-rate="0" name="sub_three.py">
+					<methods/>
+					<lines>
+						<line branch="true" condition-coverage="0% (0/2)" hits="0" number="88"/>
+					</lines>
+				</class>
+			</classes>
+		</package>
+	</packages>
+</coverage>

data/spec/cobertura_spec.rb

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+require File.expand_path("spec_helper", __dir__)
+
+module Danger
+  describe Danger::DangerCobertura do
+    it "should be a plugin" do
+      expect(Danger::DangerCobertura.new(nil)).to be_a Danger::Plugin
+    end
+
+    describe "with Dangerfile" do
+      before do
+        @dangerfile = testing_dangerfile
+        @my_plugin = @dangerfile.cobertura
+        @my_plugin.report = "#{File.dirname(__FILE__)}/assets/coverage.xml"
+        @dangerfile.git.stubs(:modified_files).returns([])
+        @dangerfile.git.stubs(:added_files).returns([])
+      end
+
+      describe "warn_if_file_less_than" do
+        it "raises error if file attribute is nil" do
+          @my_plugin.report = nil
+          expect do
+            @my_plugin.warn_if_file_less_than(percentage: 50.0)
+          end.to raise_error(DangerCobertura::ERROR_FILE_NOT_SET)
+        end
+
+        it "raises error if file attribute is empty" do
+          @my_plugin.report = ""
+          expect do
+            @my_plugin.warn_if_file_less_than(percentage: 50.0)
+          end.to raise_error(DangerCobertura::ERROR_FILE_NOT_SET)
+        end
+
+        it "raises error if file is not found" do
+          @my_plugin.report = "cant/find/my/file.xml"
+          expect do
+            @my_plugin.warn_if_file_less_than(percentage: 50.0)
+          end.to raise_error(/#{@my_plugin.report}/)
+        end
+
+        it "adds warn if total coverage lower than given" do
+          @dangerfile.git.stubs(:modified_files).returns(%w(sub_folder/sub_two.py top_level_one.py))
+          @my_plugin.warn_if_file_less_than(percentage: 90.0)
+
+          expect(@dangerfile.status_report[:warnings]).to include("sub_two.py has less than 90.0% coverage")
+          expect(@dangerfile.status_report[:warnings]).not_to include("top_level_one.py has less than 90.0% coverage")
+        end
+
+        it "does not add warn if coverage not" do
+          @dangerfile.git.stubs(:modified_files).returns(["sub_folder/sub_two.py"])
+          @my_plugin.warn_if_file_less_than(percentage: 10.0)
+
+          expect(@dangerfile.status_report[:warnings]).to be_empty
+        end
+      end
+
+      describe "show_coverage" do
+        it "raises error if file attribute is nil" do
+          @my_plugin.report = nil
+          expect do
+            @my_plugin.show_coverage
+          end.to raise_error(DangerCobertura::ERROR_FILE_NOT_SET)
+        end
+
+        it "raises error if file attribute is empty" do
+          @my_plugin.report = ""
+          expect do
+            @my_plugin.show_coverage
+          end.to raise_error(DangerCobertura::ERROR_FILE_NOT_SET)
+        end
+
+        it "raises error if file is not found" do
+          @my_plugin.report = "cant/find/my/file.xml"
+          expect do
+            @my_plugin.show_coverage
+          end.to raise_error(/#{@my_plugin.report}/)
+        end
+
+        it "prints coverage" do
+          @dangerfile.git.stubs(:modified_files).returns(["sub_folder/sub_three.py"])
+          @my_plugin.show_coverage
+
+          expect(@dangerfile.status_report[:markdowns][0].message).to include("sub_three.py")
+          expect(@dangerfile.status_report[:markdowns][0].message).to include("0.00")
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,69 @@
+# frozen_string_literal: true
+
+require "pathname"
+ROOT = Pathname.new(File.expand_path("..", __dir__))
+$:.unshift((ROOT + "lib").to_s)
+$:.unshift((ROOT + "spec").to_s)
+
+require "bundler/setup"
+require "pry"
+
+require "rspec"
+require "danger"
+require "mocha"
+
+if `git remote -v` == ""
+  puts "You cannot run tests without setting a local git remote on this repo"
+  puts "It's a weird side-effect of Danger's internals."
+  exit(0)
+end
+
+# Use coloured output, it's the best.
+RSpec.configure do |config|
+  config.filter_gems_from_backtrace "bundler"
+  config.color = true
+  config.tty = true
+  config.mock_with :mocha
+end
+
+require "danger_plugin"
+
+# These functions are a subset of https://github.com/danger/danger/blob/master/spec/spec_helper.rb
+# If you are expanding these files, see if it's already been done ^.
+
+# A silent version of the user interface,
+# it comes with an extra function `.string` which will
+# strip all ANSI colours from the string.
+
+# rubocop:disable Lint/NestedMethodDefinition
+def testing_ui
+  @output = StringIO.new
+  def @output.winsize
+    [20, 9999]
+  end
+
+  cork = Cork::Board.new(out: @output)
+  def cork.string
+    out.string.gsub(/\e\[([;\d]+)?m/, "")
+  end
+  cork
+end
+# rubocop:enable Lint/NestedMethodDefinition
+
+# Example environment (ENV) that would come from
+# running a PR on TravisCI
+def testing_env
+  {
+    "HAS_JOSH_K_SEAL_OF_APPROVAL" => "true",
+    "TRAVIS_PULL_REQUEST" => "800",
+    "TRAVIS_REPO_SLUG" => "artsy/eigen",
+    "TRAVIS_COMMIT_RANGE" => "759adcbd0d8f...13c4dc8bb61d",
+    "DANGER_GITHUB_API_TOKEN" => "123sbdq54erfsd3422gdfio"
+  }
+end
+
+# A stubbed out Dangerfile for use in tests
+def testing_dangerfile
+  env = Danger::EnvironmentManager.new(testing_env)
+  Danger::Dangerfile.new(env, testing_ui)
+end

metadata

@@ -0,0 +1,234 @@
+--- !ruby/object:Gem::Specification
+name: danger-cobertura
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- Kyaak
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2018-11-25 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: danger-plugin-api
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: oga
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.15'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.15'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.0'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.2'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.4'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.9'
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.14'
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '4.7'
+- !ruby/object:Gem::Dependency
+  name: listen
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 3.0.7
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: A cobertura report plugin for danger.
+email:
+- kyaak.dev@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rubocop.yml"
+- ".travis.yml"
+- CHANGELOG.md
+- Gemfile
+- Guardfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- danger-cobertura.gemspec
+- lib/cobertura/coverage_item.rb
+- lib/cobertura/gem_version.rb
+- lib/cobertura/plugin.rb
+- lib/danger_cobertura.rb
+- lib/danger_plugin.rb
+- spec/assets/coverage.xml
+- spec/cobertura_spec.rb
+- spec/spec_helper.rb
+homepage: https://github.com/Kyaak/danger-cobertura
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.7.8
+signing_key: 
+specification_version: 4
+summary: This plugin allows parsing of an xml coverage report generated by cobertura
+  and displays desired metrics.
+test_files:
+- spec/assets/coverage.xml
+- spec/cobertura_spec.rb
+- spec/spec_helper.rb