rubycritic 3.4.0 → 3.5.0

data/test/samples/methods_count.rb

@@ -1,7 +0,0 @@
-class Example
-  def self.one_class_method
-  end
-
-  def one_instance_method
-  end
-end

data/test/samples/module_names.rb

@@ -1,18 +0,0 @@
-module Foo
-end
-
-module Foo
-  class Bar
-  end
-
-  class Baz
-  end
-end
-
-class Foo::Qux
-end
-
-module Foo::Quux
-  class Corge
-  end
-end

data/test/samples/no_methods.rb

@@ -1,4 +0,0 @@
-module Foo
-  class NoMethods
-  end
-end

data/test/samples/reek/not_smelly.rb

@@ -1,35 +0,0 @@
-class Perfume
-  attr_reader :perfumed
-
-  def ignoreRubyStyle(oneParameter)
-    oneVariable = oneParameter
-  end
-
-  def allow_nesting_iterators_two_levels_deep
-    loop do
-      loop do
-      end
-    end
-  end
-
-  def allow_many_statements
-    do_something
-    do_something
-    do_something
-    do_something
-    do_something
-    do_something
-  end
-
-  def allow_up_to_two_duplicate_method_calls
-    respond_to do |format|
-      if success
-        format.html { redirect_to some_path }
-        format.js { head :ok }
-      else
-        format.html { redirect_to :back, status: :bad_request }
-        format.js { render status: :bad_request }
-      end
-    end
-  end
-end

data/test/samples/reek/smelly.rb

@@ -1,17 +0,0 @@
-class Theon
-  def reeks?(reek = true)
-    reek
-  end
-
-  def flayed?(a)
-    a
-  end
-end
-
-# Reek should report
-# [1]:Theon has no descriptive comment (IrresponsibleModule)
-# [2]:Theon#reeks? has boolean parameter 'reek' (BooleanParameter)
-# This comment is below the module because otherwise Reek will interpret this
-# as a comment describing the module which would thus prevent
-# IrresponsibleModule from being reported.
-# It should ignore the UncommunicativeParameterName as it's on the .todo.reek

data/test/samples/unparsable.rb

@@ -1 +0,0 @@
-unparsable :<%=

data/test/test_helper.rb

@@ -1,64 +0,0 @@
-# frozen_string_literal: true
-
-require 'minitest/autorun'
-require 'minitest/around/spec'
-require 'minitest/pride'
-require 'mocha/mini_test'
-require 'ostruct'
-require 'diff/lcs'
-
-def context(*args, &block)
-  describe(*args, &block)
-end
-
-def capture_output_streams
-  $stdout = StringIO.new
-  $stderr = StringIO.new
-  yield
-ensure
-  $stdout = STDOUT
-  $stderr = STDERR
-end
-
-def with_cloned_fs
-  FakeFS do
-    begin
-      config = File.expand_path('..', __dir__)
-      FakeFS::FileSystem.clone(config)
-      Dir.chdir(config)
-      yield
-    ensure
-      FakeFS::FileSystem.clear
-    end
-  end
-end
-
-module MiniTest
-  module Assertions
-    ##
-    # Fails unless <tt>exp</tt> and <tt>act</tt> are both arrays and
-    # contain the same elements.
-    #
-    #     assert_matched_arrays [3,2,1], [1,2,3]
-
-    def assert_matched_arrays(exp, act)
-      exp_ary = exp.to_ary
-      assert_kind_of Array, exp_ary
-      act_ary = act.to_ary
-      assert_kind_of Array, act_ary
-      diffs = Diff::LCS.sdiff(act_ary.sort, exp_ary.sort).reject(&:unchanged?)
-      assert diffs.empty?, "There are diffs between expected and actual values:\n#{diffs.map(&:inspect).join("\n")}"
-    end
-  end
-
-  module Expectations
-    ##
-    # See MiniTest::Assertions#assert_matched_arrays
-    #
-    #     [1,2,3].must_match_array [3,2,1]
-    #
-    # :method: must_match_array
-
-    infect_an_assertion :assert_matched_arrays, :must_match_array
-  end
-end