Add some test cases for nested target discovery, which also implicitly tests nested module discovery

1 files changed, 3 insertions, 3 deletions

diff --git a/core/src/main/scala/mill/define/Module.scala b/core/src/main/scala/mill/define/Module.scala
index 33d0e267..3d3bb87c 100644
--- a/core/src/main/scala/mill/define/Module.scala
+++ b/core/src/main/scala/mill/define/Module.scala
@@ -43,12 +43,12 @@ object Module{
     lazy val segmentsToModules = traverse{m => Seq(m.millModuleSegments -> m)}
       .toMap
 
-    lazy val segmentsToTargets = traverse{_.millInternal.reflect[Target[_]]}
+    lazy val targets = traverse{_.millInternal.reflect[Target[_]]}.toSet
+
+    lazy val segmentsToTargets = targets
       .map(t => (t.ctx.segments, t))
       .toMap
 
-    lazy val targets = segmentsToTargets.valuesIterator.toSet
-
     // Ensure we do not propagate the implicit parameters as implicits within
     // the body of any inheriting class/trait/objects, as it would screw up any
     // one else trying to use sourcecode.{Enclosing,Line} to capture debug info