m 1.6.1 → 1.6.2

data/lib/m/runners/unsupported_framework.rb

@@ -6,14 +6,14 @@ module M
         []
       end
 
-      def run(_test_arguments)
+      def run _test_arguments
         not_supported
       end
 
       private
 
       def not_supported
-        STDERR.puts "This test framework is not supported! Please open up an issue at https://github.com/qrush/m !"
+        warn "This test framework is not supported! Please open up an issue at https://github.com/qrush/m !"
         false
       end
     end

data/lib/m/test_collection.rb

@@ -11,18 +11,18 @@ module M
     # internal collection
     def_delegators :@collection, :size, :<<, :each
 
-    def initialize(collection = nil)
+    def initialize collection = nil
       @collection = collection || []
     end
 
     # Slice out tests that may be within the given line.
     # Returns a new TestCollection with the results.
-    def within(lines)
+    def within lines
       # Into a new collection, filter only the tests that...
       collection = select do |test|
-        lines.none? || lines.any? { |line| (test.start_line..test.end_line).include?(line) }
+        lines.none? || lines.any? { |line| (test.start_line..test.end_line).cover?(line) }
       end
-      self.class.new(collection)
+      self.class.new collection
     end
 
     # Used to line up method names in `#sprintf` when `m` aborts
@@ -33,7 +33,7 @@ module M
     end
 
     # Be considerate when printing out tests and pre-sort them by line number
-    def by_line_number(&block)
+    def by_line_number &block
       # On each member of the collection, sort by line number and yield
       # the block into the sorted collection
       sort_by(&:start_line).each(&block)

data/lib/m/test_method.rb

@@ -8,12 +8,12 @@ module M
   #
   # Includes the name of this method, what line on the file it begins on,
   # and where it ends.
-  class TestMethod < Struct.new(:name, :start_line, :end_line)
+  TestMethod = Struct.new :name, :start_line, :end_line do
     # Set up a new test method for this test suite class
-    def self.create(suite_class, test_method)
+    def self.create suite_class, test_method
       # Hopefully it's been defined as an instance method, so we'll need to
       # look up the ruby Method instance for it
-      method = suite_class.instance_method(test_method)
+      method = suite_class.instance_method test_method
 
       # Ruby can find the starting line for us, so pull that out of the array
       start_line = method.source_location.last
@@ -24,11 +24,11 @@ module M
       #
       # The end line should be the number of line breaks in the method source,
       # added to the starting line and subtracted by one.
-      
+
       end_line = method.source.split("\n").size + start_line - 1
 
       # Shove the given attributes into a new databag
-      new(test_method, start_line, end_line)
+      new test_method, start_line, end_line
     end
   end
 end

data/lib/m/testable.rb

@@ -3,14 +3,14 @@ module M
     attr_accessor :file, :recursive, :passthrough_options
     attr_reader :lines
 
-    def initialize(file = "", lines = [], recursive = false)
+    def initialize file = "", lines = [], recursive = false
       @file = file
       @recursive = recursive
       @passthrough_options = []
       self.lines = lines
     end
 
-    def lines=(lines)
+    def lines= lines
       @lines = lines.map(&:to_i)
     end
   end

data/lib/m/version.rb

@@ -1,3 +1,3 @@
 module M
-  VERSION = "1.6.1"
+  VERSION = "1.6.2".freeze
 end

data/lib/m.rb

@@ -1,12 +1,12 @@
 ### M, your metal test runner
 # Maybe this gem should have a longer name? Metal?
-require_relative 'm/version'
-require_relative 'm/frameworks'
-require_relative 'm/runner'
+require_relative "m/version"
+require_relative "m/frameworks"
+require_relative "m/runner"
 
 module M
   # Accept arguments coming from bin/m and run tests, then bail out immediately.
-  def self.run(argv)
+  def self.run argv
     # sync output since we're going to exit hard and fast
     $stdout.sync = true
     $stderr.sync = true

data/m.gemspec

@@ -1,26 +1,25 @@
-require './lib/m'
+require "./lib/m"
 
 Gem::Specification.new do |gem|
-  gem.authors       = ["Nick Quaranto"]
-  gem.email         = ["nick@quaran.to"]
-  gem.homepage      = "https://github.com/qrush/m"
+  gem.authors = ["Nick Quaranto"]
+  gem.email = ["nick@quaran.to"]
+  gem.homepage = "https://github.com/qrush/m"
+  gem.license = "MIT"
 
-  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  gem.files         = `git ls-files`.split("\n")
-  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  gem.name          = "m"
+  gem.executables = `git ls-files -- bin/*`.split("\n").map { |f| File.basename(f) }
+  gem.files = `git ls-files`.split "\n"
+  gem.name = "m"
   gem.require_paths = ["lib"]
