m 1.6.1 → 1.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e82a11a5dd9af1f6023bb8150265770adee0b322f21878d88cad02fdd0b99430
-  data.tar.gz: 5734c191624c0b77e27b3c935e4388168fc5d6fbb673182447b69530d3b9d347
+  metadata.gz: 3242d76170ae6e9c1e655bca994edab9289d44424ec463f26909d87b759a7b65
+  data.tar.gz: 7bade78b916378359932c6a9fb6129a8c4a62c2a76ab1214103db34c2c88c55c
 SHA512:
-  metadata.gz: 6a7e89d0ff0ae83bc62d5b8de773fb3120fcd8b0db996ba8795ee11f7016ad34d722544f513fd06fda35c0cd2db2f27f16dc45241e9a839190466bb8d8be6ddf
-  data.tar.gz: dd72cc68f9f91173b455818ad1c503b868158fedbae55c1684246223c209c87f0299513af6ed9a26381a19ef30bc73ca58333a357de2ca2a1f7202d92d48ede2
+  metadata.gz: a83741c31e17d531dca7255834de1be852aef6076da60fd4425c7e3970b10d7ef30be4d60fdd12ad11cf7a47dd45e75a2afc06897d306c76c41982762b1bdb75
+  data.tar.gz: 9666eff8dcb0241fa07645289d6d1bc4805552c709d96659ed3ed408ca9d0e3af209661000145b8aaafd1acb34933d94923bf3e7473619c7d056ad214d48ea84

data/.github/workflows/ruby-ci.yml

@@ -13,17 +13,17 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        appraisal:
+        gemfile:
           - minitest4
           - minitest5
           - test_unit_gem
         ruby:
-          - ruby-2.6
           - ruby-2.7
           - ruby-3.0
+          - ruby-3.1
           - truffleruby
     env:
-      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.appraisal }}.gemfile
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
     steps:
       - name: Checkout
         uses: actions/checkout@v2

data/.gitignore

@@ -3,6 +3,7 @@
 .bundle
 .config
 .yardoc
