mocobeta commented on pull request #572: URL: https://github.com/apache/lucene/pull/572#issuecomment-1002086718
> They don't have to be - you can "overlay" test classes over main or the other way around. What you can't do is "merge" two module descriptors - and you need additional opens/ module dependencies for running tests If my understanding is correct I was thinking similar things (opening packages in a module to its test module to allow reflective accesses for tests), but I'm a bit afraid that it could make the purpose of the modular tests unclear - it should be kept as black-box tests for APIs to outside modules to me. But this is a different concern here and should be discussed in another issue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
