diff options
| author | bamchoh <[email protected]> | 2017-12-09 02:12:37 +0900 |
|---|---|---|
| committer | GitHub <[email protected]> | 2017-12-09 02:12:37 +0900 |
| commit | 81dc383291fa5c31e0b07983ae5ba8a2906e0719 (patch) | |
| tree | 8d2e05fe58dbc2d514a5401eb49309d41943fb7c | |
| parent | 550c8782162d319edcbaec3bfdeec2d269dfbdfc (diff) | |
| download | mruby-81dc383291fa5c31e0b07983ae5ba8a2906e0719.tar.gz mruby-81dc383291fa5c31e0b07983ae5ba8a2906e0719.zip | |
Use same format between Fail and Skip
We can see gem name in skip message by this fix
| -rw-r--r-- | test/assert.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/test/assert.rb b/test/assert.rb index fb7ae9ab8..a9baae5e1 100644 --- a/test/assert.rb +++ b/test/assert.rb @@ -20,7 +20,8 @@ end def assertion_string(err, str, iso=nil, e=nil, bt=nil) msg = "#{err}#{str}" msg += " [#{iso}]" if iso && iso != '' - msg += " => #{e.message}" if e + msg += " => #{e.cause}" if e && e.respond_to?(:cause) + msg += " => #{e.message}" if e && !e.respond_to?(:cause) msg += " (mrbgems: #{GEMNAME})" if Object.const_defined?(:GEMNAME) if $mrbtest_assert && $mrbtest_assert.size > 0 $mrbtest_assert.each do |idx, assert_msg, diff| @@ -56,7 +57,7 @@ def assert(str = 'Assertion failed', iso = '') rescue Exception => e bt = e.backtrace if $mrbtest_verbose if e.class.to_s == 'MRubyTestSkip' - $asserts.push "Skip: #{str} #{iso} #{e.cause}" + $asserts.push(assertion_string('Skip: ', str, iso, e, nil)) t_print('?') else $asserts.push(assertion_string("#{e.class}: ", str, iso, e, bt)) |