| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | | | | | | |
|
| |\ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | | |
Remove a duplicate test of mysql_rake_test
|
| | |/ / / / / / / |
|
| |\ \ \ \ \ \ \ \
| | |/ / / / / / /
| |/| | | | | | | |
Add a Monitor to ModelSchema#load_schema
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
[Vikrant Chaudhary, David Abdemoulaie, Matthew Draper]
|
| |\ \ \ \ \ \ \ \
| | |_|/ / / / / /
| |/| | | | | | | |
Extract `JSONSharedTestCases`
|
| | | |/ / / / /
| | |/| | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Both `mysql2/json_test.rb` and `postgresql/json_test.rb` have same test
cases.
|
| |\ \ \ \ \ \ \
| | |_|/ / / / /
| |/| | | | | | |
Fix crashing on circular left join references with scoping
|
| | | |_|_|/ /
| | |/| | | |
| | | | | | |
| | | | | | | |
Follow up of #25702.
|
| |/ / / / / |
|
| |\ \ \ \ \
| | | | | | |
| | | | | | | |
Make helper methods in tests to private
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
`make_model` and `make_no_pk_hm_t` in `HasManyThroughAssociationsTest`
are not a test case. it should be private.
|
| |\ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Both reference id and type should be `NOT NULL` if `null: false` is specified
|
| | |/ / / / /
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This is a regression due to #28282.
Fixes #29136.
|
| |\ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Remove a duplicate test of inverse_associations_test in AR
|
| | | |/ / / /
| | |/| | | | |
|
| |\ \ \ \ \ \
| | | | | | | |
| | | | | | | | |
Remove unused `left_joins_values` generation
|
| | |/ / / / /
| | | | | | |
| | | | | | |
| | | | | | | |
This was added at #22125 but `left_joins_values` is never used.
|
| |/ / / / / |
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This retains the existing behavior of
ActiveSupport::Cache.expand_cache_key (as used by etaging) where the
cache key includes the version.
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Addresses #29021
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When using `sql` as the schema format, or even just doing `rake
db:structure:dump`, it would be good to respect the list of ignored
tables that has been configured.
|
| | | | |
| | | | |
| | | | |
| | | | | |
Fixes #29025.
|
| | |/ /
| |/| | |
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
```
% ARCONN=sqlite3 be ruby -w -Itest test/cases/quoting_test.rb
test/cases/quoting_test.rb:92: warning: ambiguous first argument; put parentheses or a space even after `/' operator
test/cases/quoting_test.rb:96: warning: ambiguous first argument; put parentheses or a space even after `/' operator
Using sqlite3
Run options: --seed 9495
.....................................
Finished in 0.046403s, 797.3622 runs/s, 1120.6172 assertions/s.
37 runs, 52 assertions, 0 failures, 0 errors, 0 skips
```
|
| |\ \
| | | |
| | | | |
Clarify deprecation message for #quoted_id
|
| | | |
| | | |
| | | |
| | | |
| | | | |
In this case, it's the method definition that's more at fault, rather
than the current caller.
|
| |/ /
| | |
| | |
| | | |
`DelegationTest#call_method` is no longer used since 9d79334a.
|
| | |
| | |
| | |
| | | |
Since legacy `Relation#bind_values` was removed in b06f64c.
|
| |\ \
| | | |
| | | | |
Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`
|
| | | |
| | | |
| | | |
| | | | |
These are used in tests from anywhere.
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
The same test exists in `test/cases/relation/merging_test.rb`.
https://github.com/rails/rails/blob/v5.1.0/activerecord/test/cases/relation/merging_test.rb#L94-L101
|
| | |
| | |
| | |
| | |
| | | |
Since we have been using this `Arel::Table` since 111ccc832bc977b15af12c14e7ca078dad2d4373,
in order to properly handle queries, it's important that we properly type cast arguments.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`supports_statement_cache?` was introduced in 3.1.0.beta1 (104d0b2) for
bind parameter substitution, but it is no longer used in 3.1.0.rc1
(73ff679). Originally it should respect `prepared_statements` rather
than `supports_statement_cache?` (fd39847).
One more thing, named `supports_statement_cache?` is pretty misreading.
We have `StatementCache` and `StatementPool`. However,
`supports_statement_cache?` doesn't mean `StatementCache`, but
`StatementPool` unlike its name.
https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/statement_cache.rb
https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/connection_adapters/statement_pool.rb
|
| | |
| | |
| | |
| | |
| | |
| | | |
Just use one `Event` class. Reduces duplication, makes the tests easier
to read. It might seem like each tests needs a different kind of Struct,
since we make a new one for each test case.
|
| | |
| | |
| | |
| | | |
Fix #28905
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
This change reverted in eac6f369 but it is needed for data integrity.
See #25328.
|
| | |
| | |
| | |
| | |
| | |
| | | |
`DefaultScopingWithThreadTest` expects that there are two or more of
`developers` data, but have not created data in the test.
Therefore, tests may fail depending on execution order.
|
| | |
| | |
| | |
| | |
| | | |
This will make sure it doesn't change the state of the current proccess
when removing the owners constant.
|
| | |
| | |
| | |
| | | |
This will avoid us to close the connection of the saved connection pool.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
- The test `PrimaryKeyAnyTypeTest#test_any_type_primary_key` was failing
if ran after running all tests from `CompositePrimaryKeyTest`.
- This was happening because `CompositePrimaryKeyTest` was changing the
primary key of the barcodes table which was cached in schema cache.
- As we were always going to drop the `barcodes` table at the end of
tests in both `PrimaryKeyTest` and `CompositePrimaryKeyTest`, solved
this issue by using different table name for tests in
`CompositePrimaryKeyTest`.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Reproduction command -
ARCONN=postgresql be ruby -w -Itest test/cases/autosave_association_test.rb --seed 34101
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
- Tried specifying `id` for the `readers` records but it is
interconnected with so many tests that many random tests started
failing.
- So switched to the approach of deleting all readers in the
`create_resets_cached_counters` test.
|
|\ \
| | |
| | | |
PostgreSQL 10 does not convert `CURRENT_DATE` into `('now'::text)::date`
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Address #28797
In the previous versions of PostgreSQL, `CURRENT_DATE` converted to `('now'::text)::date`
and `CURRENT_TIMESTAMP` converted to `now()`.
Refer these discussions and commit at PostgreSQL :
https://www.postgresql.org/message-id/flat/5878.1463098164%40sss.pgh.pa.us#5878.1463098164@sss.pgh.pa.us
https://github.com/postgres/postgres/commit/0bb51aa96783e8a6c473c2b5e3725e23e95db834
|
|\ \ \
| | | |
| | | | |
Set consistent typecast ENV["VERBOSE"]
|