kocolosk commented on a change in pull request #26:
URL: https://github.com/apache/couchdb-erlfdb/pull/26#discussion_r650200723
##########
File path: src/erlfdb_util.erl
##########
@@ -48,6 +48,8 @@ get_test_db(Options) ->
init_test_cluster(Options) ->
+ % Hack to ensure erlfdb app environment is loaded during unit tests
+ ok = application:ensure_started(erlfdb),
Review comment:
That's a good point. I think if we did add a non-trivial dependency to
erlfdb in the future I'd like to see this function fail so we can take the
opportunity to explicitly consider whether that dependency should be started in
the unit tests as well.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