-  gem.version       = M::VERSION
+  gem.version = M::VERSION
 
   gem.add_runtime_dependency "method_source", ">= 0.6.7"
   gem.add_runtime_dependency "rake", ">= 0.9.2.2"
 
   gem.add_development_dependency "activesupport"
-  gem.add_development_dependency "rdiscount" unless defined? JRUBY_VERSION
-  gem.add_development_dependency "rocco" unless defined? JRUBY_VERSION
-  gem.add_development_dependency "appraisal"
+  gem.add_development_dependency "standard"
 
-  gem.required_ruby_version = ">= 1.9"
+  gem.required_ruby_version = ">= 2.7"
 
-  gem.summary = gem.description = %q{Run test/unit tests by line number. Metal!}
+  gem.summary = gem.description = "Run test/unit tests by line number. Metal!"
+  gem.metadata["rubygems_mfa_required"] = "true"
 end

data/test/Rakefile

@@ -1,6 +1,6 @@
 require "rake/testtask"
 
 Rake::TestTask.new do |t|
-  t.libs << '.'
-  t.pattern = 'examples/*_test.rb'
+  t.libs << "."
+  t.pattern = "examples/*_test.rb"
 end

data/test/active_support_test.rb

@@ -1,53 +1,53 @@
-require 'test_helper'
+require "test_helper"
 
 class ActiveSupportTest < MTest
   def test_run_simple_test_by_line_number
-    output = m('examples/active_support_example_test.rb:12')
+    output = m "examples/active_support_example_test.rb:9"
     assert_output(/1 (runs|tests), 1 assertions/, output)
   end
 
   def test_runs_entire_test_without_line_number
-    output = m('examples/active_support_example_test.rb')
+    output = m "examples/active_support_example_test.rb"
     assert_output(/4 (runs|tests)/, output)
   end
 
   def test_run_inside_of_test
-    output = m('examples/active_support_example_test.rb:13')
+    output = m "examples/active_support_example_test.rb:10"
     assert_output(/1 (runs|tests), 1 assertions/, output)
   end
 
   def test_run_on_end_of_test
-    output = m('examples/active_support_example_test.rb:14')
+    output = m "examples/active_support_example_test.rb:11"
     assert_output(/1 (runs|tests), 1 assertions/, output)
   end
 
   def test_run_inside_big_test
-    output = m('examples/active_support_example_test.rb:18')
+    output = m "examples/active_support_example_test.rb:15"
     assert_output(/1 (runs|tests), 3 assertions/, output)
   end
 
   def test_run_on_blank_line_orders_tests_by_line_number
-    output = m('examples/active_support_example_test.rb:2')
+    output = m "examples/active_support_example_test.rb:8"
 
     assert !$?.success?
-    expected = <<-EOF
-No tests found on line 2. Valid tests to run:
-
-      test_normal: m examples/active_support_example_test.rb:8
-      test_carrot: m examples/active_support_example_test.rb:12
-      test_daikon: m examples/active_support_example_test.rb:16
-test_eggplant_fig: m examples/active_support_example_test.rb:22
-EOF
+    expected = <<~OUTPUT
+      No tests found on line 8. Valid tests to run:
+
+            test_normal: m examples/active_support_example_test.rb:5
+            test_carrot: m examples/active_support_example_test.rb:9
+            test_daikon: m examples/active_support_example_test.rb:13
+      test_eggplant_fig: m examples/active_support_example_test.rb:19
+    OUTPUT
     assert_equal expected.strip, output
   end
 
   def test_run_on_test_with_spaces
-    output = m('examples/active_support_example_test.rb:22')
+    output = m "examples/active_support_example_test.rb:19"
     assert_output(/1 (runs|tests), 1 assertions/, output)
   end
 
   def test_run_on_test_with_unescaped_regular_express_characters
-    output = m('examples/active_support_unescaped_example_test.rb:8')
+    output = m "examples/active_support_unescaped_example_test.rb:5"
     assert_output(/1 (runs|tests), 1 assertions/, output)
   end
 end

data/test/allocations.rb

@@ -1,23 +1,21 @@
-$LOAD_PATH.unshift 'lib'
-require 'm'
-require 'allocation_stats'
+$LOAD_PATH.unshift "lib"
+require "m"
+require "allocation_stats"
 
-def benchmark_allocations(burn: 1)
-  stats = AllocationStats.new(burn: burn).trace do
-    yield
-  end
+def benchmark_allocations burn: 1, &block
+  stats = AllocationStats.new(burn: burn).trace(&block)
 
