qunit makes yuicompressor choke (yui treats 'throws' as a reserved keyword)

[cberzan]

The qunit API defines a function throws (http://api.qunitjs.com/throws/), but throws is a reserved keyword in JavaScript (http://bclary.com/2004/11/07/#a-7.5.3). One side effect is that qunit cannot be passed through yuicompressor:

> java -jar .../yuicompressor-2.4.6.jar .../qunit/qunit-1.9.0.js 

[ERROR] 490:8:invalid property id

[ERROR] 490:9:syntax error

[ERROR] 528:1:missing ) in parenthetical

[ERROR] 540:35:missing name after . operator

[ERROR] 1932:1:syntax error

[ERROR] 1932:29:syntax error

[ERROR] 1:0:Compilation produced 6 syntax errors.
org.mozilla.javascript.EvaluatorException: Compilation produced 6 syntax errors.
    at com.yahoo.platform.yui.compressor.YUICompressor$1.runtimeError(YUICompressor.java:154)
    at org.mozilla.javascript.Parser.parse(Parser.java:392)
    at org.mozilla.javascript.Parser.parse(Parser.java:337)
    at com.yahoo.platform.yui.compressor.JavaScriptCompressor.parse(JavaScriptCompressor.java:312)
    at com.yahoo.platform.yui.compressor.JavaScriptCompressor.<init>(JavaScriptCompressor.java:533)
    at com.yahoo.platform.yui.compressor.YUICompressor.main(YUICompressor.java:131)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
    at java.lang.reflect.Method.invoke(Method.java:597)
    at com.yahoo.platform.yui.compressor.Bootstrap.main(Bootstrap.java:21)

[cberzan]

Just found this #267. I guess you made the decision to use throws because it is no longer a reserved keyword in the latest ECMAScript. I updated the relevant bug report in YUICompressor (http://yuilibrary.com/projects/yuicompressor/ticket/2528028) instead.

[jzaefferer]

Okay, here's hoping yuicompressor will fix this upstream.