danger-spec_postfix 0.0.1 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 15c7d8c52818ceb68184f56d5cd1ec453d54b074d31b002861ec124fc2bf09d0
-  data.tar.gz: be8b28379ac2001acc446ca49481c70ea3e812aba5dc0c1bb3213cfe7f863ec4
+  metadata.gz: d7013f18e0074dbc9108182afd4409ee9457760bcf0114a3a107c4fc448ac2a3
+  data.tar.gz: 4530c9d8fdc42da2af6dc6d68e01f213fbb71b3c4fe536b182859e1f4addcb1c
 SHA512:
-  metadata.gz: e8129452944e17e50dd6821a47ec71adf77b9796cd896c5a9eb86540838dcb37586c0dd80961a5bf7a7cb95632a31799b0d393c6d996a9b89e1813d6114372a8
-  data.tar.gz: 2ba88f4ccc36da3eff3902f2e0c85bb72899b331ca5dbf4d9d88b74518a41679cbf2f4071bc6395f348a558c957723e46e0ef412bdcfcc10cc697a9641a82ae1
+  metadata.gz: 4989e8df6156ddec96e0ec46563fe823a162449de773e6f6a69e3bcde7e2fd2475c26a20ce100cd3fcad8d54025fb7c9bfc78357a13add250c04897fd9f642fe
+  data.tar.gz: 825eaf57bc666f914692af8879725ab03becca562a71aad50751b8f0a856e7e7b3eff3b60e1bcd7e8966139542cb2bfa6ef26cead6974bbac9b02e4c05336bd1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-spec_postfix (0.0.1)
+    danger-spec_postfix (0.0.3)
       danger-plugin-api (~> 1.0)
 
 GEM
@@ -96,7 +96,7 @@ GEM
       rb-inotify (>= 0.9.7)
     lumberjack (1.2.8)
     method_source (1.0.0)
-    minitest (5.16.3)
+    minitest (5.17.0)
     multipart-post (2.2.3)
     nap (1.1.0)
     nenv (0.3.0)
@@ -132,7 +132,7 @@ GEM
     rspec-expectations (3.12.2)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
-    rspec-mocks (3.12.2)
+    rspec-mocks (3.12.3)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
     rspec-support (3.12.0)

data/danger-spec_postfix.gemspec

@@ -13,7 +13,7 @@ Gem::Specification.new do |spec|
   spec.homepage      = 'https://github.com/rambler-digital-solutions/danger-spec_postfix'
   spec.license       = 'MIT'
 
-  spec.files         = `git ls-files`.split($/)
+  spec.files         = `git ls-files`.split($INPUT_RECORD_SEPARATOR)
   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']

data/lib/spec_postfix/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SpecPostfix
-  VERSION = '0.0.1'
+  VERSION = '0.0.3'
 end

data/lib/spec_postfix/plugin.rb

@@ -1,38 +1,20 @@
 # frozen_string_literal: true
 
-require 'active_support/configurable'
-
 module Danger
-  # Check that all test files in spec/ folder have `_spec` postfix.
-  # Results are passed out as a string with warning.
-  #
-  # @example Running linter
+  # Lints the test files. Will fail if any has no '_spec' postfix.
+  # Generates a `string` with warning.
   #
-  #          # Runs a linter
-  #          spec_postfix.lint
+  # @param   [String] files
+  #          A globbed string which should return the files that you want to lint, defaults to nil.
+  #          if nil, modified and added files from the diff will be used.
+  # @return  [void]
   #
   class DangerSpecPostfix < Plugin
-    # Lints the test files. Will fail if any has no '_spec' postfix.
-    # Generates a `string` with warning.
-    #
-    # @param   [String] files
-    #          A globbed string which should return the files that you want to lint, defaults to nil.
-    #          if nil, modified and added files from the diff will be used.
-    # @return  [void]
-    #
-    def self.configuration
-      @configuration ||= Configuration.new
-    end
-
-    def self.configure
-      yield configuration
-    end
-
-    def lint
+    def lint(exceptions: [])
       changed_files.select { |f| f.match?(%r{^spec/.*rb$}) }
