Hi Phil, Prasanta, I have updated the fix as you suggested, (i.e. added the regression test). The new webrev is located at http://cr.openjdk.java.net/~dmarkov/8163889/webrev.01/ Could you review the new version, please?
Thanks, Dmitry > On 20 Jan 2017, at 19:50, Phil Race <[email protected]> wrote: > > I haven't looked at the fix (yet) but I definitely agree that a manual > regression test > for this is better than none. What else should we do ? Just not test > printing ? > > In my view which I've expressed to SQE for a really long time, if you aren't > testing with > printers installed you aren't testing the whole platform. Whilst it may be > convenient > that tests (silently) don't complain when there are no printers, it is a > slippery slope .. > > -phil. > > On 01/20/2017 04:04 AM, Prasanta Sadhukhan wrote: >> >> It is possible to create manual regression test for this problem. Also the >> test will require some additional set up steps such as printer installation >> and so on. It seems to me that is overhead for person who runs it. However >> if you insist on test creation, I will add it. >