-  columns = if ENV['DETAIL']
-              [:sourcefile, :sourceline, :class_plus]
-            else
-              [:class_plus]
-            end
+  columns = if ENV["DETAIL"]
+    [:sourcefile, :sourceline, :class_plus]
+  else
+    [:class_plus]
+  end
 
   puts stats.allocations(alias_paths: true).group_by(*columns).sort_by_size.to_text
 end
 
 benchmark_allocations do
   10.times do
-    M::Runner.new(['test/examples/minitest_5_example_test.rb:19']).run
+    M::Runner.new(["test/examples/minitest_5_example_test.rb:19"]).run
   end
 end

data/test/bench.rb

@@ -1,35 +1,35 @@
-require 'benchmark/ips'
+require "benchmark/ips"
 
 Benchmark.ips do |bench|
   bench.report("running m on a file that doesn't exist") do
-    `ruby -Ilib ./bin/m failwhale 2>/dev/null`
+    `bundle exec ruby -Ilib ./bin/m failwhale 2>/dev/null`
   end
 
   bench.report("running m on an empty file") do
-    `ruby -Ilib ./bin/m test/examples/empty_example_test.rb 2>/dev/null`
+    `bundle exec ruby -Ilib ./bin/m test/examples/empty_example_test.rb 2>/dev/null`
   end
 
   bench.report("running m on an entire file with minitest4") do
-    `appraisal minitest4 ruby -Ilib ./bin/m test/examples/minitest_4_example_test.rb 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/minitest4.gemfile bundle exec ruby -Ilib ./bin/m test/examples/minitest_4_example_test.rb 2>/dev/null`
   end
 
   bench.report("running m on an entire file with minitest5") do
-    `appraisal minitest5 ruby -Ilib ./bin/m test/examples/minitest_5_example_test.rb 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/minitest5.gemfile bundle exec ruby -Ilib ./bin/m test/examples/minitest_5_example_test.rb 2>/dev/null`
   end
 
   bench.report("running m on an entire file with test-unit gem") do
-    `appraisal test_unit_gem ruby -Ilib ./bin/m test/examples/test_unit_example_test.rb 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/test_unit_gem.gemfile bundle exec ruby -Ilib ./bin/m test/examples/test_unit_example_test.rb 2>/dev/null`
   end
 
   bench.report("running m on a specific test with minitest4") do
-    `appraisal minitest4 ruby -Ilib ./bin/m test/examples/minitest_4_example_test.rb:19 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/minitest4.gemfile bundle exec ruby -Ilib ./bin/m test/examples/minitest_4_example_test.rb:19 2>/dev/null`
   end
 
   bench.report("running m on a specific test with minitest5") do
-    `appraisal minitest5 ruby -Ilib ./bin/m test/examples/minitest_5_example_test.rb:19 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/minitest5.gemfile bundle exec ruby -Ilib ./bin/m test/examples/minitest_5_example_test.rb:19 2>/dev/null`
   end
 
   bench.report("running m on a specific test with test-unit gem") do
-    `appraisal test_unit_gem ruby -Ilib ./bin/m test/examples/test_unit_example_test.rb:15 2>/dev/null`
+    `BUNDLE_GEMFILE=gemfiles/test_unit_gem.gemfile bundle exec ruby -Ilib ./bin/m test/examples/test_unit_example_test.rb:15 2>/dev/null`
   end
 end

data/test/empty_test.rb

@@ -1,8 +1,8 @@
-require 'test_helper'
+require "test_helper"
 
 class EmptyTest < MTest
   def test_run_simple_test_by_line_number
-    output = m('examples/empty_example_test.rb')
+    output = m "examples/empty_example_test.rb"
     assert !$?.success?
     assert_match(/There were no tests found./, output)
   end

data/test/everything_test.rb

@@ -1,13 +1,13 @@
-require 'test_helper'
+require "test_helper"
 
 class EverythingTest < MTest
   def test_runs_entire_test_suite_with_no_arguments
-    output = m('')
+    output = m ""
     assert_output(/12 (runs|tests)/, output)
   end
 
   def test_missing_file_gives_a_decent_error_message
-    output = m('examples/thisdoesnexist_test.rb')
+    output = m "examples/thisdoesnexist_test.rb"
     assert !$?.success?
     assert_match(/Failed loading test file/, output)
     if defined? JRUBY_VERSION
@@ -18,20 +18,20 @@ class EverythingTest < MTest
   end
 
   def test_running_tests_within_a_subdirectory
-    output = m('examples/subdir')
+    output = m "examples/subdir"
     assert_output(/3 (runs|tests)/, output)
 