-        .reject { |f| f.end_with?('_spec.rb') }
-        .reject { |f| DangerSpecPostfix.configuration.exceptions.any? { |e| f.start_with?(e) } }
-        .each   { |f| warn(warning_generator(f)) }
+                   .reject { |f| f.end_with?('_spec.rb') }
+                   .reject { |f| exceptions.any? { |e| f.start_with?(e) } }
+                   .each   { |f| warn(warning_generator(f)) }
     end
 
     private
@@ -45,10 +27,4 @@ module Danger
       "Tests should have `_spec` postfix: #{file}"
     end
   end
-
-  class DangerSpecPostfix::Configuration
-    include ::ActiveSupport::Configurable
-
-    config_accessor(:exceptions) { ['spec/shared_examples/', 'spec/factories/', 'spec/support/', 'spec/rails_helper.rb', 'spec/spec_helper.rb'] }
-  end
 end

data/spec/spec_helper.rb

@@ -2,8 +2,8 @@
 
 require 'pathname'
 ROOT = Pathname.new(File.expand_path('..', __dir__))
-$:.unshift("#{ROOT}lib")
-$:.unshift("#{ROOT}spec")
+$LOAD_PATH.unshift("#{ROOT}lib")
+$LOAD_PATH.unshift("#{ROOT}spec")
 
 require 'bundler/setup'
 require 'pry'

data/spec/spec_postfix_spec.rb

@@ -36,10 +36,9 @@ module Danger
       end
 
       context 'when is irrelevant (exceptions)' do
-        before do
-          described_class.configure do |config|
-            config.exceptions = 'not_tests/'
-          end
+        subject do
+          @spec_postfix.lint(exceptions: ['not_tests/'])
+          @spec_postfix.status_report[:warnings]
         end
 
         let(:file_path) { 'not_tests/factory.rb' }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: danger-spec_postfix
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.3
 platform: ruby
 authors:
 - Igor Udalov
@@ -174,25 +174,16 @@ files:
 - Gemfile
 - Gemfile.lock
 - Guardfile
-- LICENSE
 - LICENSE.txt
 - README.md
 - Rakefile
 - danger-spec_postfix.gemspec
-- danger_plugin.rb
-- danger_spec_postfix.rb
-- gem_version.rb
 - lib/danger_plugin.rb
 - lib/danger_spec_postfix.rb
 - lib/spec_postfix/gem_version.rb
 - lib/spec_postfix/plugin.rb
-- plugin.rb
 - spec/spec_helper.rb
 - spec/spec_postfix_spec.rb
-- spec_helper.rb
-- spec_postfix/gem_version.rb
-- spec_postfix/plugin.rb
-- spec_postfix_spec.rb
 homepage: https://github.com/rambler-digital-solutions/danger-spec_postfix
 licenses:
 - MIT

data/LICENSE

@@ -1,21 +0,0 @@
-MIT License
-
-Copyright (c) 2023 Rambler Digital Solutions
-
-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/danger_plugin.rb

@@ -1,3 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_postfix/plugin'

data/danger_spec_postfix.rb

@@ -1,3 +0,0 @@
-# frozen_string_literal: true
-
-require 'spec_postfix/gem_version'

data/gem_version.rb

@@ -1,5 +0,0 @@
-# frozen_string_literal: true
-
-module SpecPostfix
-  VERSION = '0.0.1'
-end

data/plugin.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require 'active_support/configurable'
-
-module Danger
-  # Check that all test files in spec/ folder have `_spec` postfix.
-  # Results are passed out as a string with warning.
-  #
-  # @example Running linter
-  #
-  #          # Runs a linter
-  #          spec_postfix.lint
-  #
-  class DangerSpecPostfix < Plugin
-    # Lints the test files. Will fail if any has no '_spec' postfix.
-    # Generates a `string` with warning.
-    #
-    # @param   [String] files
-    #          A globbed string which should return the files that you want to lint, defaults to nil.
-    #          if nil, modified and added files from the diff will be used.
-    # @return  [void]
-    #
-    def self.configuration
-      @configuration ||= Configuration.new
-    end
-
-    def self.configure
-      yield configuration
-    end
-
-    def lint
-      changed_files.select { |f| f.match?(%r{^spec/.*rb$}) }
-        .reject { |f| f.end_with?('_spec.rb') }
-        .reject { |f| DangerSpecPostfix.configuration.exceptions.any? { |e| f.start_with?(e) } }
-        .each   { |f| warn(warning_generator(f)) }
-    end
-
-    private
-
-    def changed_files
-      (git.modified_files + git.added_files)
-    end
-
-    def warning_generator(file)
-      "Tests should have `_spec` postfix: #{file}"
-    end
-  end
-
-  class DangerSpecPostfix::Configuration
-    include ::ActiveSupport::Configurable
-
-    config_accessor(:exceptions) { ['spec/shared_examples/', 'spec/factories/', 'spec/support/', 'spec/rails_helper.rb', 'spec/spec_helper.rb'] }
-  end
-end