+gemfiles/*.gemfile.lock
 Gemfile.lock
 InstalledFiles
 _yardoc

data/.standard.yml

@@ -0,0 +1 @@
+ruby_version: 2.7

data/Gemfile

@@ -1,10 +1,8 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
-gem 'coveralls', require: false
-gem 'benchmark-ips'
+gem "benchmark-ips"
+gem "coveralls", require: false
 
-if RUBY_VERSION !~ /^2.0/ && !defined?(JRUBY_VERSION)
-  gem 'allocation_stats'
-end
+gem "allocation_stats" if RUBY_VERSION !~ /^2.0/ && !defined?(JRUBY_VERSION)
 
 gemspec

data/README.md

@@ -20,23 +20,10 @@ If you’re using Bundler, you’ll need to include it in your Gemfile. Toss it
 
 ``` ruby
 group :test do
-  gem 'm', '~> 1.5.0'
+  gem "m"
 end
 ```
 
-Developing a RubyGem? Add m as a development dependency.
-
-
-``` ruby
-Gem::Specification.new do |gem|
-  # ...
-  gem.add_development_dependency "m", "~> 1.5.0"
-end
-```
-
-m works on Ruby 2.0+ only and support is only provided for [versions currently maintained by the community](https://www.ruby-lang.org/en/downloads/branches/).
-
-
 USAGE
 =====
 
@@ -138,15 +125,6 @@ SUPPORT
 CONTRIBUTING
 ============
 
-## Setup
-
-This project uses [Appraisal](https://github.com/thoughtbot/appraisal) to test against different versions of dependencies.
-
-To install all scenarios (appraisals):
-
-    bundle install
-    bundle exec appraisal install
-
 ## Testing
 
 You can run all the tests with:
@@ -155,11 +133,11 @@ You can run all the tests with:
 
 You can also run tests selectively. For minitest 4 run:
 
-    appraisal minitest4 rake test
+    bundle exec rake test:minitest4
 
 and the ones for minitest 5 with:
 
-    appraisal minitest5 rake test
+    bundle exec rake test:minitest5
 
 
 LICENSE

data/Rakefile

@@ -1,96 +1,48 @@
 #!/usr/bin/env rake
-require 'rubygems'
-require 'bundler/setup'
-require 'appraisal'
-require 'coveralls'
-require 'bundler/gem_tasks'
-require 'rake/clean'
-require 'rake/testtask'
+require "rubygems"
+require "bundler/setup"
+require "coveralls"
+require "bundler/gem_tasks"
+require "rake/clean"
+require "rake/testtask"
+require "standard/rake"
 
-task :default => [:test]
+task default: [:test, "standard:fix"]
 
 Rake::TestTask.new do |t|
-  t.libs << 'test'
-  t.libs << 'lib'
-  t.pattern = 'test/*_test.rb'
+  t.libs << "test"
+  t.libs << "lib"
+  t.pattern = "test/*_test.rb"
+end
+
+desc "Run all tests and get merged test coverage"
+namespace :test do
+  Dir.glob("gemfiles/*.gemfile").each do |gemfile_path|
+    name = /gemfiles\/(.*).gemfile/.match(gemfile_path)[1]
+    desc "Run #{name} tests"
+    task name do |rake_task|
+      gemfile_name = rake_task.name.split(":").last
+      gemfile_path = "gemfiles/#{gemfile_name}.gemfile"
+      Bundler.with_original_env do
+        sh "BUNDLE_GEMFILE=#{gemfile_path} bundle exec rake"
+      end
+    end
+  end
 end
 
-desc 'Run all tests and get merged test coverage'
+desc "Run all tests and get merged test coverage"
 task :tests do
-  system "rake test" or exit!(1)
-  system "appraisal minitest4 rake test" or exit!(1)
-  system "appraisal minitest5 rake test TEST=test/minitest_5_test.rb" or exit!(1)
-  system "appraisal test_unit_gem rake test TEST=test/test_unit_test.rb" or exit!(1)
+  Dir.glob("gemfiles/*.gemfile").each do |gemfile_path|
+    Bundler.with_original_env do
+      sh "BUNDLE_GEMFILE=#{gemfile_path} bundle exec rake"
+    end
+  end
   Coveralls.push!
 end
 
-desc 'Run simple benchmarks'
+desc "Run simple benchmarks"
 task :bench do
   current_commit = `git rev-parse HEAD`
-  file_name = "benchmarks/#{Time.now.strftime('%Y%m%d')}-benchmark.log"
+  file_name = "benchmarks/#{Time.now.strftime "%Y%m%d"}-benchmark.log"
   exec "echo -e 'Data for commit: #{current_commit}' > #{file_name} && ruby test/bench.rb >> #{file_name}"
 end
-
-# ROCCO ===============================================================
-
-begin
-  require 'rdiscount'
-rescue LoadError => e
-  warn e.inspect
-end
-begin
-  require 'rocco/tasks'
-  Rocco::make 'docs/'
-rescue LoadError => e
-  warn e.inspect
-end
-
-desc 'Build rocco docs'
-task :docs => :rocco
-directory 'docs/'
-
-desc 'Build docs and open in browser for the reading'
-task :read => :docs do
-  sh 'open docs/lib/m.html'
-end
-
-# Make index.html a copy of rocco.html
-file 'docs/index.html' => 'docs/lib/m.html' do |f|
-  cp 'docs/lib/m.html', 'docs/index.html', :preserve => true
-end
-
-task :docs => 'docs/index.html'
-CLEAN.include 'docs/index.html'
-
-# Alias for docs task
-task :doc => :docs
-
-# GITHUB PAGES ===============================================================
-
-desc "really kill docs folder"
-task :clean_docs do
-  sh "rm -rf docs/"
-end
-
-desc 'Update gh-pages branch'
-task :pages => [:clean_docs, 'docs/.git', :docs] do
-  rev = `git rev-parse --short HEAD`.strip
-  Dir.chdir 'docs' do
-    sh "mv lib/m m"
-    sh "mv lib/m.html m.html"
-    sh "git add -A"
-    sh "git commit -m 'rebuild pages from #{rev}'" do |ok,res|
-      if ok
-        verbose { puts "gh-pages updated" }
-        sh "git push -q o HEAD:gh-pages"
-      end
-    end
-  end
-end
-
-# Update the pages/ directory clone
-file 'docs/.git' => ['docs/'] do |f|
-  sh "cd docs && git init -q && git remote add o ../.git" if !File.exist?(f.name)
-  sh "cd docs && git fetch -q o && git reset -q --hard o/gh-pages && git rm -r . && git commit -m 'blank out' && touch ."
-end
-CLOBBER.include 'docs/.git'

data/bin/m

@@ -1,4 +1,4 @@
 #!/usr/bin/env ruby
 
-require 'm'
-M.run(ARGV)
+require "m"
+M.run ARGV

data/gemfiles/minitest4.gemfile

@@ -1,10 +1,8 @@
-# This file was generated by Appraisal
-
 source "https://rubygems.org"
 
-gem "coveralls", require: false
-gem "benchmark-ips"
 gem "allocation_stats"
+gem "benchmark-ips"
+gem "coveralls", require: false
 gem "minitest", "~> 4"
 
 gemspec path: "../"

data/gemfiles/minitest5.gemfile

@@ -1,10 +1,8 @@
-# This file was generated by Appraisal
-
 source "https://rubygems.org"
 
-gem "coveralls", require: false
-gem "benchmark-ips"
 gem "allocation_stats"
+gem "benchmark-ips"
+gem "coveralls", require: false
 gem "minitest", "~> 5"
 
 gemspec path: "../"

data/gemfiles/test_unit_gem.gemfile

@@ -1,10 +1,8 @@
-# This file was generated by Appraisal
-
 source "https://rubygems.org"
 
-gem "coveralls", require: false
-gem "benchmark-ips"
 gem "allocation_stats"
+gem "benchmark-ips"
+gem "coveralls", require: false
 gem "test-unit"
 
 gemspec path: "../"

data/lib/error_tests/error_test.rb

@@ -1,4 +1,4 @@
-require_relative "../../test/test_helper.rb"
+require_relative "../../test/test_helper"
 
 class ErrorTest < MTest
   def test_purposeful_error

data/lib/m/executor.rb

@@ -1,46 +1,46 @@
-require_relative 'runners/base'
-require_relative 'runners/minitest_5'
-require_relative 'runners/minitest_4'
-require_relative 'runners/test_unit'
-require_relative 'runners/unsupported_framework'
+require_relative "runners/base"
+require_relative "runners/minitest_5"
+require_relative "runners/minitest_4"
+require_relative "runners/test_unit"
+require_relative "runners/unsupported_framework"
 
 module M
   class Executor
-    def initialize(testable)
+    def initialize testable
       @testable = testable
     end
 
     def execute
       # Locate tests to run that may be inside of this line. There could be more than one!
-      tests_to_run = tests.within(testable.lines)
+      tests_to_run = tests.within testable.lines
 
       # If we found any tests,
-      if tests_to_run.size > 0
+      if tests_to_run.size.positive?
         # assemble the regexp to run these tests,
-        test_names = tests_to_run.map { |test| Regexp.escape(test.name) }.join('|')
+        test_names = tests_to_run.map { |test| Regexp.escape(test.name) }.join("|")
 
         # set up the args needed for the runner
         test_arguments = ["-n", "/^(#{test_names})$/"]
 
         # directly run the tests from here and exit with the status of the tests passing or failing
-        runner.run(test_arguments + testable.passthrough_options)
-      elsif tests.size > 0
+        runner.run test_arguments + testable.passthrough_options
+      elsif tests.size.positive?
         # Otherwise we found no tests on this line, so you need to pick one.
-        message = "No tests found on line #{testable.lines.join(', ')}. Valid tests to run:\n\n"
+        message = "No tests found on line #{testable.lines.join ", "}. Valid tests to run:\n\n"
 
         # For every test ordered by line number,
         # spit out the test name and line number where it starts,
         tests.by_line_number do |test|
-          message << "#{sprintf("%0#{tests.column_size}s", test.name)}: m #{testable.file}:#{test.start_line}\n"
+          message << "#{format "%0#{tests.column_size}s", test.name}: m #{testable.file}:#{test.start_line}\n"
         end
 
         # Spit out helpful message and bail
-        STDERR.puts message
+        warn message
         false
       else
         # There were no tests at all
         message = "There were no tests found.\n\n"
-        STDERR.puts message
+        warn message
         false
       end
     end
@@ -58,27 +58,25 @@ module M
         # With each suite and array of tests,
         # and with each test method present in this test file,
         # shove a new test method into this collection.
-        suites.inject(TestCollection.new) do |collection, (suite_class, test_methods)|
+        suites.each_with_object TestCollection.new do |(suite_class, test_methods), collection|
           test_methods.each do |test_method|
             collection << TestMethod.create(suite_class, test_method)
           end
-        collection
         end
       end
     end
 
-
     # Finds all test suites in this test file, with test methods included.
     def suites
       # Since we're not using `ruby -Itest -Ilib` to run the tests, we need to add this directory to the `LOAD_PATH`
-      $:.unshift "./test", "./spec", "./lib"
+      $LOAD_PATH.unshift "./test", "./spec", "./lib"
 
       begin
         # Fire up this Ruby file. Let's hope it actually has tests.
         require "./#{testable.file}"
       rescue LoadError => e
         # Fail with a happier error message instead of spitting out a backtrace from this gem
-        STDERR.puts "Failed loading test file:\n#{e.message}"
+        warn "Failed loading test file:\n#{e.message}"
         return []
       end
 
@@ -87,9 +85,7 @@ module M
       # Use some janky internal APIs to group test methods by test suite.
       suites.each_with_object({}) do |suite_class, test_suites|
         # End up with a hash of suite class name to an array of test methods, so we can later find them and ignore empty test suites
-        if runner.test_methods(suite_class).any?
-          test_suites[suite_class] = runner.test_methods(suite_class)
-        end
+        test_suites[suite_class] = runner.test_methods(suite_class) if runner.test_methods(suite_class).any?
       end
     end
 

data/lib/m/frameworks.rb

@@ -1,5 +1,25 @@
 module M
   class Frameworks
+    def self.minitest_version_major
+      if defined?(Minitest::Unit::VERSION)
+        Minitest::Unit::VERSION.slice(/\d+/)
+      elsif defined?(Minitest::VERSION)
+        Minitest::VERSION.slice(/\d+/)
+      end
+    end
+
+    def self.minitest5?
+      minitest_version_major == "5"
+    end
+
+    def self.minitest4?
+      minitest_version_major == "4"
+    end
+
+    def self.test_unit?
+      defined?(Test::Unit)
+    end
+
     def self.framework_runner
       new.framework_runner
     end
@@ -29,21 +49,5 @@ module M
     def test_unit?
       self.class.test_unit?
     end
-
-    def self.minitest_version_major
-      defined?(Minitest) ? Minitest::Unit::VERSION.slice(/\d+/) : nil
-    end
-
-    def self.minitest5?
-      minitest_version_major == "5"
-    end
-
-    def self.minitest4?
-      minitest_version_major == "4"
-    end
-
-    def self.test_unit?
-      defined?(Test::Unit)
-    end
   end
 end

data/lib/m/parser.rb

@@ -1,8 +1,8 @@
-require_relative 'testable'
+require_relative "testable"
 
 module M
   class Parser
-    def initialize(argv)
+    def initialize argv
       @argv = argv
       @testable = Testable.new
     end
@@ -15,12 +15,12 @@ module M
       else
         parse_options! argv
 
-        if argv.first.start_with?("--")
+        if argv.first.start_with? "--"
           exec "rake test #{argv.join}"
           exit 0
         else
           # Parse out ARGV, it should be coming in in a format like `test/test_file.rb:9:19`
-          parsed = argv.shift.split(':')
+          parsed = argv.shift.split ":"
           testable.file = parsed.shift
           testable.lines = parsed if testable.lines.none?
           # Anything else on ARGV will be passed along to the runner
@@ -28,26 +28,25 @@ module M
         end
 
         # If this file is a directory, not a file, run the tests inside of this directory
-        if Dir.exist?(testable.file)
-          # Make a new rake test task with a hopefully unique name, and run every test looking file in it
-          require "rake/testtask"
-          Rake::TestTask.new(:m_custom) do |t|
-            t.libs << 'test'
-            t.libs << 'spec'
-            t.test_files = FileList[wildcard("test"), wildcard("spec")]
-            t.warning = false
-          end
-          # Invoke the rake task and exit, hopefully it'll work!
-          begin
-            Rake::Task['m_custom'].invoke
-          rescue RuntimeError
-            exit(1)
-          ensure
-            exit($?.exitstatus)
-          end
-        else
-          return testable
+        return testable unless Dir.exist? testable.file
+
+        # Make a new rake test task with a hopefully unique name, and run every test looking file in it
+        require "rake/testtask"
+        Rake::TestTask.new :m_custom do |t|
+          t.libs << "test"
+          t.libs << "spec"
+          t.test_files = FileList[wildcard("test"), wildcard("spec")]
+          t.warning = false
         end
+        # Invoke the rake task and exit, hopefully it'll work!
+        begin
+          Rake::Task["m_custom"].invoke
+        rescue RuntimeError
+          exit 1
+        ensure
+          exit $?.exitstatus
+        end
+
       end
     end
 
@@ -55,29 +54,29 @@ module M
 
     attr_reader :argv, :testable
 
-    def parse_options!(argv)
-      require 'optparse'
+    def parse_options! argv
+      require "optparse"
 
       OptionParser.new do |opts|
-        opts.banner  = 'Options:'
+        opts.banner = "Options:"
         opts.version = M::VERSION
 
-        opts.on '-h', '--help', 'Display this help.' do
+        opts.on "-h", "--help", "Display this help." do
           puts "Usage: m [OPTIONS] [FILES]\n\n", opts
           exit
         end
 
-        opts.on '--version', 'Display the version.' do
+        opts.on "--version", "Display the version." do
           puts "m #{M::VERSION}"
           exit
         end
 
-        opts.on '-l', '--line LINE', Integer, 'Line number for file.' do |line|
+        opts.on "-l", "--line LINE", Integer, "Line number for file." do |line|
           p "parsing line #{line}"
           testable.lines = [line]
         end
 
-        opts.on '-r', '--recursive DIR', 'Search provided directory recursively.' do |directory|
+        opts.on "-r", "--recursive DIR", "Search provided directory recursively." do |directory|
           testable.recursive = true
           argv << directory
         end
@@ -86,7 +85,7 @@ module M
       end
     end
 
-    def wildcard(type)
+    def wildcard type
       if testable.recursive
         "#{testable.file}/**/*#{type}*.rb"
       else

