yanne / api-testing

I am using this to test GitHub API
1 stars 0 forks source link

Test failure in libdoc test: tools.jar required #305

Open yanne opened 10 years ago

yanne commented 10 years ago

None

This issue was originally opened at Google Code on Mar 25, 2011.

yanne commented 10 years ago

Original comment by mrmall...@gmail.com on Mar 31, 2011.

this is very tricky as each jvm distro have different location of tools.jar. i'll give it a look.

can you check on this though:

http://maven.apache.org/general.html#tools-jar-dependency

instead add it as a dependency to the maven plugin

yanne commented 10 years ago

Original comment by dietrich...@googlemail.com on Apr 2, 2011.

That helped to solve the tools.jar issue, I added the following to the pom-libdoc.xml to see if it works, and the test is no longer complaining about a missing tools.jar. I am using an absolute path because ${java.home} did not work. However, there is a different issue now: libdoc.py requires a class from Java 6 and cannot find it since I am running with Java 5. Well that's just the way it is, probably we can't do anything about it. I'll try to come up with a test that does not document a java testlib.

default-tools.jar java.vendor Sun Microsystems Inc. com.googlecode robotframework-maven-plugin src/test/java/com/googlecode/ExampleLib.java com.sun tools 1.6 system C:/programme/java/jdk1.6.0_21/lib/tools.jar
yanne commented 10 years ago

Original comment by dietrich...@googlemail.com on Apr 2, 2011.

I have a test now that runs fine with an html resource file. I would propose to replace the java lib test by this one. What do you think?

yanne commented 10 years ago

Original comment by mrmall...@gmail.com on Apr 3, 2011.

just gave you commit rights. go check it in and let's review it together.

i'll try to add something more "universal" regarding the tools.jar issue. as i am on a mac, this is why i don't have that problem.

yanne commented 10 years ago

Original comment by mrmall...@gmail.com on Apr 4, 2011.

i've already checked in changes to accommodate both sun and apple jdk's. please see if this works for you

yanne commented 10 years ago

Original comment by dietrich...@googlemail.com on Apr 8, 2011.

This appears to be solved, the issue has disappeared.