> On Jul 5, 2016, at 1:36 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: > > >> On Jul 5, 2016, at 12:42 PM, Alexandre (Shura) Iline >> <alexandre.il...@oracle.com> wrote: >> >> This made sense, than you, Mandy. >> >> Please review new version: >> http://cr.openjdk.java.net/~shurailine/8158670/webrev.02/ > > You can use Layer::findModule instead of Configuration::findModule.
That is correct. Changed in place. > > You can also use List::equals. I am assuming you are suggesting to use List::equals in the bottom part of the test where the expected result is compared with the actual list of providers. The whole reason I redid that section to provide more information in the jtr file, both for a case of a failure and to find out what providers were actually expected for given configuration. I do not see how List:equals help me with that. Information on size mismatch is useful, and also the information on unexpected provider name. Shura > > Mandy