data/lib/m/runner.rb

@@ -1,11 +1,11 @@
-require_relative 'parser'
-require_relative 'executor'
+require_relative "parser"
+require_relative "executor"
 
 ### Runners are in charge of running your tests, depending on the framework
 # Instead of slamming all of this junk in an `M` class, it's here instead.
 module M
   class Runner
-    def initialize(argv)
+    def initialize argv
       @argv = argv
     end
 

data/lib/m/runners/base.rb

@@ -2,14 +2,14 @@ module M
   module Runners
     class Base
       def suites
-        raise 'Not implemented'
+        raise "Not implemented"
       end
 
-      def run(_test_arguments)
-        raise 'Not implemented'
+      def run _test_arguments
+        raise "Not implemented"
       end
 
-      def test_methods(suite_class)
+      def test_methods suite_class
         suite_class.test_methods
       end
     end

data/lib/m/runners/minitest_4.rb

@@ -5,7 +5,7 @@ module M
         MiniTest::Unit::TestCase.test_suites
       end
 
-      def run(test_arguments)
+      def run test_arguments
         MiniTest::Unit.runner.run test_arguments
       end
     end

data/lib/m/runners/minitest_5.rb

@@ -2,19 +2,17 @@ module M
   module Runners
     class Minitest5 < Base
       def suites
