Hello Michael,
this is great.
Since you started your fork, there have been changes done by Jan Matèrne to
detect Java 9 and by myself to use separate folders for each test - all under
subfolders of ${java.io.tmpdir}
Is it possible to rebase your fork first from https://github.com/apache/ant ?
Or what is the best way to reconcile the Svn head and your work ?
If you do not have the time to rebase yourself I can help you do it, but I am
not sure what would be the best way.
Also I wonder whether Ant and Ivy should not migrate to GIT.
Antoine
On Apr 12, 2014, at 4:35 PM, Matt Sicker <boa...@gmail.com> wrote:
> This is pretty awesome!
>
>
> On 12 April 2014 07:37, Michael Clarke <michael.m.cla...@gmail.com> wrote:
>
>> Hi,
>>
>> I've now completed the migration of Ant's test cases to JUnit4 and would
>> like to give other developers a chance to review/comment on these changes
>> before I merge them back into SVN. My changes can be found on Github at
>> https://github.com/mc1arke/ant/tree/JUnit4Conversion (
>> https://github.com/mc1arke/ant/compare/JUnit4Conversion.patch for anyone
>> who wants to apply a patch against their local workspace).
>>
>> Notable changes:
>> 1. Deprecation of org.apache.tools.ant.BuildFileTest
>> and org.apache.tools.ant.types.selectors.BaseSelectorTest
>> 2. Introduction of org.apache.tools.ant.BuildFileRule and
>> org.apache.tools.ant.types.selectors.BaseSelectorRule to replace deprecated
>> classes, but with the removal of methods directly relating to asserting
>> values
>> 3. Introduction of org.apache.tools.ant.AntAssert to provide additional
>> Asserts beyond the default JUnit ones, and thecreation of
>> org.apache.tools.ant.FileUtilities to provide common file utilities used in
>> many tests
>> 4. Addition of @Test annotation to all Test methods, and addition or
>> @Ignore annotations to methods that had previously been commented out or
>> named in a way that prevented JUnit 3 seeing/running them.
>> 5. Use of JUnit's Assume to dynamically skip tests that previously silently
>> returned if certain conditions weren't met (e.g. the Symlink tests not
>> running on Windows)
>> 6. TODO markers added to tests that previously used exception handling
>> checks but didn't check the value of the exception being returned. I'll
>> look at coming back to these in the future to add proper asserts and remove
>> the TODOs.
>> 7. Removal of Thread.sleep in tests, and the sleep command in associated
>> XML build files where this has been used to force a difference in file
>> creation timestamps, and the use of File.setLastModified() instead. This
>> has knocked 1 minute 30 seconds off the JUnit execution time using
>> Cloudbees Buildhive Jenkins instance, although I've not looked at whether
>> similar can be done for the AntUnit tests.
>> 8. Updated the documentation for writing tests to refer to the new 'Rule'
>> classes rather than the previous Test classes
>>
>> I'll hold of committing these changes to SVN for a couple of days to give
>> people a chance to comment. Unless I hear any significant objections to
>> these changes by the middle of Wednesday, I'll look commit to SVN on
>> Wednesday night (UK time).
>>
>> Thanks,
>> Michael
>>
>
>
>
> --
> Matt Sicker <boa...@gmail.com>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@ant.apache.org
For additional commands, e-mail: dev-h...@ant.apache.org
