Github user tillrohrmann commented on a diff in the pull request:
    --- Diff: 
    @@ -117,6 +138,7 @@ public TestEnvironment getTestEnvironment() {
        public void before() throws Exception {
    +           miniClusterResourceConfiguration = 
    --- End diff --
    I think we are mixing concerns by letting the `MiniClusterResource` change 
its behaviour across multiple tests. I think we should not pass in a 
configuration supplier which can lazily instantiate a configuration. Instead if 
you need a different `MiniClusterResource` per test, then the test should 
instantiate the respective `MiniClusterResource` with the proper configuration.


Reply via email to