activerecord-cockroachdb-adapter
activerecord-cockroachdb-adapter copied to clipboard
[ActiveRecord 5.2 Test Failure] test/cases/adapters/postgresql/rename_table_test.rb
Failure:
PostgresqlRenameTableTest#test_renaming_a_table_also_renames_the_primary_key_index [/Users/alimi/repos/rails/activerecord/test/cases/adapters/postgresql/rename_table_test.rb:18]:
Expected: 1
Actual: 0
bin/rails test Users/alimi/repos/rails/activerecord/test/cases/adapters/postgresql/rename_table_test.rb:16
See the Contributing Guide for instructions on running tests.
If the test isn't valid against CockroachDB
- Add a ruby file to
test/excludes
that matches the name of the test class if one doesn't already exist. For example to exclude a test fromActiveRecord::AdapterTest
, createtest/excludes/ActiveRecord/AdapterTest.rb
. - Add an exclude statement to the file with the name of the test to exclude and a description. For example to exclude
test_indexes
fromActiveRecord::AdapterTest
: https://github.com/cockroachdb/activerecord-cockroachdb-adapter/blob/c3cb637bd855b36bbf9e76b7947162054b95022e/test/excludes/ActiveRecord/AdapterTest.rb#L1 - Finally, if the test can run against CockroachDB with a few changes, add it to
test/cases
. Everything from the ActiveRecord test suite will be available, so a lot of the excluded test can be copied over. Namespace the test under theCockroachDB
module to avoid name collisions. Seetest/cases/adapter_test.rb
for an example.
See https://github.com/cockroachdb/activerecord-cockroachdb-adapter/issues/48.
Any update on this failing test?
For me to convince our organisation to switch over to cockroachdb, it would be very helpful if most ActiveRecord tests where passing.
Renaming tables is a common operation, and if this test doesn't pass that's somewhat worrying.