Bug #10248
closedPossible missing test or bug for Module#include and Module#prepend
Description
I've been implementing Module#prepend for JRuby. I ran into this test https://github.com/jruby/jruby/blob/master/test/jruby/test_method_cache.rb#L19
This was unexpected behavior for me. I've looked through both the MRI and rubyspec test cases and don't see a similar test.
If this behavior is intended, this test should be included in the suite. Currently, MRI passes this test for both include and prepend. When implementing include or prepend without referencing the MRI source code directly, the implementation won't necessarily pass this test.
Updated by headius (Charles Nutter) almost 10 years ago
A couple questions:
- Which of the two cases tested is unexpected (or is it both)? The two cases are (in pseudo-spec-ese):
- If a module A has been included into class B, and then additional modules are included into A, calls to B will not see them without a re-include of A.
- If a module A has been inlcluded into class B, and then methods are modified in A, calls to B will see them without a re-include of A.
Ignore the mentions of "flushing" in the spec...the above behavior is what it really tests.
It works this way because at include time, a reference to the module is inserted into the target hierarchy, and only its method table gets searched; other modules that the first module includes are themselves included into the hierarchy.
Included modules are only searched for methods and constants they directly define. The module's method and constant tables are virtually proxied into the target hierarchy, but the module's superclasses are only included at include time.
I don't know why this is the case.
Updated by Eregon (Benoit Daloze) almost 10 years ago
Module#ancestors is still respected for which modules to look up so it is somewhat consistent, but rather confusing indeed.
Including a module in a class computes the module ancestors when it is included, and further (module) transitive inclusions are therefore not considered.
But re-including the module works in this case as the last line shows.
I do not know the reason of this but it very likely is spec.
> class A; end
> module B; end
> class C < A; include B; end
> module X; end
> module B; include X; end
> B.ancestors
=> [B, X]
> C.ancestors
=> [C, B, A, Object, Kernel, BasicObject]
> class D < A; include B; end
> D.ancestors
=> [D, B, X, A, Object, Kernel, BasicObject]
> class C < A; include B; end
> C.ancestors
=> [C, B, X, A, Object, Kernel, BasicObject]
Updated by nobu (Nobuyoshi Nakada) almost 10 years ago
It's a TODO, which should be fixed in the future, and there are some related tickets.
Updated by tduehr (Timur Duehr) almost 10 years ago
Nobuyoshi Nakada wrote:
It's a TODO, which should be fixed in the future, and there are some related tickets.
Is that to change the behavior or add the tests?
Updated by tduehr (Timur Duehr) almost 10 years ago
Ahh... just found it. https://bugs.ruby-lang.org/issues/9112
Thanks
Updated by tduehr (Timur Duehr) almost 10 years ago
I've created a patch to the MRI test suite used in JRuby to check for this behavior.
https://github.com/tduehr/jruby/commit/5bf0da4e4fff1a9e122ff427ff34586025db9955
I'm now using #9112 for discussion on new behavior with a patch to hopefully come before too long.
Updated by headius (Charles Nutter) almost 10 years ago
Perhaps we can incorporate the tests now, if it's considered spec behavior? If it is not considered spec behavior, then this is a little fuzzy to me. I'd like to have clarification one way or the other.
I can commit the tests if approved.
Updated by jeremyevans0 (Jeremy Evans) about 4 years ago
- Status changed from Open to Closed