| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
assert_raise + assert_equal on the returned exception message work
just fine, there is no need for extra work, specially like this overly
complicated helper.
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
ActiveRecord#touch should accept multiple attributes
Conflicts:
activerecord/CHANGELOG.md
|
| |/ / / / |
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | | |
Backport test from #14410
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Previously, any of the connection checkouts could have failed, and this
test would pass.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
.. not a general timeout.
Now, if a thread checks out a connection then dies, we can immediately
recover that connection and re-use it.
This should alleviate the pool exhaustion discussed in #12867. More
importantly, it entirely avoids the potential issues of the reaper
attempting to check whether connections are still active: as long as the
owning thread is alive, the connection is its business alone.
As a no-op reap is now trivial (only entails checking a thread status
per connection), we can also perform one in-line any time we decide to
sleep for a connection.
|
| | | |
| | | |
| | | |
| | | | |
It wasn't doing anything beyond clearing the statement cache.
|
| | | |
| | | |
| | | |
| | | | |
Closes #14406.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Allows fixtures to use their $LABEL as part of a string instead
of limiting use to the entire value.
mark:
first_name: $LABEL
username: $LABEL1973
email: $LABEL@$LABELmail.com
users(:mark).first_name # => mark
users(:mark).username # => mark1973
users(:mark).email # => mark@markmail.com
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
tgxworld/use_teardown_helper_method_in_activerecord
Use teardown helper method.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Follow-Up to https://github.com/rails/rails/pull/14348
Ensure that SQLCounter.clear_log is called after each test.
This is a step to prevent side effects when running tests. This will allow us to run them in random order.
|
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes #14383.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Update callbacks executed on AR::Base#touch [skip ci]
|
| | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | | |
As of https://github.com/rails/rails/pull/12031 after_commit and
after_rollback are also executed
|
|/ / / /
| | | |
| | | |
| | | | |
mitigates #14323
|
|/ / / |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Make select_all on query cache accept a Relation without binds.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
[fixes #14361]
[related #13886]
|
| | | | |
|
|/ / /
| | |
| | |
| | | |
`exists?`
|
| | |
| | |
| | |
| | | |
please pass the id of the AR object by calling `.id` on the model first.
|
| | |
| | |
| | |
| | |
| | | |
Pass the id of the object to the method by calling `.id` on the AR
object.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Apparently we've been using a buggy feature for the past 6 years:
https://bugs.ruby-lang.org/issues/9593
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Two bits of example code use sqlite as an adapter, which doesn't exist.
Using the code verbatim will raise a LoadError exception:
ActiveRecord::Base.establish_connection(
"adapter" => "sqlite",
"database" => "db.sqlite"
)
# => LoadError: Could not load 'active_record/connection_adapters/sqlite_adapter'...
Considering this is code a lot of people new to Rails might be running,
it's especially confusing.
Closes #14367 [ci skip]
|
|\ \ \
| | | |
| | | | |
Enhance docs for update_attribute [skip ci]
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Avoid rounding problems with `.usec` method rounding the seconds when the
field doesn't persist the `.usec` piece.
|
|/ / / |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Only use BINARY for mysql case sensitive uniqueness check when column has a case insensitive collation.
Conflicts:
activerecord/CHANGELOG.md
|
| | | |
| | | |
| | | |
| | | | |
case insensitive collation.
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | | |
You might want to branch it to include this only for 5.6, but
passing these values to < 5.6 doesn't cause issues either.
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Fixes STI when 2+ levels deep.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
PR #14052 Added a regression where it was only looking for methods in one
level up, So when the method was defined in a 2+ levels up the
inheritance chain, the method was not found as defined.
|
| | | | | |
|
|/ / / /
| | | |
| | | |
| | | | |
citext makes it possible to use AR Hash finders for case-insensitive matching as sql UPPER/LOWER functions are not needed.
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Save has_one associations only if record has changes
Conflicts:
activerecord/CHANGELOG.md
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Prevents save related callbacks such as `after_commit` being
triggered when `has_one` objects are already persisted and have no
changes.
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
This will avoid the confusing flunk logic
|
| | | | | |
|
| | | | | |
|