Github user StefanRRichter commented on a diff in the pull request:
    --- Diff: 
    @@ -39,13 +47,27 @@
      * Tests for the CANCEL command.
     public class CliFrontendCancelTest extends TestLogger {
    +   @Parameterized.Parameters(name = "Mode = {0}")
    --- End diff --
    Everything about introducing this parameter is duplicated in all the test 
classes. I would create a common super class that does this parameter iteration 
and implements the `getCli()` and `getConfiguration()` methods, instead of 
having them as static methods.


Reply via email to