Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Create with_env helper for tests. | Zuhao Wan | 2014-06-22 | 1 | -14/+13 |
| | |||||
* | Merge pull request #15856 from zuhao/refactor_activesupport_decoding_test | Yves Senn | 2014-06-22 | 1 | -11/+19 |
|\ | | | | | Use with_parse_json_times helper in tests. | ||||
| * | Use with_parse_json_times helper in tests. | Zuhao Wan | 2014-06-22 | 1 | -11/+19 |
| | | |||||
* | | Avoid hardcoded magic number in test teardown. | Zuhao Wan | 2014-06-22 | 1 | -14/+19 |
|/ | |||||
* | Add regression test for NameError#name | Arthur Neves | 2014-06-20 | 2 | -2/+5 |
| | |||||
* | Make dependencies.rb add a name to NameError | Arthur Neves | 2014-06-20 | 2 | -4/+7 |
| | |||||
* | Merge pull request #15799 from ↵ | Yves Senn | 2014-06-18 | 10 | -122/+47 |
|\ | | | | | | | | | zuhao/refactor_activesupport_time_zone_test_helpers Extract out with_env_tz helper method. | ||||
| * | Extract out with_env_tz helper method. | Zuhao Wan | 2014-06-18 | 10 | -122/+47 |
| | | | | | | | | | | It’s used at so many places that extracting it out into a helper file is worth doing. | ||||
* | | Fix documentation from #15669 [ci skip] | Zachary Scott | 2014-06-17 | 1 | -3/+5 |
| | | |||||
* | | [ci skip] fix doc for the Hash#assert_valid_keys | Aditya Kapoor | 2014-06-17 | 1 | -3/+3 |
| | | |||||
* | | Move test teardown into `ensure` block. | Zuhao Wan | 2014-06-17 | 1 | -1/+2 |
|/ | |||||
* | Formatting fix in AS changelog [ci skip] | Juanjo Bazán | 2014-06-16 | 1 | -2/+5 |
| | |||||
* | Merge pull request #15684 from aditya-kapoor/add-tests-for-assert-valid-keys | Rafael Mendonça França | 2014-06-16 | 1 | -0/+15 |
|\ | | | | | Add test cases for Hash#asset_valid_keys | ||||
| * | Add test cases for Hash#asset_valid_keys | Aditya Kapoor | 2014-06-14 | 1 | -0/+15 |
| | | |||||
* | | remove unnecessary calling of `I18n.backend.store_translations(empty, {})` | Kuldeep Aggarwal | 2014-06-15 | 2 | -13/+4 |
| | | | | | | | | | | Clean up I18n stored translations after test continuation with #15714 | ||||
* | | Merge pull request #15679 from akshay-vishnoi/test-cases | Rafael Mendonça França | 2014-06-13 | 1 | -1/+19 |
|\ \ | |/ |/| | Add missing test cases for #assert_no_difference | ||||
| * | Add missing test cases for #assert_no_difference | Akshay Vishnoi | 2014-06-13 | 1 | -1/+19 |
| | | |||||
* | | Merge pull request #15680 from akshay-vishnoi/doc_changes | Sean Griffin | 2014-06-12 | 1 | -1/+1 |
|\ \ | | | | | | | [ci skip] BigDecimal#duplicable? returns false in 1.9.x | ||||
| * | | [ci skip] BigDecimal#duplicable? returns false in 1.9.x | Akshay Vishnoi | 2014-06-13 | 1 | -1/+1 |
| |/ | |||||
* / | [ci skip] Correct output of #assert_not | Akshay Vishnoi | 2014-06-12 | 1 | -1/+1 |
|/ | |||||
* | Revert "remove unnecssary require of `to_param`, as `to_query` is already ↵ | Rafael Mendonça França | 2014-06-11 | 4 | -0/+4 |
| | | | | | | | | | | | | | | | | | | | | | require `to_param`" Revert "No need to require to_param, it is already required in to_query.rb" This reverts commits ccdd97662e1fb00c23c90d59f65c091904959561 and d697ee14264a90a39cdbe87857656d8b314ac9b7. Reason by @jeremy: These requires are not for implementation dependency. They ensure that requiring array conversions provides to_query, to_param, and other array conversion behaviors. The fact that to_query is implemented in terms of to_param is just a coincidence. If to_query removed its to_param require, then someone requiring array conversions would no longer have to_param available. This change removes these intentional dependencies on to_param in favor of implementation side effects—an undesirable move that's susceptible to regression. | ||||
* | remove unnecssary require of `to_param`, as `to_query` is already require ↵ | Kuldeep Aggarwal | 2014-06-12 | 3 | -3/+0 |
| | | | | `to_param` | ||||
* | No need to require to_param, it is already required in to_query.rb | Akshay Vishnoi | 2014-06-11 | 1 | -1/+0 |
| | |||||
* | Move test for blank.rb under object | Akshay Vishnoi | 2014-06-11 | 1 | -0/+0 |
| | |||||
* | Merge pull request #15628 from akshay-vishnoi/test-cases | Yves Senn | 2014-06-11 | 1 | -0/+4 |
|\ | | | | | Add test cases for #in? and #presence_in | ||||
| * | Add test cases for #in? and #presence_in | Akshay Vishnoi | 2014-06-11 | 1 | -0/+4 |
| | | |||||
* | | remove deprecated code. Rely on `describe` provided by minitest | Aaron Patterson | 2014-06-10 | 1 | -24/+0 |
| | | |||||
* | | Merge pull request #15616 from apanzerj/fix_long_key_paths | Rafael Mendonça França | 2014-06-10 | 3 | -0/+13 |
|\ \ | | | | | | | fix error with long keys in ActiveSupport::Cache::FileStore | ||||
| * | | fix error with long keys in ActiveSupport::Cache::FileStore | Adam Panzer | 2014-06-10 | 3 | -0/+13 |
| |/ | |||||
* / | [ci skip] Add Docs for strip_insignificant_zeros option in number helpers | Akshay Vishnoi | 2014-06-10 | 1 | -8/+7 |
|/ | |||||
* | [ci skip] Correct output and use valid options of number helpers | Akshay Vishnoi | 2014-06-10 | 1 | -2/+2 |
| | |||||
* | Fix bug that make HashWithIndifferentAccess work differently of Hash | Rafael Mendonça França | 2014-06-09 | 2 | -1/+9 |
| | | | | | Before HashWithIndifferentAccess were doing deep_dup of the inner hashes when Hash doesn't do. Now both are behaving in the same way. | ||||
* | Add Missing Test Cases for the constantize | Aditya Kapoor | 2014-06-07 | 1 | -0/+11 |
| | |||||
* | Merge pull request #15421 from gchan/time_with_zone_precision | Matthew Draper | 2014-06-05 | 3 | -1/+36 |
|\ | | | | | | | Fixed `ActiveSupport::TimeWithZone#-` so precision is not unnecessarily lost | ||||
| * | Fixed `ActiveSupport::TimeWithZone#-` so precision is not unnecessarily lost | Gordon Chan | 2014-05-30 | 3 | -1/+36 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | ||||
* | | Correct result, previously showing wrong result | Akshay Vishnoi | 2014-06-04 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #15447 from akshay-vishnoi/test-cases | Rafael Mendonça França | 2014-06-03 | 1 | -0/+8 |
|\ \ | | | | | | | Test `except!` in other cases too | ||||
| * | | Test `except!` in other cases too | Akshay Vishnoi | 2014-05-31 | 1 | -0/+8 |
| | | | |||||
* | | | drastically reduce object allocations | Aaron Patterson | 2014-06-02 | 1 | -6/+13 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before this change, we were allocating AS::SafeBuffer objects that were being interpolated in to a string, so the safe buffer object was being thrown away. This change only allocates a string (vs a string *and* a safebuffer) and interpolates the string. On my test application, this reduced the AS::SafeBuffer objects from 1527k per request to about 500 per request. | ||||
* | | | reduce AS::SafeBuffer allocations | Aaron Patterson | 2014-06-02 | 1 | -1/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | html_escape_interpolated_argument is only used in mutation methods: https://github.com/rails/rails/blob/c07d09559ec171e1904b55c7ad7e8c7d586ca51b/activesupport/lib/active_support/core_ext/string/output_safety.rb#L174 https://github.com/rails/rails/blob/c07d09559ec171e1904b55c7ad7e8c7d586ca51b/activesupport/lib/active_support/core_ext/string/output_safety.rb#L179 The return value doesn't need to be converted to an AS::SafeBuffer since we know that the current object is an AS::SafeBuffer and will be mutated, and the return value from html_escape_interpolated_argument will be thrown away | ||||
* | | | concat is a hotspot (via AV#append=), so just directly define the methods | Aaron Patterson | 2014-06-02 | 1 | -4/+6 |
| | | | |||||
* | | | Merge pull request #15379 from xuanxu/rational_precision | Rafael Mendonça França | 2014-06-02 | 3 | -5/+16 |
|\ \ \ | | | | | | | | | Fix AS::NumberHelper results with rationals | ||||
| * | | | Fix AS::NumberHelper results with rationals | Juanjo Bazán | 2014-05-31 | 3 | -5/+16 |
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | :precision was incorrectly being applied to Rationals before: ActiveSupport::NumberHelper.number_to_rounded Rational(10, 3), precision: 2 => "3.3" after: ActiveSupport::NumberHelper.number_to_rounded Rational(10, 3), precision: 2 => "3.33" | ||||
* | | | Merge pull request #15464 from tgxworld/improve_notifications_logic | Matthew Draper | 2014-06-03 | 2 | -8/+18 |
|\ \ \ | | | | | | | | | Improvements to ActiveSupport::Notifications. | ||||
| * | | | Avoid looping through subscribers when unsubscribing with a subscriber object. | Guo Xiang Tan | 2014-06-01 | 1 | -5/+10 |
| | | | | |||||
| * | | | Update documentation to reflect unsubscription with name. | Guo Xiang Tan | 2014-06-01 | 2 | -4/+9 |
| | | | | |||||
* | | | | Feature requests should be made on the mailing list, not submitted to | Zachary Scott | 2014-06-01 | 1 | -1/+6 |
|/ / / | | | | | | | | | | the issue tracker. See also #15455 [ci skip] | ||||
* / / | [ci skip] Correct output of Hash#symbolize_keys | Akshay Vishnoi | 2014-05-31 | 1 | -2/+2 |
|/ / | |||||
* | | Tiny formatting fix in AS change log [ci skip] | Robin Dupret | 2014-05-31 | 1 | -4/+4 |
| | | |||||
* | | [ci skip] Correct documentation of HashWithIndifferentAccess#dup | Akshay Vishnoi | 2014-05-31 | 1 | -2/+2 |
| | |