Comment 1 Andrew Cattle 2008-09-22 10:45:16 EDT

Created attachment 113127 [details]
Patch containing all my tests for MirrorApplication

Comment 2 Andrew Cattle 2008-09-22 10:47:17 EDT

Created attachment 113128 [details]
test data for MirrorAppliaction test cases

insert the "mirror" folder into the testData folder in the org.eclipse.equinox.p2.tests package

Comment 3 Pascal Rapicault 2008-09-22 21:56:49 EDT

Andrew, due to recent changes in HEAD, parts of the patch are no longer applicable. Could you please create a new patch and see me for review and release. Thx.

Comment 4 Andrew Cattle 2008-09-23 09:12:52 EDT

Created attachment 113246 [details]
Updated patch containing testcases

Comment 5 Andrew Cattle 2008-09-23 16:00:54 EDT

Created attachment 113294 [details]
Updated patch (agian) containing testcases

Hopefully this one is finally able to be applied.

Comment 6 DJ Houghton 2008-09-23 16:40:10 EDT

Note in order for the tests to pass you must have the patches from bug 247889 and bug 247214 loaded.

Comment 7 DJ Houghton 2008-09-23 17:13:43 EDT

Created attachment 113303 [details]
patch

Here is an updated patch which contains the changes for the above referenced bug  reports. Andrew, please update as discussed and attach a new patch when completed. Thanks.

Comment 8 Andrew Cattle 2008-09-29 13:49:23 EDT

Created attachment 113775 [details]
Newest version of MirrorApplication tests

I refactored the code to a large extent and added several test cases for invalid inputs. Requires the extra test data I'm about to upload to be placed with the other test data.

Also requires bug 247889 to be fixed.

Comment 9 Andrew Cattle 2008-09-29 13:54:06 EDT

Created attachment 113778 [details]
Additional test data required for refactored test cases

Includes a copy of mirrorSourceRepo1 but with a different version number

Comment 10 DJ Houghton 2008-09-30 17:03:16 EDT

Released modified code and test data to HEAD.