Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

remove duplicated require statements in AR test cases #7887

Merged
merged 1 commit into from

2 participants

@senny
Owner

While scanning a few AR test cases I noticed a duplicated require statement for a model. I then did a search to reveal the duplicated requires (I found a couple other ones).

This PR removes the unused (duplicated) require statements.

@senny
Owner

@rafaelfranca can you check this?

@vijaydev vijaydev merged commit ecf460e into rails:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Oct 9, 2012
  1. @senny
This page is out of date. Refresh to see the latest.
View
1  activerecord/test/cases/associations/has_many_through_associations_test.rb
@@ -19,7 +19,6 @@
require 'models/subscription'
require 'models/essay'
require 'models/category'
-require 'models/owner'
require 'models/categorization'
require 'models/member'
require 'models/membership'
View
1  activerecord/test/cases/autosave_association_test.rb
@@ -16,7 +16,6 @@
require 'models/tag'
require 'models/tagging'
require 'models/treasure'
-require 'models/company'
require 'models/eye'
class TestAutosaveAssociationsInGeneral < ActiveRecord::TestCase
View
1  activerecord/test/cases/reflection_test.rb
@@ -3,7 +3,6 @@
require 'models/customer'
require 'models/company'
require 'models/company_in_module'
-require 'models/subscriber'
require 'models/ship'
require 'models/pirate'
require 'models/price_estimate'
View
1  activerecord/test/cases/relations_test.rb
@@ -5,7 +5,6 @@
require 'models/topic'
require 'models/comment'
require 'models/author'
-require 'models/comment'
require 'models/entrant'
require 'models/developer'
require 'models/reply'
Something went wrong with that request. Please try again.