-    output = m('examples')
+    output = m "examples"
     assert_output(/12 (runs|tests)/, output)
   end
 
   def test_running_tests_with_failures_within_a_subdirectory
-    output = m('examples/subdir_with_failures')
+    output = m "examples/subdir_with_failures"
     assert_output_for_failed_execution(/1 (runs|tests), 1 assertions, 1 failures/, output)
   end
 
   def test_blank_file_is_quieter
-    output = m('bananas')
+    output = m "bananas"
     assert(/Valid tests to run/ !~ output)
   end
 end

data/test/examples/active_support_example_test.rb

@@ -1,10 +1,7 @@
-require 'active_support'
-require 'active_support/test_case'
+require "active_support"
+require "active_support/test_case"
 
 class ActiveSupportExampleTest < ActiveSupport::TestCase
-  setup do
-  end
-
   def test_normal
     assert_equal 1, 1
   end

data/test/examples/active_support_unescaped_example_test.rb

@@ -1,10 +1,7 @@
-require 'active_support'
-require 'active_support/test_case'
+require "active_support"
+require "active_support/test_case"
 
 class ActiveSupportExampleTest < ActiveSupport::TestCase
-  setup do
-  end
-
   test "#assert_equal compares to objects (and ensures they are equal)" do
     assert_equal 1, 1
   end

data/test/examples/empty_example_test.rb

@@ -1,4 +1,4 @@
-require_relative '../test_helper'
+require_relative "../test_helper"
 
 class EmptyExampleTest < MTest
 end

data/test/examples/minitest_4_example_test.rb

@@ -1,4 +1,4 @@
-require 'minitest/unit'
+require "minitest/unit"
 
 if M::Frameworks.minitest4?
   class Meme

data/test/examples/minitest_5_example_test.rb

@@ -1,4 +1,4 @@
-require 'minitest/autorun'
+require "minitest/autorun"
 
 if M::Frameworks.minitest5?
   class Meme

data/test/examples/minitest_example_test.rb

@@ -1,4 +1,4 @@
-require 'minitest/unit'
+require "minitest/unit"
 
 class Meme
   def i_can_has_cheezburger?
@@ -32,5 +32,5 @@ class TestMeme < Test
     assert_equal "OHAI!", @meme.i_can_has_cheezburger?
   end
 
-  Minitest.after_run { p "ran after run block" } if Minitest.respond_to?(:after_run)
+  Minitest.after_run { p "ran after run block" } if Minitest.respond_to? :after_run
 end

data/test/examples/multiple_example_test.rb

@@ -1,8 +1,8 @@
-require 'active_support'
-require 'active_support/test_case'
+require "active_support"
+require "active_support/test_case"
 
 class MultipleExampleTest < ActiveSupport::TestCase
-  %w(grape habanero iceplant).each do |fruit|
+  ["grape", "habanero", "iceplant"].each do |fruit|
     test "#{fruit} is a fruit" do
       assert_equal 1, 1
     end

data/test/examples/subdir/a_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../test_helper'
+require_relative "../../test_helper"
 
 class ATest < MTest
   def test_a

data/test/examples/subdir/another_subdir/d_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../test_helper'
+require_relative "../../../test_helper"
 
 class DTest < MTest
   def test_d

data/test/examples/subdir/another_subdir/yet_another_subdir/e_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../../../test_helper'
+require_relative "../../../../test_helper"
 
 class ETest < MTest
   def test_e

data/test/examples/subdir/b_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../test_helper'
+require_relative "../../test_helper"
 
 class BTest < MTest
   def test_b

data/test/examples/subdir/c_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../test_helper'
+require_relative "../../test_helper"
 
 class CTest < MTest
   def test_c

data/test/examples/subdir_with_failures/a_test.rb

@@ -1,4 +1,4 @@
-require_relative '../../test_helper'
+require_relative "../../test_helper"
 
 class ATest < MTest
   def test_a

data/test/examples/test_unit_example_test.rb

@@ -1,5 +1,5 @@
-require_relative '../test_helper'
-try_loading('test-unit')
+require_relative "../test_helper"
+try_loading "test-unit"
 
 if M::Frameworks.test_unit?
   class TestUnitExampleTest < Test::Unit::TestCase

data/test/exit_codes_test.rb

@@ -2,32 +2,32 @@ require "test_helper"
 
 class ExitCodesTest < MTest
   def test_failing_test_returns_1
-    m("examples/subdir_with_failures/a_test")
+    m "examples/subdir_with_failures/a_test"
     refute $?.success?, "expected exit code to be 1 but it was #{$?.exitstatus}"
   end
 
   def test_test_with_error_returns_1
