Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Get rid of the underscore versions of attribute methods! | Jon Leighton | 2011-12-01 | 1 | -0/+5 | |
| | | | | This makes me happy! | |||||
* | don't alter global state in test | Jon Leighton | 2011-11-30 | 1 | -2/+3 | |
| | ||||||
* | If the table behind has no primary key, do not ask again and just return nil. | Julius de Bruijn | 2011-11-30 | 1 | -0/+5 | |
| | ||||||
* | omg computer science! | Jon Leighton | 2011-11-30 | 1 | -5/+1 | |
| | | | | | | | Implement a mini state machine for serialized attributes. This means we do not have to deserialize the values upon initialization, which means that if we never actually access the attribute, we never have to deserialize it. | |||||
* | Use inheritance to avoid special-case code for the 'id' method | Jon Leighton | 2011-11-30 | 1 | -1/+1 | |
| | ||||||
* | Update ActiveRecord#attribute_present? to work as documented | Justin Mazzi | 2011-11-05 | 1 | -0/+18 | |
| | | | | | | | "Returns true if the specified attribute has been set by the user or by a database load and is neither nil nor empty?" Fixes #1613 | |||||
* | Stop trying to be clever about when to define attribute methods. | Jon Leighton | 2011-09-14 | 1 | -24/+0 | |
| | | | | | | | | | There is no meaningful performance penalty in defining attribute methods, since it only happens once. There is also no reason *not* to define them, since they get thrown in an included module, so they will not 'overwrite' anything. In fact, this is desirable, since it allows us to call super. | |||||
* | Deprecate using method_missing for attributes that are columns. | Jon Leighton | 2011-09-13 | 1 | -0/+15 | |
| | | | | | | This shouldn't ever happen unless people are doing something particularly weird, but adding a deprecation in case there are bugs not caught by our tests. | |||||
* | Let Ruby deal with method visibility. | Jon Leighton | 2011-09-13 | 1 | -3/+3 | |
| | | | | | | Check respond_to_without_attributes? in method_missing. If there is any method that responds (even private), let super handle it and raise NoMethodError if necessary. | |||||
* | Always generate attribute methods on the base class. | Jon Leighton | 2011-09-13 | 1 | -0/+16 | |
| | | | | | | | | | | | | | | This fixes a situation I encountered where a subclass would cache the name of a generated attribute method in @_defined_class_methods. Then, when the superclass has it's attribute methods undefined, the subclass would always have to dispatch through method_missing, because the presence of the attribute in @_defined_class_methods would mean that it is never generated again, even if undefine_attribute_methods is called on the subclass. There various other confusing edge cases like this. STI classes share columns, so let's just keep all the attribute method generation state isolated to the base class. | |||||
* | Revert "Merge pull request #2543 from htanata/unused_require_in_ar_test" | Santiago Pastorino | 2011-08-16 | 1 | -0/+1 | |
| | | | | | This reverts commit 87152f2604e73b218df90befda576f0acfed0bbf, reversing changes made to 0d3615f04c79f6e90d8ab33fdfc920b8faac9cb8. | |||||
* | Remove unused require. | Hendy Tanata | 2011-08-16 | 1 | -1/+0 | |
| | ||||||
* | Resolve warnings by instantizing @attrubtes as nil | Brian Cardarella | 2011-07-30 | 1 | -0/+1 | |
| | ||||||
* | please use ruby -I lib:test path/to/test.rb, or export RUBY_OPT | Aaron Patterson | 2011-06-06 | 1 | -1/+1 | |
| | ||||||
* | Refactor Active Record test connection setup. Please see the ↵ | Jon Leighton | 2011-06-04 | 1 | -1/+1 | |
| | | | | RUNNING_UNIT_TESTS file for details, but essentially you can now configure things in test/config.yml. You can also run tests directly via the command line, e.g. ruby path/to/test.rb (no rake needed, uses default db connection from test/config.yml). This will help us fix the CI by enabling us to isolate the different Rails versions to different databases. | |||||
* | Fix broken test. You know, the merge button will be the end of us... | Jon Leighton | 2011-06-01 | 1 | -3/+3 | |
| | ||||||
* | Adding comment | Arun Agrawal | 2011-06-01 | 1 | -0/+1 | |
| | ||||||
* | AttributeMethodTest test fix #jruby | Vishnu Atrai | 2011-06-01 | 1 | -1/+5 | |
| | ||||||
* | Test AR::Base#inspect on an allocated object not yet initialized | Franck Verrot | 2011-05-25 | 1 | -0/+8 | |
| | ||||||
* | Undo performances regressions I introduced in ↵ | Jon Leighton | 2011-04-15 | 1 | -0/+11 | |
| | | | | bbe0a507f287c20ab4ae8a244fbfc810665deda5 and add test for an edge case. Add comments to explain the intent of the code. | |||||
* | Remove unnecessary code from define_read_method and add assertion to make ↵ | Jon Leighton | 2011-04-15 | 1 | -0/+1 | |
| | | | | sure the underscored version is actually generated | |||||
* | Remove `#among?` from Active Support | Prem Sichanugrist | 2011-04-13 | 1 | -1/+1 | |
| | | | | | | After a long list of discussion about the performance problem from using varargs and the reason that we can't find a great pair for it, it would be best to remove support for it for now. It will come back if we can find a good pair for it. For now, Bon Voyage, `#among?`. | |||||
* | Change Object#either? to Object#among? -- thanks to @jamesarosen for the ↵ | David Heinemeier Hansson | 2011-04-12 | 1 | -1/+1 | |
| | | | | suggestion! | |||||
* | Using Object#in? and Object#either? in various places | Prem Sichanugrist | 2011-04-11 | 1 | -1/+2 | |
| | | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?. | |||||
* | cleaning up typecast test a little | Aaron Patterson | 2011-03-26 | 1 | -6/+8 | |
| | ||||||
* | refactoring tz to a variable rather than repeating it | Aaron Patterson | 2011-03-26 | 1 | -2/+4 | |
| | ||||||
* | adding a test for attributes after type cast. thanks nragaz. :heart: | Aaron Patterson | 2011-03-26 | 1 | -0/+17 | |
| | ||||||
* | fixing whitespace errors. :bomb: | Aaron Patterson | 2011-03-25 | 1 | -2/+2 | |
| | ||||||
* | Fix before_type_cast for timezone aware attributes by caching converted ↵ | Adam Meehan | 2011-03-23 | 1 | -15/+11 | |
| | | | | value on write. Also remove read method reload arg on timezone attributes. | |||||
* | bugfix for serialized_attributes to be class specific | Raimonds Simanovskis | 2011-02-09 | 1 | -0/+5 | |
| | | | | previously serialized_attributes were kept as class attribute of ActiveRecord::Base - if some attribute was defined as serialized in one subclass then it was serialized in all other subclasses as well (if it had the same name) | |||||
* | do not to_s where you are testing that a string value is stored for the ↵ | Akira Matsuda | 2011-02-07 | 1 | -1/+1 | |
| | | | | | | before_type_cast Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | avoid nil.dup | Akira Matsuda | 2011-02-07 | 1 | -0/+8 | |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | fix for test_read_attributes_before_type_cast_on_datetime - Oracle adapter ↵ | Raimonds Simanovskis | 2011-02-02 | 1 | -2/+2 | |
| | | | | also returns Time value | |||||
* | Make before_type_cast available for datetime fields | Akira Matsuda | 2011-02-01 | 1 | -16/+15 | |
| | | | | | | [#3973 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Add test for e0e3adf | Jeremy Kemper | 2011-01-07 | 1 | -0/+6 | |
| | ||||||
* | Allow primary_key to be an attribute when the model is a new record | Santiago Pastorino | 2010-12-28 | 1 | -1/+1 | |
| | ||||||
* | fixing space errors | Aaron Patterson | 2010-12-02 | 1 | -3/+1 | |
| | ||||||
* | Work around a strange piece of Syck behaviour where it checks ↵ | Michael Koziarski | 2010-12-02 | 1 | -0/+11 | |
| | | | | | | Model#respond_to? before initializing the object. Things like YAML.load(YAML.dump(@post)) won't work without this. | |||||
* | removing many unused variables | Aaron Patterson | 2010-11-16 | 1 | -1/+1 | |
| | ||||||
* | mysql tests are mostly passing | Aaron Patterson | 2010-10-26 | 1 | -3/+3 | |
| | ||||||
* | Fewer object allocations | Jeremy Kemper | 2010-09-09 | 1 | -3/+3 | |
| | ||||||
* | Cache unserialized attributes | Jeremy Kemper | 2010-09-09 | 1 | -9/+12 | |
| | ||||||
* | Do not use time zone in test_read_attributes_before_type_cast_on_datetime ↵ | Raimonds Simanovskis | 2010-08-22 | 1 | -3/+3 | |
| | | | | | | for Oracle database As currently string_to_time method is not doing time zone conversion to database time zone | |||||
* | Both tests are using the same model, move the model to another file and add ↵ | Santiago Pastorino | 2010-08-14 | 1 | -1/+2 | |
| | | | | the missing require | |||||
* | Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵ | Santiago Pastorino | 2010-08-14 | 1 | -4/+4 | |
| | | | | 's/[ \t]*$//' -i {} \;) | |||||
* | ignore this test for mysql2 | Brian Lopez | 2010-08-09 | 1 | -13/+15 | |
| | ||||||
* | Making Active Record base_test.rb thinner by moving tests | Neeraj Singh | 2010-08-03 | 1 | -2/+276 | |
| | | | | | | | | | | | | to relevant files. Number of assertions before refactoring: 2391 tests, 7579 assertions, 0 failures, 0 errors Number of assertions after refactoring: 2391 tests, 7579 assertions, 0 failures, 0 errors Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Use better assertion methods for testing | Neeraj Singh | 2010-05-19 | 1 | -1/+1 | |
| | | | | | | [#4645 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | 1=2 is invalid expression in Oracle SELECT | Raimonds Simanovskis | 2009-08-06 | 1 | -2/+12 | |
| | ||||||
* | Extract generic attribute method generation to AMo | Joshua Peek | 2009-08-04 | 1 | -47/+0 | |
| |