lucene-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Erick Erickson (Jira)" <>
Subject [jira] [Commented] (SOLR-13988) Harden CreateCollectionCleanupTest
Date Sun, 01 Dec 2019 19:47:00 GMT


Erick Erickson commented on SOLR-13988:

bq. An alternative take would be to create a valid temp location...

That's the first thing I tried. The intermediate sub-folders apparently get created (at least
on my mac) and no exception gets thrown so the test fails as written. It's an open question
whether this change is true to the spirit of the test though.

Do you see a problem with changing permissions?

> Harden CreateCollectionCleanupTest
> ----------------------------------
>                 Key: SOLR-13988
>                 URL:
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Minor
>         Attachments: SOLR-13988.patch
> This test counts on being unable to create
> {code}
> /some_invalid_dir/foo
> {code}
> And if it does somehow succeed, it'll affect the filesystem (which it apparently can
at least on Windows under some circumstances and maybe Unix if you're super-user).
> It seems safer to create a temp dir and explicitly disable write permissions on it and
use that instead. See attached patch. While I don't have a Windows machine to test on, the
javadocs assure me that I can disable write perms this way on Windows...
> [~anshum][~tflobbe] Does this seem OK to you?

This message was sent by Atlassian Jira

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message