-    m("../lib/error_tests/error_test")
+    m "../lib/error_tests/error_test"
     refute $?.success?, "expected exit code to be 1 but it was #{$?.exitstatus}"
   end
 
   def test_dir_with_failure_returns_1
-    m("examples/subdir_with_failures")
+    m "examples/subdir_with_failures"
     refute $?.success?, "expected exit code to be 1 but it was #{$?.exitstatus}"
   end
 
   def test_dir_with_error_returns_1
-    m("../lib/error_tests")
+    m "../lib/error_tests"
     refute $?.success?, "expected exit code to be 1 but it was #{$?.exitstatus}"
   end
 
   def test_without_errors_or_failures_returns_0
-    m("examples/subdir/a_test")
+    m "examples/subdir/a_test"
     assert $?.success?, "expected exit code to be 0 but it was #{$?.exitstatus}"
   end
 
   def test_dir_without_errors_or_failures_returns_0
-    m("examples/subdir")
+    m "examples/subdir"
     assert $?.success?, "expected exit code to be 0 but it was #{$?.exitstatus}"
   end
 end

data/test/minitest_4_test.rb

@@ -1,34 +1,34 @@
-require 'test_helper'
+require "test_helper"
 
 if M::Frameworks.minitest4?
   class Minitest4Test < MTest
     def test_run_simple_test_by_line_number
-      output = m('examples/minitest_4_example_test.rb:19')
+      output = m "examples/minitest_4_example_test.rb:19"
       assert_output(/1 tests, 1 assertions/, output)
     end
 
     def test_runs_entire_test_without_line_number
-      output = m('examples/minitest_4_example_test.rb')
+      output = m "examples/minitest_4_example_test.rb"
       assert_output(/3 tests/, output)
     end
 
     def test_run_inside_of_test
-      output = m('examples/minitest_4_example_test.rb:20')
+      output = m "examples/minitest_4_example_test.rb:20"
       assert_output(/1 tests, 1 assertions/, output)
     end
 
     def test_run_on_end_of_test
-      output = m('examples/minitest_4_example_test.rb:21')
+      output = m "examples/minitest_4_example_test.rb:21"
       assert_output(/1 tests, 1 assertions/, output)
     end
 
     def test_run_inside_big_test
-      output = m('examples/minitest_4_example_test.rb:26')
+      output = m "examples/minitest_4_example_test.rb:26"
       assert_output(/1 tests, 6 assertions/, output)
     end
 
     def test_run_on_blank_line
-      output = m('examples/minitest_4_example_test.rb:2')
+      output = m "examples/minitest_4_example_test.rb:2"
 
       assert !$?.success?
       assert_match(/No tests found on line 2. Valid tests to run:/, output)

data/test/minitest_5_test.rb

@@ -1,33 +1,33 @@
-require 'test_helper'
+require "test_helper"
 if M::Frameworks.minitest5?
   class Minitest5Test < MTest
     def test_run_simple_test_by_line_number
-      output = m('examples/minitest_5_example_test.rb:19')
+      output = m "examples/minitest_5_example_test.rb:19"
       assert_output(/1 runs, 1 assertions/, output)
     end
 
     def test_runs_entire_test_without_line_number
-      output = m('examples/minitest_5_example_test.rb')
+      output = m "examples/minitest_5_example_test.rb"
       assert_output(/3 runs/, output)
     end
 
     def test_run_inside_of_test
-      output = m('examples/minitest_5_example_test.rb:20')
+      output = m "examples/minitest_5_example_test.rb:20"
       assert_output(/1 runs, 1 assertions/, output)
     end
 
     def test_run_on_end_of_test
-      output = m('examples/minitest_5_example_test.rb:21')
+      output = m "examples/minitest_5_example_test.rb:21"
       assert_output(/1 runs, 1 assertions/, output)
     end
 
     def test_run_inside_big_test
-      output = m('examples/minitest_5_example_test.rb:26')
+      output = m "examples/minitest_5_example_test.rb:26"
       assert_output(/1 runs, 6 assertions/, output)
     end
 
     def test_run_on_blank_line
-      output = m('examples/minitest_5_example_test.rb:3')
+      output = m "examples/minitest_5_example_test.rb:3"
 
       assert !$?.success?
       assert_match(/No tests found on line 3. Valid tests to run:/, output)
@@ -36,7 +36,7 @@ if M::Frameworks.minitest5?
     end
 
     def test_run_with_after_run_block
-      output = m('examples/minitest_5_example_test.rb')
+      output = m "examples/minitest_5_example_test.rb"
 
       assert_output(/ran after run block/, output)
     end