diff options
| author | David Wagner <david@marvid.fr> | 2016-10-19 00:20:36 +0300 |
|---|---|---|
| committer | David Wagner <david@marvid.fr> | 2016-10-24 23:29:48 +0300 |
| commit | 03b6108f6f293830e50d113068877be155549fdd (patch) | |
| tree | 036b693ccf6e556ba3ba404cdee918af18dec0ae /spec/models/merge_request_spec.rb | |
| parent | 4658c9a234b270897b0e9042a3f51fcdc376146b (diff) | |
Remove redundant class_name and foreign_key overrides
They were Rails' default and are unnecessarily overridden.
Signed-off-by: David Wagner <david@marvid.fr>
Diffstat (limited to 'spec/models/merge_request_spec.rb')
| -rw-r--r-- | spec/models/merge_request_spec.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb index 6e5137602aa..3155eff9ee1 100644 --- a/spec/models/merge_request_spec.rb +++ b/spec/models/merge_request_spec.rb @@ -6,8 +6,8 @@ describe MergeRequest, models: true do subject { create(:merge_request) } describe 'associations' do - it { is_expected.to belong_to(:target_project).with_foreign_key(:target_project_id).class_name('Project') } - it { is_expected.to belong_to(:source_project).with_foreign_key(:source_project_id).class_name('Project') } + it { is_expected.to belong_to(:target_project).class_name('Project') } + it { is_expected.to belong_to(:source_project).class_name('Project') } it { is_expected.to belong_to(:merge_user).class_name("User") } it { is_expected.to have_many(:merge_request_diffs).dependent(:destroy) } end |