-        if Minitest.respond_to?(:seed)
-          Minitest.seed = (ENV["SEED"] || srand).to_i % 0xFFFF
-        end
+        Minitest.seed = (ENV["SEED"] || srand).to_i % 0xFFFF if Minitest.respond_to? :seed
         Minitest::Runnable.runnables
       end
 
-      def run(test_arguments)
+      def run test_arguments
         output = Minitest.run test_arguments
         ::Minitest.class_variable_get(:@@after_run).reverse_each(&:call)
         output
       end
 
-      def test_methods(suite_class)
+      def test_methods suite_class
         suite_class.runnable_methods
       end
     end

data/lib/m/runners/test_unit.rb

@@ -2,22 +2,22 @@ module M
   module Runners
     class TestUnit < Base
       def suites
-        if Test::Unit::TestCase.respond_to?(:test_suites)
+        if Test::Unit::TestCase.respond_to? :test_suites
           Test::Unit::TestCase.test_suites
         else
           Test::Unit::TestCase::DESCENDANTS
         end
       end
 
-      def run(test_arguments)
-        Test::Unit::AutoRunner.run(false, nil, test_arguments)
+      def run test_arguments
+        Test::Unit::AutoRunner.run false, nil, test_arguments
       end
 
-      def test_methods(suite_class)
-        if suite_class.respond_to?(:test_methods)
+      def test_methods suite_class
+        if suite_class.respond_to? :test_methods
           suite_class.test_methods
         else
-          suite_class.public_instance_methods(true).grep(/^test/).map { |m| m.to_s }
+          suite_class.public_instance_methods(true).grep(/^test/).map(&:to_s)
         end
       end
     end