Skip to content

Adding test to PostgreSQL database to mimic case_insensitive indexes that breaks RescueFromDuplicate.missing_unique_indexes#17

Open
danielnc wants to merge 1 commit intoShopify:mainfrom
CareMessagePlatform:master
Open

Adding test to PostgreSQL database to mimic case_insensitive indexes that breaks RescueFromDuplicate.missing_unique_indexes#17
danielnc wants to merge 1 commit intoShopify:mainfrom
CareMessagePlatform:master

Conversation

@danielnc
Copy link
Copy Markdown

@danielnc danielnc commented Feb 8, 2019

I am not 100% sure how to properly fix this test, but I opened this PR to start the discussion, happy to work on it with some guidance

…that breaks RescueFromDuplicate.missing_unique_indexes
@gmalette
Copy link
Copy Markdown
Contributor

gmalette commented Mar 1, 2019

Sorry for the late reply, I just found time to have a look at this.
It appears to me that this is actually an issue in Active Record. The ActiveRecord::ConnectionAdapters::IndexDefinition#columns doesn't return an array for the columns in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants