Skip to content

Shindo to Minitest: common4#135

Merged
gildub merged 1 commit intofog:masterfrom
gildub:shindo_to_minitest-common4
May 25, 2016
Merged

Shindo to Minitest: common4#135
gildub merged 1 commit intofog:masterfrom
gildub:shindo_to_minitest-common4

Conversation

@gildub
Copy link
Collaborator

@gildub gildub commented May 25, 2016

Adds a migrated version of collection_helper.rb & model_helper.rb
helpers to Minitest in /test/helpers

Also, adds a UNIT_TESTS_CLEAN flag to skip broken test

module Minitest
class Test
# Some tests need to be fixed. There are skipped unless the following is true
UNIT_TESTS_CLEAN = false.freeze

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Freezing immutable objects is pointless.


# JRuby 1.7.5+ issue causes a SystemStackError: stack level too deep
# https://github.com/jruby/jruby/issues/1265
if RUBY_PLATFORM == "java" and JRUBY_VERSION =~ /1\.7\.[5-8]/

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use && instead of and.

@coveralls
Copy link

coveralls commented May 25, 2016

Coverage Status

Changes Unknown when pulling 4be03bf on gildub:shindo_to_minitest-common4 into * on fog:master*.

if collection.respond_to?(enum_method)
it "##{enum_method}" do
block_called = false
collection.send(enum_method) { block_called = true; 0 }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not use semicolons to terminate expressions.

@coveralls
Copy link

coveralls commented May 25, 2016

Coverage Status

Changes Unknown when pulling 406d597 on gildub:shindo_to_minitest-common4 into * on fog:master*.

Adds a migrated version of collection_helper.rb & model_helper.rb
helpers to Minitest in /test/helpers

Also, adds a UNIT_TESTS_CLEAN flag to skip broken test
@gildub gildub merged commit 22213c1 into fog:master May 25, 2016
@gildub
Copy link
Collaborator Author

gildub commented May 25, 2016

This is the fourth part for the common base needed for the incoming "Shindo to Minitest" sequel.
This impacts only the new Minitest set (test/) so I merged it directly for the following "Shindo to Minitests" Pull Requests to be able to be submitted.

@gildub gildub deleted the shindo_to_minitest-common4 branch May 25, 2016 13:18
@coveralls
Copy link

coveralls commented May 25, 2016

Coverage Status

Changes Unknown when pulling eb64cc0 on gildub:shindo_to_minitest-common4 into * on fog:master*.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants