Remove duplication in include test #11549

merged 1 commit into from Jul 22, 2013


None yet

2 participants


This is following #11492

This code duplication goes back to 4 years ago, in 783deae.
Back then, ActiveSupport::Concern had a #use method, which was removed later on in the profit of the currently used native #include.

This use method was a bit intrusive.
Therefore, it quite made sense to test that reincluding a module when it has already been didn't just remove it.

Since we now rely on the native include method now, though, there is no use for this anymore.

cc @senny

@spastorino spastorino merged commit 891a573 into rails:master Jul 22, 2013

1 check passed

default The Travis CI build passed
@dmathieu dmathieu deleted the unknown repository branch Jul 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment