You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2018/09/18 16:47:48 UTC

[GitHub] [geode] kirklund commented on pull request #2488: GEODE-5741: Apply fixes from StartLocator test to StartServer test

For these permissions tests, you could potentially abstract all file manipulation away from the class(es) under test, add a constructor that let's you pass in YourFileManipulationClass and then have the test create a YourFileManipulationClass that throws permissions exceptions.

We don't really need to test with the file system throwing exceptions at our code. We just want to verify the behavior of "start server" when it's unable to create a directory. Maybe a mocking based test would be better.

[ Full content available at: https://github.com/apache/geode/pull/2488 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org