danger-local_rules 0.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: fda2810b2f0e1b23d702c417b31411612e122c94d1b35ae3cd2ec62c53c1ddf1
+  data.tar.gz: 5186f9dd722e92189adfcdf5d3d3b082970aba869ca52f783eb862e7c4f7bff5
+SHA512:
+  metadata.gz: 7f5ee9435d73834639b9d85f72e94f3018dc87760a8b0b85a84f821b30d35cd70a9cd5f6c9f869520592d670731b0fd286c8121e7e8aec117566a643cb5f612b
+  data.tar.gz: 6104cd1344f4fac77a9baa46619632c14f1adae5369aa9e7fb0aa8f555fd2302e60a41be215c65547bb9303a34774eb2ece46f3236483a16c74d15b24eabf0df

data/.bundle/config

@@ -0,0 +1,3 @@
+---
+BUNDLE_PATH: "vendor/bundle"
+BUNDLE_JOBS: "4"

data/.gitignore

@@ -0,0 +1,7 @@
+.bundle/
+.DS_Store
+.idea/
+.yardoc
+Gemfile.lock
+pkg
+vendor/

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,10 @@
+language: ruby
+cache:
+  directories:
+    - bundle
+
+rvm:
+  - 2.5.0
+
+script:
+    - bundle exec rake spec

data/Gemfile

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+# Specify your gem's dependencies in danger-local_rules.gemspec
+gemspec

data/Guardfile

@@ -0,0 +1,19 @@
+# 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 Takuma Homma <nagomimatcha@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,19 @@
+# danger-local_rules
+
+A description of danger-local_rules.
+
+## Installation
+
+    $ gem install danger-local_rules
+
+## Usage
+
+    $ local_rules.check
+
+## 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,10 @@
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:specs)
+
+task default: :specs
+
+task :spec do
+  Rake::Task['specs'].invoke
+end

data/danger-local_rules.gemspec

@@ -0,0 +1,40 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'local_rules/gem_version.rb'
+
+Gem::Specification.new do |spec|
+  spec.name          = 'danger-local_rules'
+  spec.version       = LocalRules::VERSION
+  spec.authors       = ['mataku']
+  spec.email         = ['nagomimatcha@gmail.com']
+  spec.description   = %q{Manage local rules by Danger.}
+  spec.summary       = %q{Manage local rules by Danger.}
+  spec.homepage      = 'https://github.com/mataku/danger-local_rules'
+  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 'git_diff_parser'
+
+  # General ruby development
+  spec.add_development_dependency 'bundler', '~> 1.3'
+  spec.add_development_dependency 'rake', '~> 10.0'
+
+  # Testing support
+  spec.add_development_dependency 'rspec', '~> 3.4'
+
+  # Linting code and docs
+  spec.add_development_dependency "rubocop"
+  spec.add_development_dependency "yard"
+
+  # Makes testing easy via `bundle exec guard`
+  spec.add_development_dependency 'guard', '~> 2.14'
+  spec.add_development_dependency 'guard-rspec', '~> 4.7'
+
+  spec.add_development_dependency 'pry'
+end

data/lib/danger_local_rules.rb

@@ -0,0 +1 @@
+require "local_rules/gem_version"

data/lib/danger_plugin.rb

@@ -0,0 +1 @@
+require "local_rules/plugin"

data/lib/local_rules/gem_version.rb

@@ -0,0 +1,3 @@
+module LocalRules
+  VERSION = "0.0.1".freeze
+end

data/lib/local_rules/plugin.rb

@@ -0,0 +1,46 @@
+require 'git_diff_parser'
+
+module Danger
+  class DangerLocalRules < Plugin
+    def check
+      diff = github.pr_diff
+      return if diff.nil?
+
+      failure_rules = rules['failure']
+      warning_rules = rules['warning']
+      return if failure_rules.nil? && warning_rules.nil?
+
+      if diff.match(Regexp.union(failure_rules.keys + warning_rules.keys))
+        regexp_to_fail = Regexp.union(failure_rules.keys)
+        regexp_to_warn = Regexp.union(warning_rules.keys)
+
+        GitDiffParser.parse(diff).each do |changed_file|
+          next if changed_file.file == 'Dangerfile'
+          changed_file.changed_lines.each do |changed_line|
+            content = changed_line.content
+            # Only checks added contents
+            next unless content.start_with?('+')
+
+            if (content.match(regexp_to_fail))
+              content.match(regexp_to_fail) do |data|
+                fail(failure_rules[data[0]], file: changed_file.file, line: changed_line.number)
+              end
+            end
+
+            if (content.match(regexp_to_warn))
+              content.match(regexp_to_warn) do |data|
+                warn(warning_rules[data[0]], file: changed_file.file, line: changed_line.number)
+              end
+            end
+          end
+        end
+      end
+    end
+
+    private
+
+    def rules
+      YAML.load_file('.danger_local_rules.yml')
+    end
+  end
+end

data/spec/fixtures/pr_diff_failure_sample.txt

@@ -0,0 +1,17 @@
+diff --git a/spec/local_rules_spec.rb b/spec/local_rules_spec.rb
+index 846751e..c6d4023 100644
+--- a/spec/local_rules_spec.rb
++++ b/spec/local_rules_spec.rb
+@@ -1,41 +1,9 @@
+-require File.expand_path("../spec_helper", __FILE__)
++require File.expand_path('../spec_helper', __FILE__)
+
+ module Danger
+   describe Danger::DangerLocalRules do
+-    it "should be a plugin" do
++    it 'should be a plugin' do
++    legacy
++    typo
+       expect(Danger::DangerLocalRules.new(nil)).to be_a Danger::Pl
+ugin
+     end

