diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-02-27 07:29:17 +0900 |
|---|---|---|
| committer | GitHub <[email protected]> | 2019-02-27 07:29:17 +0900 |
| commit | 018f4bd32e1b4ebeb452a361148e4d35207cf412 (patch) | |
| tree | ed7f53310e04316a69f163ae407cf068e2226fa4 | |
| parent | d544401f1f34bfe6ac1d367d0a8b534bf0601c6c (diff) | |
| parent | fdb7f9dad3be87e782831aae94188039dd0fecf9 (diff) | |
| download | mruby-018f4bd32e1b4ebeb452a361148e4d35207cf412.tar.gz mruby-018f4bd32e1b4ebeb452a361148e4d35207cf412.zip | |
Merge pull request #4299 from shuujii/remove-unneeded-separator-in-test-skip-error-messages
Remove unneeded `=>` in test skip/error messages
| -rw-r--r-- | test/assert.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/test/assert.rb b/test/assert.rb index a1d3482f4..d6e394a9f 100644 --- a/test/assert.rb +++ b/test/assert.rb @@ -20,8 +20,10 @@ end def assertion_string(err, str, iso=nil, e=nil, bt=nil) msg = "#{err}#{str}" msg += " [#{iso}]" if iso && iso != '' - msg += " => #{e.cause}" if e && e.respond_to?(:cause) - msg += " => #{e.message}" if e && !e.respond_to?(:cause) + if e + m = e.respond_to?(:cause) ? e.cause : e.message + msg += " => #{m}" if m && !m.empty? + end msg += " (#{GEMNAME == 'mruby-test' ? 'core' : "mrbgems: #{GEMNAME}"})" if $mrbtest_assert && $mrbtest_assert.size > 0 $mrbtest_assert.each do |idx, assert_msg, diff| |