diff options
author | Ian Bull | 2012-09-20 19:33:51 +0000 |
---|---|---|
committer | Ian Bull | 2012-10-01 21:44:26 +0000 |
commit | ec7d2f597acf5f0048fb6d6b204a951786e42630 (patch) | |
tree | 42c496e7fe7052262dec0caf6c70480e5e5bbbd9 /bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror | |
parent | 2f4b8b75353f177c3857fe09b9be6546f78ff3bb (diff) | |
download | rt.equinox.p2-ec7d2f597acf5f0048fb6d6b204a951786e42630.tar.gz rt.equinox.p2-ec7d2f597acf5f0048fb6d6b204a951786e42630.tar.xz rt.equinox.p2-ec7d2f597acf5f0048fb6d6b204a951786e42630.zip |
Revert "Revert "Bug 387557 - stable tests""v20121001-214426
This reverts commit 0f85cc10f6086cde746b969917a23ad5cd24d167.
Diffstat (limited to 'bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror')
2 files changed, 5 insertions, 71 deletions
diff --git a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/ArtifactMirrorApplicationTest.java b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/ArtifactMirrorApplicationTest.java index 0cc2ef618..23806ad3a 100644 --- a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/ArtifactMirrorApplicationTest.java +++ b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/ArtifactMirrorApplicationTest.java @@ -295,39 +295,6 @@ public class ArtifactMirrorApplicationTest extends AbstractProvisioningTest { } /** - * ensures that all files with entries in the repo have corresponding files on disk. - * Not Biconditional. - */ - private void assertFileSizes(String message, SimpleArtifactRepository expected, SimpleArtifactRepository actual) { - IQueryResult expectedKeys = expected.query(ArtifactKeyQuery.ALL_KEYS, null); - for (Iterator iterator = expectedKeys.iterator(); iterator.hasNext();) { - IArtifactKey key = (IArtifactKey) iterator.next(); - IArtifactDescriptor[] expectedDescriptors = expected.getArtifactDescriptors(key); - IArtifactDescriptor[] actualDescriptors = actual.getArtifactDescriptors(key); - - if (expectedDescriptors == null || actualDescriptors == null) - if (!(expectedDescriptors == null && actualDescriptors == null)) - fail(message + " missing key " + key); - - top: for (int j = 0; j < expectedDescriptors.length; j++) { - for (int k = 0; k < actualDescriptors.length; k++) { - if (Arrays.equals(expectedDescriptors[j].getProcessingSteps(), actualDescriptors[k].getProcessingSteps())) { - File expectedFile = expected.getArtifactFile(expectedDescriptors[j]); - File actualFile = actual.getArtifactFile(actualDescriptors[k]); - if (expectedFile == null || actualFile == null) - fail(message + " descriptor mismatch"); - if (!(expectedFile.exists() && actualFile.exists())) - fail(message + " file does not exist"); - assertTrue(expectedFile.length() == actualFile.length()); - continue top; - } - } - fail(message + "Missing expected descriptor" + expectedDescriptors[j]); - } - } - } - - /** * Tests mirroring all artifacts in a repository to an empty repository * Source contains A, B * Target contains @@ -884,7 +851,7 @@ public class ArtifactMirrorApplicationTest extends AbstractProvisioningTest { //Verify Contents assertContentEquals("26.4", getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); //Verify files on disk - assertFileSizes("26.5", (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); + assertEqualArtifacts("26.5", (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); } catch (ProvisionException e) { fail("26.6", e); } @@ -951,7 +918,7 @@ public class ArtifactMirrorApplicationTest extends AbstractProvisioningTest { fail("27.3 no canonical found for " + key.toString()); //ensure the canonical matches that in the expected - assertFileSizes("27.3", (SimpleArtifactRepository) destinationRepo, (SimpleArtifactRepository) packedRepo); + assertEqualArtifacts("27.3", (SimpleArtifactRepository) destinationRepo, (SimpleArtifactRepository) packedRepo); } } } diff --git a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/NewMirrorApplicationArtifactTest.java b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/NewMirrorApplicationArtifactTest.java index 7b240a5ce..624373c05 100644 --- a/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/NewMirrorApplicationArtifactTest.java +++ b/bundles/org.eclipse.equinox.p2.tests/src/org/eclipse/equinox/p2/tests/mirror/NewMirrorApplicationArtifactTest.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2008, 2010 IBM Corporation and others. + * Copyright (c) 2008, 2012 IBM Corporation and others. * All rights reserved. This program and the accompanying materials * are made available under the terms of the Eclipse Public License v1.0 * which accompanies this distribution, and is available at @@ -321,39 +321,6 @@ public class NewMirrorApplicationArtifactTest extends AbstractProvisioningTest { } /** - * ensures that all files with entries in the repo have corresponding files on disk. - * Not Biconditional. - */ - private void assertFileSizes(String message, SimpleArtifactRepository expected, SimpleArtifactRepository actual) { - IQueryResult expectedKeys = expected.query(ArtifactKeyQuery.ALL_KEYS, null); - for (Iterator iterator = expectedKeys.iterator(); iterator.hasNext();) { - IArtifactKey key = (IArtifactKey) iterator.next(); - IArtifactDescriptor[] expectedDescriptors = expected.getArtifactDescriptors(key); - IArtifactDescriptor[] actualDescriptors = actual.getArtifactDescriptors(key); - - if (expectedDescriptors == null || actualDescriptors == null) - if (!(expectedDescriptors == null && actualDescriptors == null)) - fail(message + " missing key " + key); - - top: for (int j = 0; j < expectedDescriptors.length; j++) { - for (int k = 0; k < actualDescriptors.length; k++) { - if (Arrays.equals(expectedDescriptors[j].getProcessingSteps(), actualDescriptors[k].getProcessingSteps())) { - File expectedFile = expected.getArtifactFile(expectedDescriptors[j]); - File actualFile = actual.getArtifactFile(actualDescriptors[k]); - if (expectedFile == null || actualFile == null) - fail(message + " descriptor mismatch"); - if (!(expectedFile.exists() && actualFile.exists())) - fail(message + " file does not exist"); - assertEquals("Unexpected difference in " + actualFile.getName(), expectedFile.length(), actualFile.length()); - continue top; - } - } - fail(message + "Missing expected descriptor" + expectedDescriptors[j]); - } - } - } - - /** * Tests mirroring all artifacts in a repository to an empty repository * Source contains A, B * Target contains @@ -904,7 +871,7 @@ public class NewMirrorApplicationArtifactTest extends AbstractProvisioningTest { //Verify Contents assertContentEquals("26.4", getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); //Verify files on disk - assertFileSizes("26.5", (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); + assertEqualArtifacts("26.5", (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(packedRepoLocation.toURI(), null), (SimpleArtifactRepository) getArtifactRepositoryManager().loadRepository(destRepoLocation.toURI(), null)); } catch (ProvisionException e) { fail("26.6", e); } @@ -971,7 +938,7 @@ public class NewMirrorApplicationArtifactTest extends AbstractProvisioningTest { fail("27.3 no canonical found for " + key.toString()); //ensure the canonical matches that in the expected - assertFileSizes("27.3", (SimpleArtifactRepository) destinationRepo, (SimpleArtifactRepository) packedRepo); + assertEqualArtifacts("27.3", (SimpleArtifactRepository) destinationRepo, (SimpleArtifactRepository) packedRepo); } } } |