[ https://issues.apache.org/jira/browse/SOLR-4195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13533011#comment-13533011 ]
Dawid Weiss commented on SOLR-4195: ----------------------------------- While I was initially hesitant about security manager based solution (in favor of applying certain assertions via bytecode injection :), I agree with Uwe that it gives more control and is platform-independent. +1 from me. > Further restrict security policy of tests to disallow writing to files > outside the test's work dir (e.g. disallow writing to build/test-files) > ---------------------------------------------------------------------------------------------------------------------------------------------- > > Key: SOLR-4195 > URL: https://issues.apache.org/jira/browse/SOLR-4195 > Project: Solr > Issue Type: Improvement > Reporter: Hoss Man > Assignee: Uwe Schindler > Attachments: SOLR-4195.patch, SOLR-4195.patch > > > Until recently, I thought the solr test framework was setup such that every > test got it's own copy of the "test-files/solr" directory to use as it's Solr > Home Dir -- then mark committed r1421543, to fix a problem where that test > was writing a file (that would later be removed) to the solr conf dir, which > would confuse another currently running test and cause it to fail. > This made me realize that what i was remembering is that the ant build files > copy the src/test-files directories into build/ prior to running the tests -- > but all tests (in that module) still share the same copy. > Subsequent discussions with folks on IRC lead me to the following > realizations.. > * making a copy of the test-files dir for each test would help eliminate > confusing by reducing non-reproducible failures if tests collide -- but might > be slow > * making a copy of the test-files dir for each test would not help identify > situations were code was mistakenly/unexpectedly writing to the solr home dir > * what would probably make the most sense, would be to make the > build/test-files directory "read only". that way by default tests would get > a read only solr home dir -- triggering failures if the code is broken and > tries to write to that dir. tests that want/need to write to the solr home > dir would have to go out of their way to clone the read only test-files/solr > directory and use it as their solr home. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org