[PATCH] New specs for Range#initialize
Reported by eljay | April 19th, 2008 @ 11:01 AM | in 1.0 preview
Please find attached the missing specs for Range#initialize in ruby core specs
Passes succesfully on Ruby 1.8.6 (Linux) and rubinius
bin/mspec spec/ruby/1.8/core/range/initialize_spec.rb
...
Finished in 0.021535 seconds
1 file, 3 examples, 3 expectations, 0 failures, 0 errors
bin/mspec -t ruby spec/ruby/1.8/core/range/initialize_spec.rb
ruby 1.8.6 (2007-03-13 patchlevel 0) [i686-linux]
...
Finished in 0.015070 seconds
1 file, 3 examples, 3 expectations, 0 failures, 0 errors
Comments and changes to this ticket
-
eljay April 20th, 2008 @ 01:34 PM
OK. Shame on me... I realized that the spec I just added in Range#initialize was actually removed in the previous commit on this file by Charles Nutter. See commit e68bd05defe5ab749110af507c86769c9a036b25.
The other method I added in the spec can be migrated to the Range.new spec. So forget about this patch alltogether.
Nice try... but I'll have to find another occasion to contribute my first patch :-)
-
Federico Builes April 20th, 2008 @ 01:44 PM
- → State changed from “new” to “invalid”
eljay: The other method is already tested in the new spec so I'm closing the ticket.
Thanks anyway for taking the time and efforts to write this, hope to read more from you!
Please Login or create a free account to add a new comment.
You can update this ticket by sending an email to from your email client. (help)
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
