In a case like this where we don't expose an endpoint to configure a Berkshelf path, I don't think this is a bad pattern. We are trying to expose this to our test suite which can easily be done with env vars. It's also made the tests quite brittle by caching these variables which is what the change was actually done for.