https://codereview.appspot.com/202360043/diff/2/tests/com/google/caja/ancillary/opt/JsOptimizerTest.java
File tests/com/google/caja/ancillary/opt/JsOptimizerTest.java (right):

https://codereview.appspot.com/202360043/diff/2/tests/com/google/caja/ancillary/opt/JsOptimizerTest.java#newcode87
tests/com/google/caja/ancillary/opt/JsOptimizerTest.java:87: public
final void testThisNotUnintentionallyPassed1() throws Exception {
On 2015/02/26 18:32:38, kpreid_google wrote:
This test isn't quite testing what its name suggests, but I agree the
name
parallelism is relevant. How about a comment like "/** Test that
(0,...) is not
inserted when it is not necessary */"?

Done.

https://codereview.appspot.com/202360043/

--

--- You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to