search

rails-sqlserver / activerecord-sqlserver-adapter

SQL Server Adapter For Rails
MIT License
968 stars 558 forks source link

Ensure pre-7.1 migrations use legacy index names when using rename_table #1175

Closed aidanharan closed 1 month ago

aidanharan commented 1 month ago

Implemented change made in https://github.com/rails/rails/pull/50837

Fixes the ActiveRecord::Migration::CompatibilityTest#test_rename_table_errors_on_too_long_index_name_7_0 test.

https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/actions/runs/9109903293/job/25043759601?pr=1174

 24) Failure:
ActiveRecord::Migration::CompatibilityTest#test_rename_table_errors_on_too_long_index_name_7_0 [/usr/local/bundle/bundler/gems/rails-f92c8a3539ee/activerecord/test/cases/migration/compatibility_test.rb:434]:
StandardError expected but nothing was raised.