data/spec/fixtures/pr_diff_sample.txt

@@ -0,0 +1,16 @@
+diff --git a/spec/local_rules_spec.rb b/spec/local_rules_spec.rb
+index 846751e..c6d4023 100644
+--- a/spec/local_rules_spec.rb
++++ b/spec/local_rules_spec.rb
+@@ -1,41 +1,9 @@
+-require File.expand_path("../spec_helper", __FILE__)
++require File.expand_path('../spec_helper', __FILE__)
+
+ module Danger
+   describe Danger::DangerLocalRules do
+-    it "should be a plugin" do
++    it 'should be a plugin' do
+
+       expect(Danger::DangerLocalRules.new(nil)).to be_a Danger::Pl
+ugin
+     end

data/spec/fixtures/pr_diff_warning_sample.txt

@@ -0,0 +1,17 @@
+diff --git a/spec/local_rules_spec.rb b/spec/local_rules_spec.rb
+index 846751e..c6d4023 100644
+--- a/spec/local_rules_spec.rb
++++ b/spec/local_rules_spec.rb
+@@ -1,41 +1,9 @@
+-require File.expand_path("../spec_helper", __FILE__)
++require File.expand_path('../spec_helper', __FILE__)
+
+ module Danger
+   describe Danger::DangerLocalRules do
+-    it "should be a plugin" do
++    it 'should be a plugin' do
++    TODO:
+
+       expect(Danger::DangerLocalRules.new(nil)).to be_a Danger::Pl
+ugin
+     end

data/spec/local_rules_spec.rb

@@ -0,0 +1,55 @@
+require File.expand_path('../spec_helper', __FILE__)
+
+module Danger
+  describe Danger::DangerLocalRules do
+    let(:dangerfile) { testing_dangerfile }
+    let(:plugin) { dangerfile.local_rules }
+    let(:pr_diff) { dummy_pr_diff }
+    let(:load_data) do
+      {
+        "failure" => {"legacy"=>"No!", "typo"=>"type"}, 
+        "warning" => {"TODO"=>"You should check"}
+      }
+    end
+
+    it 'should be a plugin' do
+      expect(Danger::DangerLocalRules.new(nil)).to be_a Danger::Plugin
+    end
+
+    describe '#check' do
+      let(:github_plugin)  { Danger::DangerfileGitHubPlugin }
+
+      before do
+        allow(YAML).to receive(:load_file).with('.danger_local_rules.yml').and_return(load_data)
+        allow_any_instance_of(github_plugin).to receive(:pr_diff).and_return(pr_diff)
+      end
+
+      context 'passed' do
+        it do
+          plugin.check
+          expect(dangerfile.status_report[:errors].length).to eq(0)
+        end
+      end
+
+      context 'diff which included string to warn' do
+        let(:pr_diff) { dummy_warning_pr_diff }
+
+        it 'should have a warning comment' do
+          plugin.check
+          expect(dangerfile.status_report[:errors].length).to eq(0)
+          expect(dangerfile.status_report[:warnings].length).to eq(1)
+        end
+      end
+
+      context 'diff which included 2 strings to fail' do
+        let(:pr_diff) { dummy_failure_pr_diff }
+
+        it 'should have 2 failure comments' do
+          plugin.check
+          expect(dangerfile.status_report[:errors].length).to eq(2)
+          expect(dangerfile.status_report[:warnings].length).to eq(0)
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,77 @@
+require "pathname"
+ROOT = Pathname.new(File.expand_path("../../", __FILE__))
+$:.unshift((ROOT + "lib").to_s)
+$:.unshift((ROOT + "spec").to_s)
+
+require "bundler/setup"
+require "pry"
+
+require "rspec"
+require "danger"
+
+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
+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
+
+def dummy_pr_diff
+  File.read(File.expand_path('../fixtures/pr_diff_sample.txt', __FILE__)).chomp
+end
+
+def dummy_warning_pr_diff
+  File.read(File.expand_path('../fixtures/pr_diff_warning_sample.txt', __FILE__)).chomp
+end
+
+def dummy_failure_pr_diff
+  File.read(File.expand_path('../fixtures/pr_diff_failure_sample.txt', __FILE__)).chomp
+end

metadata

@@ -0,0 +1,208 @@
+--- !ruby/object:Gem::Specification
+name: danger-local_rules
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+platform: ruby
+authors:
+- mataku
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2018-03-27 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: git_diff_parser
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !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: 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'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: yard
+  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'
+- !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: 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: Manage local rules by Danger.
+email:
+- nagomimatcha@gmail.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".bundle/config"
+- ".gitignore"
+- ".rubocop.yml"
+- ".travis.yml"
+- Gemfile
+- Guardfile
+- LICENSE.txt
+- README.md
+- Rakefile
+- danger-local_rules.gemspec
+- lib/danger_local_rules.rb
+- lib/danger_plugin.rb
+- lib/local_rules/gem_version.rb
+- lib/local_rules/plugin.rb
+- spec/fixtures/pr_diff_failure_sample.txt
+- spec/fixtures/pr_diff_sample.txt
+- spec/fixtures/pr_diff_warning_sample.txt
+- spec/local_rules_spec.rb
+- spec/spec_helper.rb
+homepage: https://github.com/mataku/danger-local_rules
+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.6
+signing_key: 
+specification_version: 4
+summary: Manage local rules by Danger.
+test_files:
+- spec/fixtures/pr_diff_failure_sample.txt
+- spec/fixtures/pr_diff_sample.txt
+- spec/fixtures/pr_diff_warning_sample.txt
+- spec/local_rules_spec.rb
+- spec/spec_helper.rb