have_db_column refactor from #1358 broke tests

[lifeiscontent]

I have a test:

it { is_expected.to have_db_column(:author_id).with_options(null: false, foreign_key: { to_table: :users }) }

which is now not possible due to the refactor of #1358 because it doesn't have foreign_key in its list of options.

[lifeiscontent]

CC: @rodriggochaves @mcmire

[lifeiscontent]

example usage here: lifeiscontent/realworld#662

[mcmire]

@lifeiscontent Sorry that this broke your test. That said, we have never officially supported checking for foreign_key so far, so prior to 4.5.0 your test was equivalent to:

it { is_expected.to have_db_column(:author_id).with_options(null: false) }

We will happily take a PR to add this as a feature though.

[mcmire]

Closing this since #1421 was closed.