data/spec_helper.rb

@@ -1,67 +0,0 @@
-# frozen_string_literal: true
-
-require 'pathname'
-ROOT = Pathname.new(File.expand_path('..', __dir__))
-$:.unshift("#{ROOT}lib")
-$:.unshift("#{ROOT}spec")
-
-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

data/spec_postfix/gem_version.rb

@@ -1,5 +0,0 @@
-# frozen_string_literal: true
-
-module SpecPostfix
-  VERSION = '0.0.1'
-end

data/spec_postfix/plugin.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require 'active_support/configurable'
-
-module Danger
-  # Check that all test files in spec/ folder have `_spec` postfix.
-  # Results are passed out as a string with warning.
-  #
-  # @example Running linter
-  #
-  #          # Runs a linter
-  #          spec_postfix.lint
-  #
-  class DangerSpecPostfix < Plugin
-    # Lints the test files. Will fail if any has no '_spec' postfix.
-    # Generates a `string` with warning.
-    #
-    # @param   [String] files
-    #          A globbed string which should return the files that you want to lint, defaults to nil.
-    #          if nil, modified and added files from the diff will be used.
-    # @return  [void]
-    #
-    def self.configuration
-      @configuration ||= Configuration.new
-    end
-
-    def self.configure
-      yield configuration
-    end
-
-    def lint
-      changed_files.select { |f| f.match?(%r{^spec/.*rb$}) }
-        .reject { |f| f.end_with?('_spec.rb') }
-        .reject { |f| DangerSpecPostfix.configuration.exceptions.any? { |e| f.start_with?(e) } }
-        .each   { |f| warn(warning_generator(f)) }
-    end
-
-    private
-
-    def changed_files
-      (git.modified_files + git.added_files)
-    end
-
-    def warning_generator(file)
-      "Tests should have `_spec` postfix: #{file}"
-    end
-  end
-
-  class DangerSpecPostfix::Configuration
-    include ::ActiveSupport::Configurable
-
-    config_accessor(:exceptions) { ['spec/shared_examples/', 'spec/factories/', 'spec/support/', 'spec/rails_helper.rb', 'spec/spec_helper.rb'] }
-  end
-end

data/spec_postfix_spec.rb

@@ -1,51 +0,0 @@
-# frozen_string_literal: true
-
-require File.expand_path('spec_helper', __dir__)
-
-module Danger
-  describe Danger::DangerSpecPostfix do
-    it 'is a plugin' do
-      expect(described_class.new(nil)).to be_a Danger::Plugin
-    end
-
-    describe 'with Dangerfile' do
-      let(:file_path) { nil }
-
-      before do
-        @spec_postfix = testing_dangerfile.spec_postfix
-
-        allow(@spec_postfix.git).to receive(:added_files).and_return([])
-        allow(@spec_postfix.git).to receive(:modified_files).and_return([file_path])
-      end
-
-      subject do
-        @spec_postfix.lint
-        @spec_postfix.status_report[:warnings]
-      end
-
-      context 'with _spec' do
-        let(:file_path) { 'spec/models/my_test_spec.rb' }
-
-        it { is_expected.to be_empty }
-      end
-
-      context 'with no _spec' do
-        let(:file_path) { 'spec/models/my_test.rb' }
-
-        it { is_expected.to eq(["Tests should have `_spec` postfix: #{file_path}"]) }
-      end
-
-      context 'when is irrelevant (exceptions)' do
-        before do
-          described_class.configure do |config|
-            config.exceptions = 'not_tests/'
-          end
-        end
-
-        let(:file_path) { 'not_tests/factory.rb' }
-
-        it { is_expected.to be_empty }
-      end
-    end
-  end
-end