[ https://issues.apache.org/jira/browse/LUCENE-10301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17463331#comment-17463331 ]
Uwe Schindler commented on LUCENE-10301: ---------------------------------------- bq. I really do think this is sufficient in the short term. In the long term, ensure TestSecrets is loaded in module mode - if it isn't fail. This shifts the accessibility checks to the JVM again. That's a good idea. bq. I think it's impossible to call those setters more than once now and from any other class then Lucene - TestSecrets has a static initializer which initializes all classes from which secrets are delegated - these static initializers call the setters in TestSecrets in turn. Yes, correct no checks needed. > The test-framework as a module (or a separate test-framework-module) > -------------------------------------------------------------------- > > Key: LUCENE-10301 > URL: https://issues.apache.org/jira/browse/LUCENE-10301 > Project: Lucene - Core > Issue Type: Sub-task > Reporter: Dawid Weiss > Priority: Major > Time Spent: 2.5h > Remaining Estimate: 0h > > The test framework has split packages. It's a follow-up to introducing > modules but eventually the modular test subprojects will need something like > the test framework too. > I'm not sure whether we should start a new subproject for this or try to > refactor the test framework, but it's a follow-up once the modules themselves > are working and testable, I think. -- This message was sent by Atlassian Jira (v8.20.1#820001) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org