Tests for AjcTaskCompileCommmand adapter

ICommand adapter for Ant taskdef is simple for the non-incremental case and ugly otherwise.

supporting -ajctaskCompiler option with AjcTaskCompileCommand

awful downcast for AjcTaskCompileCommand - should be remedied after 1.1 with ICommand interface extension

updated implementation for -ajdeCompiler.

replaced reference to null ide properties adapter (defined only in testsrc)
and created logging proxy to verify API's used.
Unfortunately also formatted code, so it looks like there are more changes than there are.

updates mainly to support maven  but also to support new ICommand wrapper for AjcTask:
- findToolsJar now accepts versioned jars
- supporting a forkclasspath to permit user  to explicitly set
- upgrades for testing
  - so multiple iterations work,
    - not altering cmd in execute()
    - only fixing up outjar once
    - reset() complete
  - so incremental compile tests work,
    - passing quit through to main

- when forking on windows, use javaw rather than java
  if possible to avoid popping up a console window

more/better junit tests for AjcTask.java fork when possible

setting classpath for out-of-eclipse testing

fix for Bugzilla Bug 37758
   Weaving rt.jar results in stack overflow

test for Bugzilla Bug 37758
   Weaving rt.jar results in stack overflow

forcing target to be 1.1 in case someone builds using 1.4.

flagging with comments that 1.1 options are unimplemented

forcing target to be 1.1 in case someone builds using 1.4.

@testcase PR#37898 advice on handler join points should not throw unpermitted checked exceptions

added paragraph for call site implementation

changed jars produced by BcWeaverJarMaker to increase coverage of tests

better error message for trying declare parents on Object

expanded test coverage

fix for NPE revealed by BcWeaverJarMaker test code

added test for Bugzilla Bug 37304
   public static fields being ignored
this test is passing in 1.1rc2

fix and test Bugzilla Bug 37739
   Unexpected Xlint:unresolvableMember warning with withincode

- typo in top-level task
- incorrect pattern to copy all junit reports

- tense, xref usage corrections
- structure model caveat

More message-visiting methods, used only by harness now.

defensively supplying ISourceLocation.NO_FILE when there is no file in the result.
This is needed for file-based message matching to not fail when no file is specified,
and might provide a more sensible error message if the file is used.
This only replaces a NPE or invalid (empty) filename strings.

updating for installer temp dir fix when File.getTempFile() fails.

absolutizing temp dir path.
When File.getTempFile() fails, workaround temp dir
is relative, which messes up the zip resource copying.

build-testing-jars target for release testing setup

clean-jars target

Testing now seems to be working (tested incrementally)
CVS and entire-run still not verified - takes long on my machine.

variable renaming, verbosity - not much significant

- now supports both eclipse bin dirs (default)
  and ant-built test jars
- added incomplete test-all-at-once
  (currently runs each from its module directory)

raising issue of reporting source location for constructor execution as starting with the first initializer.  Possibly a known limitation.

- correcting message specification of existing tests
  using file and text comparison
- adding new command-line error tests (and keyword)

test data upgrades for adding new command-line error tests

two bad-input tests for the harness

- adding badInput attribute to compiler
- adding message kinds (abort, fail, info)
- message line no longer required

stricter test run comparisons

- new data (IMessageHolder rather than lists), same algorithms
- using medium-strenght message comparison that includes soft
  comparison of file and text
- unexpected info messages not marked as error
- expected FAIL+ messages suppressed to avoid false negative

- supporting badInput for all IAjcRun
- using IMessageHolder rather than lists

- enforcing -incremental => staging
- permitting bad input using badInput="true"
  (though still checking during staging)
- permitting -XserializableAspects
- supporting new AjcMessageHandler interfaces
- supporting parent badInput in XML

- supporting parent badInput in XML
- supporting new AjcMessageHandler interfaces

- supporting parent badInput in XML

- absolutizing tempdir path if necessary
  (bug in path calculations otherwise when
   using relative path to work around failure
   of File.getTempFile())

- revamped message comparators,
  mostly adding medium-level and consolidating
  common code

- new filter to remove arbitrary elements from diffs
  (used to not report unexpected info messages in diffs)

- using easier list view of messages

patch mentioned on the list for detecting errs in incremental command line

misspelled local invalid option in negative test

Fix for taskdef bug 37576
http://bugs.eclipse.org/bugs/show_bug.cgi?id=37576

