diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2019-01-25 22:47:56 +0900 |
|---|---|---|
| committer | GitHub <[email protected]> | 2019-01-25 22:47:56 +0900 |
| commit | aee67eaf62235196a2d81712bee801e4134108a5 (patch) | |
| tree | 71ee176e306753bc04a5f1f7d6dc13b240db4abc | |
| parent | fc5dc5270cbca2c815d0a353075c01e0cb34b89f (diff) | |
| parent | 3707ef3d1cd2ed1ef0c2c15a05018efd0dc37273 (diff) | |
| download | mruby-aee67eaf62235196a2d81712bee801e4134108a5.tar.gz mruby-aee67eaf62235196a2d81712bee801e4134108a5.zip | |
Merge pull request #4240 from shuujii/refactor-sleep-test
Use `assert_raise` and `assert_nothing_raised` in `mruby-sleep` tests
| -rw-r--r-- | mrbgems/mruby-sleep/test/sleep_test.rb | 29 |
1 files changed, 4 insertions, 25 deletions
diff --git a/mrbgems/mruby-sleep/test/sleep_test.rb b/mrbgems/mruby-sleep/test/sleep_test.rb index 349f169b3..06a6d73fa 100644 --- a/mrbgems/mruby-sleep/test/sleep_test.rb +++ b/mrbgems/mruby-sleep/test/sleep_test.rb @@ -1,36 +1,15 @@ -def run_with_catching_error &b - e = nil - begin - b.call - rescue => _e - e = _e - end - - return e -end - assert("sleep works") do - e = run_with_catching_error { sleep 1 } - - assert_nil e + assert_nothing_raised { sleep(1) } end assert("sleep would not accept negative value") do - e = run_with_catching_error{ sleep(-1) } - - assert_not_equal e, nil - assert_equal e.class, ArgumentError + assert_raise(ArgumentError) { sleep(-1) } end assert("usleep works") do - e = run_with_catching_error { usleep 100 } - - assert_nil e + assert_nothing_raised { usleep(100) } end assert("usleep would not accept negative value") do - e = run_with_catching_error{ usleep(-100) } - - assert_not_equal e, nil - assert_equal e.class, ArgumentError + assert_raise(ArgumentError) { usleep(-100) } end |
