Integration with HtmlUnit

• Key: CACTUS-220
• Type: New Feature
• Status: Closed
• Resolution: Fixed
• Priority: Minor
• Assignee: Petar Tahchiev
• Reporter: Natalia Kowalczyk
• Affects Version/s: 1.7
• Fix Version/s: 1.8

Description

Cactus could integrate with HtmlUnit in the same fashion it does with HttpUnit. HtmlUnit recognizes more HTML object as scriptable and thus offers better JavaScript processing.
Attaching a patch for the WebResponseObjectFactory class.

Comments

[Natalia Kowalczyk]

CVS diff of WebResponseObjectFactory with a patch

[Vincent Massol]

Thanks Natakia, that's a great patch. Do you think you could add some junit tests in samples/servlet that proves that it works? Also, it would be great if you could submit some doco patch (they're in documentation/docs/xdocs). Having all this will make is much faster to apply your patch as otherwise we'll need to write all those. Thanks again for this very useful patch!

[Nicolas Chalumeau]

Here is a test case for this issue. I also modify in differant location classpath/dependancies to make the build work and the test pass (it work on my system) but i can't attach the patch ATM as I am offline and can't make any diff so the patch arrive this weekend.

[Nicolas Chalumeau]

Here is the patch to add htmlunitjar in the differants classpath/dependancies

[Nicolas Chalumeau]

I will commit the patch this week end

[Felipe Leme]

While I was testing Cactus's samples, I got this error testing on Tomcat 5:

[cactustests] Null Test: Caused an ERROR
[cactustests] com/gargoylesoftware/htmlunit/WebResponse
[cactustests] java.lang.NoClassDefFoundError: com/gargoylesoftware/htmlunit/WebResponse
[cactustests] at java.lang.Class.getDeclaredMethods0(Native Method)
[cactustests] at java.lang.Class.privateGetDeclaredMethods(Class.java:2365)
[cactustests] at java.lang.Class.getMethod0(Class.java:2611)
[cactustests] at java.lang.Class.getMethod(Class.java:1579)

It probably means we didn't bundle a jar needed by HTML Unit...

[Felipe Leme]

Nope, that class has no external dependency; looks like a classloader issue then...

[Petar Tahchiev]

Already integrated in the SVN HEAD. We will have to think of a better integration for the future. I am closing this issue now and will open one after we release Cactus 1.8.