aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/test/core_ext
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Define the Duration#instance_of? methodRobin Dupret2014-08-181-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since Duration is extending from ProxyObject which extends itself from BasicObject, the Duration object doesn't respond to the #instance_of? method. Thus, the #method_missing hook get triggered, delegating the method to its `value` attribute. However, Rubinius' #eql? definition relies on #instance_of?, thus this will equal to true with a Fixnum (since its `value` attribute is a Fixnum) while it should not. The previous behavior was wrong anyway, no matter the implementation.
* | | Expectations firstAkira Matsuda2014-08-181-4/+4
| | |
* | | Fix DateTime comparison with DateTime::Infinity objectRafael Mendonça França2014-08-061-0/+1
| | | | | | | | | | | | Fixes #16406
* | | Rename tests tooRafael Mendonça França2014-08-011-2/+2
| | |
* | | Rename Object#self to Object#itself to have parity with matz sanctioned ↵David Heinemeier Hansson2014-08-011-1/+1
| | | | | | | | | | | | method name for Ruby 2.2
* | | Added Object#self which returns the object itselfDavid Heinemeier Hansson2014-08-011-0/+9
| | |
* | | Merge pull request #16356 from gchan/add-hwia-testsYves Senn2014-07-311-0/+12
|\ \ \ | | | | | | | | Add tests to ensure default proc is used when `HashWithIndifferentAccess' is initialized with a block
| * | | Add tests to ensure default proc is used when `HashWithIndifferentAccess' is ↵Gordon Chan2014-07-311-0/+12
| | | | | | | | | | | | | | | | initialized with a block
* | | | Raise a descriptive error if non-positive integer passed to in_groups_of.Xavier Shay2014-07-301-0/+6
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | This is more consistent than the current behaviour of raising a `ZeroDivisionError: divided by 0` error when 0 is given, which can be non-obvious especially if `in_groups_of` is part of a longer chain of methods. The negative case was ok - "ArgumentError: invalid slice size" - but this error is clearer still.
* | | Add missing test case for Array#to_sentence, collect all test cases for ↵Akshay Vishnoi2014-07-292-24/+16
| | | | | | | | | | | | Object#to_param at one place and avoid repitition
* | | Merge pull request #16334 from akshay-vishnoi/array-testsRafael Mendonça França2014-07-297-482/+495
|\ \ \ | | | | | | | | Move array test files under array
| * | | Move array test files under arrayAkshay Vishnoi2014-07-297-482/+495
| | | |
* | | | Added failing test case for #16131Godfrey Chan2014-07-291-0/+42
| | | |
* | | | Merge pull request #15629 from akshay-vishnoi/test-to_paramRafael Mendonça França2014-07-282-0/+22
|\ \ \ \ | |/ / / |/| | | Define Hash#to_query and set Hash#to_param as alias to it; with test cases
| * | | Move to_param to to_query, also Improve testsAkshay Vishnoi2014-06-202-0/+22
| | | |
* | | | Move object test files under objectAkshay Vishnoi2014-07-293-65/+65
| | | |
* | | | Make HWIA copy the default proc too.Arthur Neves2014-07-241-0/+11
| | | | | | | | | | | | | | | | [fixes #16279]
* | | | Merge pull request #16232 from egilburg/activesupport_coverageRafael Mendonça França2014-07-218-4/+45
|\ \ \ \ | | | | | | | | | | Added some missing activesupport test coverage
| * | | | missing activesupport test coverageEugene Gilburg2014-07-198-4/+45
| | | | |
* | | | | Remove not-required file class/subclassesAkshay Vishnoi2014-07-211-1/+0
|/ / / /
* | | | Fix typos in string_ext_test.rb [ci skip]Jon Atack2014-07-161-4/+4
| | | |
* | | | Merge pull request #16190 from oss92/word_truncationMatthew Draper2014-07-171-0/+21
|\ \ \ \ | | | | | | | | | | | | | | | Word truncation
| * | | | Added truncate_words method to activesupport stringsroot2014-07-161-0/+20
|/ / / /
* | | | Move uuid_v5 and uuid_v3 to Digest::UUIDRafael Mendonça França2014-07-152-28/+24
| | | | | | | | | | | | | | | | | | | | These methods are not random so they should not belings to SecureRandom module.
* | | | Keep quietly and capture undeprecated on your suiteRafael Mendonça França2014-07-151-19/+17
| | | |
* | | | Fix tests broken in https://github.com/rails/rails/pull/16149Erik Michaels-Ober2014-07-121-1/+1
| | | |
* | | | Return an Enumerator if no block is givenErik Michaels-Ober2014-07-122-0/+44
| | | |
* | | | `DateTime#to_f` now preserves fractional seconds.John Paul Ashenfelter2014-07-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously this method always returns `.0` in the fractional part. This commit changes it to preserve the fractional seconds instead. Fixes #15994.
* | | | Add `Hash#map_values` to ActiveSupportSean Griffin2014-06-291-0/+49
| | | | | | | | | | | | | | | | | | | | Didn't get a chance to convert existing code, I'll skim through the code base to make use of this later this afternoon.
* | | | Merge pull request #15799 from ↵Yves Senn2014-06-187-107/+25
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | zuhao/refactor_activesupport_time_zone_test_helpers Extract out with_env_tz helper method.
| * | | | Extract out with_env_tz helper method.Zuhao Wan2014-06-187-107/+25
| | | | | | | | | | | | | | | | | | | | | | | | | It’s used at so many places that extracting it out into a helper file is worth doing.
* | | | | Move test teardown into `ensure` block.Zuhao Wan2014-06-171-1/+2
|/ / / /
* | | | Add test cases for Hash#asset_valid_keysAditya Kapoor2014-06-141-0/+15
| | | |
* | | | Move test for blank.rb under objectAkshay Vishnoi2014-06-111-0/+0
| | | |
* | | | Add test cases for #in? and #presence_inAkshay Vishnoi2014-06-111-0/+4
|/ / /
* | | Fix bug that make HashWithIndifferentAccess work differently of HashRafael Mendonça França2014-06-091-0/+8
| | | | | | | | | | | | | | | Before HashWithIndifferentAccess were doing deep_dup of the inner hashes when Hash doesn't do. Now both are behaving in the same way.
* | | Merge pull request #15421 from gchan/time_with_zone_precisionMatthew Draper2014-06-051-0/+15
|\ \ \ | | | | | | | | | | | | Fixed `ActiveSupport::TimeWithZone#-` so precision is not unnecessarily lost
| * | | Fixed `ActiveSupport::TimeWithZone#-` so precision is not unnecessarily lostGordon Chan2014-05-301-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When working with objects with a nanosecond component, the `-` method may unnecessarily cause loss of precision. `ActiveSupport::TimeWithZone#-` should return the same result as if we were using `Time#-`: Time.now.end_of_day - Time.now.beginning_of_day #=> 86399.999999999 Before: Time.zone.now.end_of_day.nsec #=> 999999999 Time.zone.now.end_of_day - Time.zone.now.beginning_of_day #=> 86400.0 After: Time.zone.now.end_of_day - Time.zone.now.beginning_of_day #=> 86399.999999999
* | | | Test `except!` in other cases tooAkshay Vishnoi2014-05-311-0/+8
| | | |
* | | | :scissors: removed deprecated `Numeric#ago` and friendsGodfrey Chan2014-05-301-43/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replacements: 5.ago => 5.seconds.ago 5.until => 5.seconds.until 5.since => 5.seconds.since 5.from_now => 5.seconds.from_now The removed tests does not affect coverage – we have equivalent test cases in the tests for `AS::Duration`. See #12389 for the history and rationale behind this.
* | | Merge pull request #12080 from ShayDavidson/fix_datetime_partial_datesMatthew Draper2014-05-281-0/+6
|\ \ \ | | | | | | | | | | | | Added partial days support to `DateTime`'s `advance` method.
| * | | Added partial days support to `DateTime`'s `advance` method.Shay Davidson2013-08-301-0/+6
| | | | | | | | | | | | | | | | | | | | You can now add partial days (e.g. 2.5.days) to `DateTime` with the advance method. This was acheived by mimicing the `advance` implementation in `Time`.
* | | | Merge pull request #15276 from kuldeepaggarwal/fix-array-toRafael Mendonça França2014-05-231-0/+2
|\ \ \ \ | | | | | | | | | | Array#to now accept negative position also.
| * | | | revert 96525d63Kuldeep Aggarwal2014-05-231-0/+2
| | | | | | | | | | | | | | | | | | | | `Array#to` is working for negative position
* | | | | add test cases for negative position in Array#fromKuldeep Aggarwal2014-05-231-0/+2
|/ / / /
* | | | Merge pull request #10887 from sakuro/deep_transform_keys_in_nested_arraysRafael Mendonça França2014-05-201-0/+22
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hash#deep_*_keys(!) recurse into nested arrays. Conflicts: activesupport/CHANGELOG.md
| * | | | Hash#deep_*_keys(!) recurse into nested arrays.OZAWA Sakuro2013-06-081-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Following methods now recursively transform nested arrays, too. * Hash#deep_transform_keys * Hash#deep_transform_keys! * Hash#deep_stringify_keys * Hash#deep_stringify_keys! * Hash#deep_symbolize_keys * Hash#deep_symbolize_keys!
* | | | | Fix confusing exception in ActiveSupport delegationVladimir Yarotsky2014-05-201-1/+1
| | | | |
* | | | | Update require in `array_ext_test.rb`wendi2014-05-131-3/+2
| | | | | | | | | | | | | | | | | | | | Removed `FIMXE` tag to require necessary file but not the whole core_ext
* | | | | Update and add tests in array_ext_test.rbwendi2014-05-131-2/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Fix `test_to_with_instruct` typo to `test_to_xml_with_instruct` - Rename `test_to_xml` to `test_to_xml_with_hash_elements` to make test name more specific. - Add `test_to_xml_with_non_hash_elements` and `test_to_xml_with_non_hash_different_type_elements` `to_xml` behaves different when containing elements are same and different types. - Add `test_to_xml_with_indent_set`