Skip to content

Commit 22d796c

Browse files
Steve Klebanoffctran
Steve Klebanoff
authored andcommitted
Fix show_complete_foreign_keys setting (#475)
* add show_complete_foreign_keys as setting * test for show_complete_foreign_keys setting
1 parent c24f64c commit 22d796c

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

lib/annotate.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,8 @@ module Annotate
3030
:show_indexes, :simple_indexes, :include_version, :exclude_tests,
3131
:exclude_fixtures, :exclude_factories, :ignore_model_sub_dir,
3232
:format_bare, :format_rdoc, :format_markdown, :sort, :force, :trace,
33-
:timestamp, :exclude_serializers, :classified_sort, :show_foreign_keys,
33+
:timestamp, :exclude_serializers, :classified_sort,
34+
:show_foreign_keys, :show_complete_foreign_keys,
3435
:exclude_scaffolds, :exclude_controllers, :exclude_helpers,
3536
:exclude_sti_subclasses, :ignore_unknown_models
3637
].freeze

spec/annotate/annotate_models_spec.rb

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -484,6 +484,21 @@ def mock_column(name, type, options = {})
484484
EOS
485485
end
486486

487+
describe '#set_defaults' do
488+
it 'should default show_complete_foreign_keys to false' do
489+
expect(Annotate.true?(ENV['show_complete_foreign_keys'])).to be(false)
490+
end
491+
492+
it 'should be able to set show_complete_foreign_keys to true' do
493+
Annotate.set_defaults('show_complete_foreign_keys' => 'true')
494+
expect(Annotate.true?(ENV['show_complete_foreign_keys'])).to be(true)
495+
end
496+
497+
after :each do
498+
ENV.delete('show_complete_foreign_keys')
499+
end
500+
end
501+
487502
describe '#get_schema_info with custom options' do
488503
def self.when_called_with(options = {})
489504
expected = options.delete(:returns)

0 commit comments

Comments
 (0)