[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
RE: [aspectj-dev] Fw: Ajdoc tests

Hi George,


This is resolved by my latest fixes, in which ajdoc created a directory when you specify it as output but it doesn’t exist.  I also cleaned up the test case’s modification of resources so that there are no commits needed after a test run (i.e. no generated HTML should be visible to CVS, although its there in the doc dirs).  Let me know if that’s not the case for you.


Note that the PatternsTestCase takes a quite a long time to compile.


Regarding your patch below, it looks like its suggesting to have the options be “-sourcepath 1.4”, which is not right.  The “-source 1.4” needs to be there because there is an “assert” statement in the coverage test case.







From: aspectj-dev-admin@xxxxxxxxxxx [mailto:aspectj-dev-admin@xxxxxxxxxxx] On Behalf Of George Harley1
Sent: Thursday, April 08, 2004 11:58 AM
To: aspectj-dev@xxxxxxxxxxx
Subject: [aspectj-dev] Fw: Ajdoc tests


Sorry. Should have sent this to the list ...

George C. Harley

----- Forwarded by George Harley1/UK/IBM on 08/04/2004 19:55 -----

George Harley1/UK/IBM

08/04/2004 19:55






Ajdoc tests




Hi Mik,

After a clean extract I found that the JUnit tests in AjdocModuleTests.java would not complete for me. In order to get the green bar to go all the way to the end I had to do the following ...

1) Create empty directory "doc" under testdata/simple

2) Create empty directory "doc" under testdata/spacewar

3) Make a tiny modification to one of the test cases (see patch below)...

Index: org/aspectj/tools/ajdoc/CoverageTestCase.java
RCS file: /home/technology/org.aspectj/modules/ajdoc/testsrc/org/aspectj/tools/ajdoc/CoverageTestCase.java,v
retrieving revision 1.3
diff -u -r1.3 CoverageTestCase.java
--- org/aspectj/tools/ajdoc/CoverageTestCase.java        8 Apr 2004 16:51:09 -0000        1.3
+++ org/aspectj/tools/ajdoc/CoverageTestCase.java        8 Apr 2004 18:48:50 -0000
@@ -38,7 +38,7 @@
                 String[] args = {
 //                        "-XajdocDebug",
-                        "-source",
+                        "-sourcepath",

Best regards,
George C. Harley