No functionality change.
This just makes explicit that the browser reads user preferences
rather than relying on the default value supplied by the default constructor.

Disabling test reads from user preferences store.
This was discussed on the list before rc2.

Fixed relative links to examples. This time did it properly.

Fixed broken links to bug db and IBM sites.

Fixed links to Eclipse bugs database and to mailing lists.

Fixed link to Eclipse bugs database (was dev.eclipse.org/bugs now bugs.eclipse.org/bugs)

Fixed relative links to docs (was docs/index.html now doc/index.html)
and examples (was eaxamples now doc/examples).

Fixed link to Eclipse bugs database (was dev.eclipse.org/bugs now bugs.eclipse.org/bugs)

old dominates form produces an error, not a warning

link updates for progguide id filenames

fix for Bugzilla Bug 37325
   Weaver fails with NPE for very large source files

test for Bugzilla Bug 37325
   Weaver fails with NPE for very large source files

includeClassesDir

includeClassesDir

- platform-independent relative path
- compile-time checks for new attributes read

Deleting temporary dir and config file, like harness -killTemp option (ignores -keepTemp)

including new attributes for round-trip testing

- test compile includeClassesDir attribute
- add pass|fail title prefix to run in one go

test source upgrades for new and old tests

Another strange incremental test, taken from ajcHarnessTests
and comparable with a known limitation case for injars,
this sometimes doesn't fail for me.  hmm.
(requires pending changes to sources)

new attribute for compile, includeClassesDir

moved incremental tests from failing (since they now pass)
cleaned out contents of failing that are now in ajcTests

changed suite spec from ajc-test+ to ajc-test* to allow empty suites
so that ajcTestsFailing can be empty without looking scary when run

handling Shadow.PreInitialization as a INITIALIZER_RELATION
rather than printing a warning message to System.err

this fixes the dirty output that was very visible when running the
harness with -emacssym enabled

fix to use cache for isPackage method (as indicated by previous !!! comment)
this fixes the two failing incremental test cases just added by Wes and now
moved to ajcTests as passing

Added persist member.

Backing out the change I made to add the output directory to the classpath.
This is wrong because we should resolve types wrt the cached classes in the weaver.
But if this change were left in, the test cases just added to ajcTestsFailing.xml pass.

two new incremental tests, each of which fails if run first but not otherwise (assuming the output directory is not added to the classpath).

comment documenting keyword "incremental-test"

Fixing Harness bug of adding the output directory to the classpath, which conceals a bug in the incremental compilation.
This change should have no effect on other tests, but I noticed two other failures during normal compiles caused by inability to resolve type names.

fixed testFileNodeFind that would always pass to be more strict
  and then fixed the file it was operating on to be correct

adding output dir/jar by default to classpath to permit resolving types in incremental compile
Might need to be more discriminating and/or to document.

- removed invalid test requiring -sourceroots in the argument list.  They might be in a config file.
- handling request  for fresh rebuild during incremental

updated with tests, etc.  Not working yet:
- junitModules.xml needs better setup with testable jars
- cvs unvalidated
- ajcTests unvalidated - see also ../build.xml script

incomplete update to support module jars rather than eclipse bin directories when running junit tests
the old way with bin directories still works fine.

permitting module builds without trimming, to have the option of using those jars rather than eclipse bin directories when running junit tests

changing after to after returning, with Jim's blessing.

fix for reported bug with invalid classpath entries (from the environment)
being reported as errors.  These are now just info messages.

initial draft of release build/test script.
CVS portion is untested.

fixed bug for inferring the kind of messages from the eclipse batch
compiler where == -1 was used instead of != -1

wrong handler reference

Minor updates for 1.1 rc2

moving to all-inclusive testing client for forking
(This might be wrong.. still verifying)

envp and snooping error stream in process controller

supporting a forked java run provisionally with system properties specifying fork, java home, java, and bootclasspath.
This should enable running the harness when targeting a VM that the harness or compiler can't run on,
but this does not detect failures correctly in some cases,
nor does it validate any of the system properties.

-keepTemp implementation: Suite run iterator calls clearCommand, which also calls validator.deleteTempFiles

-killTemp to delete temp directories when each test case completes

updated test-writing docs for AJDE fresh incremental builds.

new harness tests
- inc-compile "fresh" attribute and "same" tag
- compile "reuseCompiler" attribute

new attributes:
- compile reuseCompiler
- inc-compile fresh