werpu commented on PR #356:
URL: https://github.com/apache/myfaces/pull/356#issuecomment-1310011363

   Found the issue, apparently html unit bundles Mozilla Rhino and the bundled 
version or any version (still not fully done with my research on that yet) does 
not support Ecmascript 2015 syntax. After compiling the files down to es5 I get 
a more meaningful error in my internal testing framework that the keyword class 
(explicit classes were introduced in es2015) is not allowed.
   I checked on Mozillas side and found this:
   https://mozilla.github.io/rhino/compat/engines.html
   
   There are two possible fixes to move forward
   a) We can return to an ES5 compile target, which is rather pointless given 
the apis we use, and might run into compatibility issues with html unit anyway 
(we use FormData elements XhrObject, querySelectorAll etc... which are APIs 
which might not exist on ES5 level browser implementations)
   But nevertheless it is worth a shot, but my hopes are somewhat limited.
   
   b) You guys can have a look to get the unit tests out of this ES5 level 
maybe move the tests to selenium where you can use different drivers or move to 
a client side testing framework. I know this is a little but much to ask. But 
as it seems HTMLUnit for now as testrunner seems to be a dead end as long as it 
uses Rhino and/or Rhino does not move forward.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to