From: nobu@...
Date: 2015-02-01T06:32:22+00:00
Subject: [ruby-core:67928] [ruby-trunk - Misc #10809] [PATCH 2/2] Add tests for math.c related to [Feature 10785], [Feature 10800]

Issue #10809 has been updated by Nobuyoshi Nakada.


`class_eval` with a block form would be preferable to a string form in may cases.

Are you going to commit these patches by yourself?
If so, then ask matz, and follow the instruction [CommitterHowto](https://bugs.ruby-lang.org/projects/ruby/wiki/CommitterHowto)

----------------------------------------
Misc #10809: [PATCH 2/2] Add tests for math.c related to [Feature 10785], [Feature 10800]
https://bugs.ruby-lang.org/issues/10809#change-51321

* Author: kazuki tanaka
* Status: Open
* Priority: Normal
* Assignee: Nobuyoshi Nakada
----------------------------------------
Hi! Here are two patches.


## reorder_test_cbrt_and_add_a_assertion.patch:
Move Test#test_cbrt to more proper place and add a assertion for Math.cbrt(1.0/0)

## add_test_for_math_c_about_overriding_Integer_to_f.patch:

Add tests for [Feature 10785](https://bugs.ruby-lang.org/issues/10785) and  [Feature 10800](https://bugs.ruby-lang.org/issues/10800)

These two features are merged by Nobuyoshi Nakada san, so I assigned him but please feel free to reassigne.

thanks.

---Files--------------------------------
reorder_test_cbrt_and_add_a_assertion.patch (1.19 KB)
add_test_for_math_c_about_overriding_Integer_to_f.patch (1.36 KB)
use_assert_infinity.patch (2.53 KB)
revert_to_original_methods.patch (1.1 KB)


-- 
https://bugs.ruby-lang.org/