From 9336f7f941c6f345aa3a835a84413c606f23246d Mon Sep 17 00:00:00 2001 From: Patrick Ziegler Date: Fri, 1 Dec 2023 23:32:08 +0100 Subject: [PATCH] Add non-IU artifact repositories to the planner resolution #874 Given that units from non-IU locations are considered when resolving IUs, it may happen that units are contained in the resolution path, which are not contained by any repository. This may break previously functional target definitions. To resolve this problem, an in-memory artifact and metadata repository is created, based on the metadata of the non-IU locations. Those repositories are then added to the provisioning context for further processing. Amends dd321b56a92b8e86f98de6f0dae25fbc43e4f825 --- .../pde/internal/core/target/Messages.java | 4 +- .../internal/core/target/Messages.properties | 6 +- .../internal/core/target/P2TargetUtils.java | 35 ++- .../target/VirtualArtifactRepository.java | 202 ++++++++++++++++++ .../target/VirtualMetadataRepository.java | 56 +++++ .../META-INF/MANIFEST.MF | 1 + .../tests/target/IUBundleContainerTests.java | 93 +++++++- 7 files changed, 382 insertions(+), 15 deletions(-) create mode 100644 ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualArtifactRepository.java create mode 100644 ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualMetadataRepository.java diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.java index a9ffafcfcb4..61227d89d7c 100755 --- a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.java +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2009, 2021 IBM Corporation and others. + * Copyright (c) 2009, 2023 IBM Corporation and others. * * This program and the accompanying materials * are made available under the terms of the Eclipse Public License 2.0 @@ -80,6 +80,8 @@ public class Messages extends NLS { public static String RemoteTargetHandle_invalid_URI; public static String RemoteTargetHandle_malformed_URL; public static String RemoteTargetHandle_ioproblem; + public static String VirtualArtifactRepository_0; + public static String VirtualArtifactRepository_1; static { diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.properties b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.properties index 2465fc78c5b..0ca65c26be8 100755 --- a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.properties +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/Messages.properties @@ -1,5 +1,5 @@ ############################################################################### -# Copyright (c) 2009, 2021 IBM Corporation and others. +# Copyright (c) 2009, 2023 IBM Corporation and others. # # This program and the accompanying materials # are made available under the terms of the Eclipse Public License 2.0 @@ -73,4 +73,6 @@ TargetRefrenceLocationFactory_Unsupported_Type=Type {0} is not supported by this TargetRefrenceLocationFactory_Parsing_Failed=Parsing location content failed: {0} RemoteTargetHandle_invalid_URI=Invalid URI: {0} RemoteTargetHandle_malformed_URL=URI {0} can not be converted to an URL: {1} -RemoteTargetHandle_ioproblem=Reading URI {0} failed: {1} \ No newline at end of file +RemoteTargetHandle_ioproblem=Reading URI {0} failed: {1} +VirtualArtifactRepository_0=Target bundle is not resolved: {0} +VirtualArtifactRepository_1=Artifact location not found for descriptor: {0} \ No newline at end of file diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/P2TargetUtils.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/P2TargetUtils.java index 15175d3d8ef..593447e3a99 100644 --- a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/P2TargetUtils.java +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/P2TargetUtils.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2010, 2018 EclipseSource Inc. and others. + * Copyright (c) 2010, 2023 EclipseSource Inc. and others. * * This program and the accompanying materials * are made available under the terms of the Eclipse Public License 2.0 @@ -1078,11 +1078,24 @@ private void resolveWithPlanner(ITargetDefinition target, IProgressMonitor monit request.setInstallableUnitProfileProperty(unit, PROP_INSTALLED_IU, Boolean.toString(true)); } - ProvisioningContext context = new ProvisioningContext(getAgent()); + List extraArtifactRepositories = new ArrayList<>(); + List extraMetadataRepositories = new ArrayList<>(); + addAdditionalProvisionIUs(target, extraArtifactRepositories, extraMetadataRepositories); + ProvisioningContext context = new ProvisioningContext(getAgent()) { + @Override + public IQueryable getArtifactRepositories(IProgressMonitor monitor) { + return QueryUtil.compoundQueryable(super.getArtifactRepositories(monitor), + (query, ignore) -> query.perform(extraArtifactRepositories.iterator())); + } + @Override + public IQueryable getMetadata(IProgressMonitor monitor) { + return QueryUtil.compoundQueryable(super.getMetadata(monitor), + QueryUtil.compoundQueryable(extraMetadataRepositories)); + } + }; context.setProperty(ProvisioningContext.FOLLOW_REPOSITORY_REFERENCES, Boolean.toString(true)); context.setMetadataRepositories(getMetadataRepositories(target).toArray(URI[]::new)); context.setArtifactRepositories(getArtifactRepositories(target).toArray(URI[]::new)); - context.setExtraInstallableUnits(getAdditionalProvisionIUs(target)); IProvisioningPlan plan = planner.getProvisioningPlan(request, context, subMonitor.split(20)); IStatus status = plan.getStatus(); @@ -1553,8 +1566,9 @@ private Collection getMetadataRepositories(ITargetDefinition target) throws return result; } - private List getAdditionalProvisionIUs(ITargetDefinition target) throws CoreException { - List result = new ArrayList<>(); + private void addAdditionalProvisionIUs(ITargetDefinition target, + Collection extraArtifactRepositories, + Collection extraMetadataRepositories) throws CoreException { ITargetLocation[] containers = target.getTargetLocations(); if (containers != null) { for (ITargetLocation container : containers) { @@ -1564,17 +1578,20 @@ private List getAdditionalProvisionIUs(ITargetDefinition targe } if (container instanceof TargetReferenceBundleContainer targetRefContainer) { ITargetDefinition referencedTargetDefinition = targetRefContainer.getTargetDefinition(); - result.addAll(getAdditionalProvisionIUs(referencedTargetDefinition)); + addAdditionalProvisionIUs(referencedTargetDefinition, extraArtifactRepositories, + extraMetadataRepositories); continue; } if (!container.isResolved()) { container.resolve(target, new NullProgressMonitor()); } - InstallableUnitGenerator.generateInstallableUnits(container.getBundles(), container.getFeatures()) - .forEach(result::add); + extraArtifactRepositories.add(new VirtualArtifactRepository(getAgent(), container)); + List installableUnits = InstallableUnitGenerator // + .generateInstallableUnits(container.getBundles(), container.getFeatures()) // + .toList(); + extraMetadataRepositories.add(new VirtualMetadataRepository(getAgent(), installableUnits)); } } - return result; } private static final String NATIVE_ARTIFACTS = "nativeArtifacts"; //$NON-NLS-1$ diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualArtifactRepository.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualArtifactRepository.java new file mode 100644 index 00000000000..e9496ce94d0 --- /dev/null +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualArtifactRepository.java @@ -0,0 +1,202 @@ +/******************************************************************************* + * Copyright (c) 2023 Patrick Ziegler and others. + * + * This program and the accompanying materials + * are made available under the terms of the Eclipse Public License 2.0 + * which accompanies this distribution, and is available at + * https://www.eclipse.org/legal/epl-2.0/ + * + * SPDX-License-Identifier: EPL-2.0 + * + * Contributors: + * Patrick Ziegler - initial API and implementation + *******************************************************************************/ +package org.eclipse.pde.internal.core.target; + +import java.io.File; +import java.io.FileNotFoundException; +import java.io.InputStream; +import java.io.OutputStream; +import java.io.PrintWriter; +import java.net.URI; +import java.nio.file.Files; +import java.util.HashMap; +import java.util.Map; +import java.util.UUID; +import java.util.jar.Attributes; +import java.util.jar.JarEntry; +import java.util.jar.JarOutputStream; +import java.util.jar.Manifest; + +import org.eclipse.core.runtime.Assert; +import org.eclipse.core.runtime.CoreException; +import org.eclipse.core.runtime.IProgressMonitor; +import org.eclipse.core.runtime.IStatus; +import org.eclipse.core.runtime.MultiStatus; +import org.eclipse.core.runtime.Status; +import org.eclipse.core.runtime.SubMonitor; +import org.eclipse.equinox.frameworkadmin.BundleInfo; +import org.eclipse.equinox.internal.p2.metadata.expression.QueryResult; +import org.eclipse.equinox.p2.core.IProvisioningAgent; +import org.eclipse.equinox.p2.core.ProvisionException; +import org.eclipse.equinox.p2.metadata.IArtifactKey; +import org.eclipse.equinox.p2.publisher.eclipse.BundlesAction; +import org.eclipse.equinox.p2.publisher.eclipse.FeaturesAction; +import org.eclipse.equinox.p2.query.IQuery; +import org.eclipse.equinox.p2.query.IQueryResult; +import org.eclipse.equinox.p2.query.IQueryable; +import org.eclipse.equinox.p2.repository.artifact.IArtifactDescriptor; +import org.eclipse.equinox.p2.repository.artifact.IArtifactRequest; +import org.eclipse.equinox.p2.repository.artifact.spi.AbstractArtifactRepository; +import org.eclipse.equinox.p2.repository.artifact.spi.ArtifactDescriptor; +import org.eclipse.osgi.util.NLS; +import org.eclipse.pde.core.target.ITargetLocation; +import org.eclipse.pde.core.target.TargetBundle; +import org.eclipse.pde.core.target.TargetFeature; +import org.eclipse.pde.internal.core.PDECore; +import org.eclipse.pde.internal.core.ifeature.IFeature; +import org.eclipse.pde.internal.core.ifeature.IFeatureModel; + +/** + * In-Memory representation of a artifact repository based on a non-IU target + * location. This repository is used during the planner resolution of an IU + * target location to supply artifacts from other (non-IU) locations. + */ +@SuppressWarnings("restriction") +public class VirtualArtifactRepository extends AbstractArtifactRepository { + private static final String NAME = "Non-IU Artifact Repository @ "; //$NON-NLS-1$ + private static final String DESCRIPTION = """ + In-Memory repository created for a single Non-IU repository, used + during the planner resolution of a real IU repository. + """; //$NON-NLS-1$ + private static final String MEMORY_PREFIX = "memory://"; //$NON-NLS-1$ + // BundleInfo or IFeatureModel + private final Map artifacts = new HashMap<>(); + + public VirtualArtifactRepository(IProvisioningAgent agent, ITargetLocation targetLocation) { + super(agent, NAME + getLocationSafe(targetLocation), targetLocation.getType(), null, + URI.create(MEMORY_PREFIX + UUID.randomUUID()), DESCRIPTION + '\n' + targetLocation.serialize(), null, + null); + Assert.isTrue(targetLocation.isResolved()); + for (TargetBundle targetBundle : targetLocation.getBundles()) { + if (!targetBundle.getStatus().isOK()) { + PDECore.log(Status.warning(NLS.bind(Messages.VirtualArtifactRepository_0, targetBundle))); + continue; + } + BundleInfo bundleInfo = targetBundle.getBundleInfo(); + IArtifactKey artifactKey = BundlesAction.createBundleArtifactKey(bundleInfo.getSymbolicName(), + bundleInfo.getVersion()); + IArtifactDescriptor artifactDesriptor = new ArtifactDescriptor(artifactKey); + artifacts.put(artifactDesriptor, bundleInfo); + } + for (TargetFeature targetFeature : targetLocation.getFeatures()) { + IArtifactKey artifactKey = FeaturesAction.createFeatureArtifactKey(targetFeature.getId(), + targetFeature.getVersion()); + IArtifactDescriptor artifactDesriptor = new ArtifactDescriptor(artifactKey); + artifacts.put(artifactDesriptor, targetFeature.getFeatureModel()); + } + } + + private static String getLocationSafe(ITargetLocation targetLocation) { + try { + return targetLocation.getLocation(false); + } catch (CoreException e) { + return ""; //$NON-NLS-1$ + } + } + + @Override + public IStatus getRawArtifact(IArtifactDescriptor descriptor, OutputStream destination, IProgressMonitor monitor) { + Object artifactModel = artifacts.get(descriptor); + if (artifactModel == null) { + return Status.error(NLS.bind(Messages.VirtualArtifactRepository_1, descriptor)); + } + try { + transferArtifact(artifactModel, destination); + return Status.OK_STATUS; + } catch (Exception e) { + return Status.error(e.getLocalizedMessage(), e); + } + } + + private void transferArtifact(Object artifactModel, OutputStream destination) throws Exception { + if (artifactModel instanceof BundleInfo bundleInfo) { + URI location = bundleInfo.getLocation(); + if (location == null) { + throw new FileNotFoundException(bundleInfo.getSymbolicName()); + } + try (InputStream is = location.toURL().openStream()) { + is.transferTo(destination); + } + } else if (artifactModel instanceof IFeatureModel featureModel) { + String installLocation = featureModel.getInstallLocation(); + if (installLocation != null) { + File featureJar = new File(installLocation); + if (featureJar.isFile()) { + Files.copy(featureJar.toPath(), destination); + return; + } + } + IFeature feature = featureModel.getFeature(); + // Generate in-memory feature jar (with only the feature.xml) + Manifest manifest = new Manifest(); + manifest.getMainAttributes().put(Attributes.Name.MANIFEST_VERSION, "1.0"); //$NON-NLS-1$ + try (JarOutputStream jos = new JarOutputStream(destination, manifest)) { + jos.putNextEntry(new JarEntry("feature.xml")); //$NON-NLS-1$ + feature.write("", new PrintWriter(jos, true)); //$NON-NLS-1$ + jos.closeEntry(); + } + } else { + throw new IllegalArgumentException(artifactModel.toString()); + } + } + + @Override + public IQueryable descriptorQueryable() { + return (query, monitor) -> query.perform(artifacts.keySet().iterator()); + } + + @Override + public IQueryResult query(IQuery query, IProgressMonitor monitor) { + return new QueryResult<>(artifacts.keySet().stream().map(IArtifactDescriptor::getArtifactKey).iterator()); + } + + @Override + public boolean contains(IArtifactDescriptor descriptor) { + return artifacts.containsKey(descriptor); + } + + @Override + public boolean contains(IArtifactKey key) { + return artifacts.keySet().stream().anyMatch(descriptor -> key.equals(descriptor.getArtifactKey())); + } + + @Override + public IStatus getArtifact(IArtifactDescriptor descriptor, OutputStream destination, IProgressMonitor monitor) { + return getRawArtifact(descriptor, destination, monitor); + } + + @Override + public IArtifactDescriptor[] getArtifactDescriptors(IArtifactKey key) { + return artifacts.keySet().stream() // + .filter(descriptor -> key.equals(descriptor.getArtifactKey())) // + .toArray(IArtifactDescriptor[]::new); + } + + @Override + public IStatus getArtifacts(IArtifactRequest[] requests, IProgressMonitor monitor) { + MultiStatus multiStatus = new MultiStatus(getClass(), IStatus.INFO, "Perform Artifact Requests"); //$NON-NLS-1$ + SubMonitor subMonitor = SubMonitor.convert(monitor, requests.length); + for (IArtifactRequest request : requests) { + request.perform(this, subMonitor.split(1)); + multiStatus.add(request.getResult()); + } + return multiStatus.isOK() ? Status.OK_STATUS : multiStatus; + } + + @Override + public OutputStream getOutputStream(IArtifactDescriptor descriptor) throws ProvisionException { + throw new ProvisionException("Artifact repository must not be modified!"); //$NON-NLS-1$ + } + +} diff --git a/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualMetadataRepository.java b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualMetadataRepository.java new file mode 100644 index 00000000000..b5dcc0e2218 --- /dev/null +++ b/ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/VirtualMetadataRepository.java @@ -0,0 +1,56 @@ +/******************************************************************************* + * Copyright (c) 2023 Patrick Ziegler and others. + * + * This program and the accompanying materials + * are made available under the terms of the Eclipse Public License 2.0 + * which accompanies this distribution, and is available at + * https://www.eclipse.org/legal/epl-2.0/ + * + * SPDX-License-Identifier: EPL-2.0 + * + * Contributors: + * Patrick Ziegler - initial API and implementation + *******************************************************************************/ +package org.eclipse.pde.internal.core.target; + +import java.util.Collection; +import java.util.Collections; +import java.util.List; + +import org.eclipse.core.runtime.IProgressMonitor; +import org.eclipse.equinox.p2.core.IProvisioningAgent; +import org.eclipse.equinox.p2.metadata.IInstallableUnit; +import org.eclipse.equinox.p2.query.IQuery; +import org.eclipse.equinox.p2.query.IQueryResult; +import org.eclipse.equinox.p2.repository.IRepositoryReference; +import org.eclipse.equinox.p2.repository.metadata.spi.AbstractMetadataRepository; + +/** + * In-Memory representation of a metadata repository based on a non-IU target + * location. This repository is used during the planner resolution of an IU + * target location to supply the metadata from other (non-IU) locations. + */ +public class VirtualMetadataRepository extends AbstractMetadataRepository { + private final List installableUnits; + + public VirtualMetadataRepository(IProvisioningAgent agent, List installableUnits) { + super(agent); + this.installableUnits = List.copyOf(installableUnits); + } + + + @Override + public Collection getReferences() { + return Collections.emptySet(); + } + + @Override + public IQueryResult query(IQuery query, IProgressMonitor monitor) { + return query.perform(installableUnits.iterator()); + } + + @Override + public void initialize(RepositoryState state) { + // nothing to do + } +} diff --git a/ui/org.eclipse.pde.ui.tests/META-INF/MANIFEST.MF b/ui/org.eclipse.pde.ui.tests/META-INF/MANIFEST.MF index cdb72b61dfc..9955a310935 100644 --- a/ui/org.eclipse.pde.ui.tests/META-INF/MANIFEST.MF +++ b/ui/org.eclipse.pde.ui.tests/META-INF/MANIFEST.MF @@ -30,6 +30,7 @@ Require-Bundle: org.eclipse.pde.ui, org.eclipse.equinox.p2.repository, org.eclipse.equinox.p2.metadata, org.eclipse.equinox.p2.engine, + org.eclipse.equinox.p2.publisher.eclipse, org.eclipse.ui.forms, org.eclipse.ui.workbench.texteditor, org.eclipse.jface.text, diff --git a/ui/org.eclipse.pde.ui.tests/src/org/eclipse/pde/ui/tests/target/IUBundleContainerTests.java b/ui/org.eclipse.pde.ui.tests/src/org/eclipse/pde/ui/tests/target/IUBundleContainerTests.java index c97b2e16727..f23a2e374e1 100644 --- a/ui/org.eclipse.pde.ui.tests/src/org/eclipse/pde/ui/tests/target/IUBundleContainerTests.java +++ b/ui/org.eclipse.pde.ui.tests/src/org/eclipse/pde/ui/tests/target/IUBundleContainerTests.java @@ -1,5 +1,5 @@ /******************************************************************************* - * Copyright (c) 2009, 2018 IBM Corporation and others. + * Copyright (c) 2009, 2023 IBM Corporation and others. * * This program and the accompanying materials * are made available under the terms of the Eclipse Public License 2.0 @@ -36,9 +36,15 @@ import org.eclipse.core.runtime.IPath; import org.eclipse.core.runtime.IStatus; import org.eclipse.equinox.frameworkadmin.BundleInfo; +import org.eclipse.equinox.p2.metadata.IArtifactKey; import org.eclipse.equinox.p2.metadata.IInstallableUnit; +import org.eclipse.equinox.p2.publisher.eclipse.BundlesAction; +import org.eclipse.equinox.p2.publisher.eclipse.FeaturesAction; import org.eclipse.equinox.p2.query.IQueryResult; import org.eclipse.equinox.p2.query.QueryUtil; +import org.eclipse.equinox.p2.repository.artifact.IArtifactDescriptor; +import org.eclipse.equinox.p2.repository.artifact.IArtifactRepository; +import org.eclipse.equinox.p2.repository.artifact.spi.ArtifactDescriptor; import org.eclipse.equinox.p2.repository.metadata.IMetadataRepository; import org.eclipse.equinox.p2.repository.metadata.IMetadataRepositoryManager; import org.eclipse.pde.core.plugin.IPluginModelBase; @@ -47,11 +53,13 @@ import org.eclipse.pde.core.target.ITargetLocation; import org.eclipse.pde.core.target.ITargetPlatformService; import org.eclipse.pde.internal.core.PDECore; +import org.eclipse.pde.internal.core.target.DirectoryBundleContainer; import org.eclipse.pde.internal.core.target.IUBundleContainer; import org.eclipse.pde.internal.core.target.P2TargetUtils; import org.eclipse.pde.internal.core.target.TargetDefinition; import org.eclipse.pde.internal.core.target.TargetDefinitionPersistenceHelper; import org.eclipse.pde.internal.core.target.TargetPersistence38Helper; +import org.eclipse.pde.internal.core.target.VirtualArtifactRepository; import org.eclipse.pde.ui.tests.PDETestsPlugin; import org.junit.Test; import org.w3c.dom.Document; @@ -182,6 +190,68 @@ public void testResolveSingleBundle() throws Exception { doResolutionTest(new String[]{"bundle.a1"}, bundles); } + /** + * Tests whether the in-memory artifact repository is correctly created from + * a non-IU target location. + */ + @Test + public void testResolveVirtualArtifactRepository() throws Exception { + IArtifactRepository repo = createVirtualArtifactRepository(); + IArtifactKey artifactKey; + // Bundles + artifactKey = BundlesAction.createBundleArtifactKey("org.junit4", "4.7.0.v20100104"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.junit", "3.8.2.v20090203-1005"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.junit", "4.7.0.v20091118-1515"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.eclipse.jdt", "3.6.0.v200912170819"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.eclipse.jdt.source", "3.6.0.v201001051537"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.eclipse.jdt.launching", "3.5.100.v20091203"); + assertTrue(repo.contains(artifactKey)); + artifactKey = BundlesAction.createBundleArtifactKey("org.eclipse.jdt.launching.source", "3.5.100.v20100104"); + assertTrue(repo.contains(artifactKey)); + // Features + artifactKey = FeaturesAction.createFeatureArtifactKey("org.eclipse.jdt", + "3.6.0.v20100105-0800-7z8VFR9FMTb52_pOyKHhoek1"); + assertTrue(repo.contains(artifactKey)); + artifactKey = FeaturesAction.createFeatureArtifactKey("org.eclipse.jdt.source", + "3.6.0.v20100105-0800-7z8VFR9FMTb52_pOyKHhoek1"); + assertTrue(repo.contains(artifactKey)); + } + + /** + * Tests whether the artifacts can be downloaded from the in-memory + * repository. + */ + @Test + public void testGetVirtualRepositoryArtifact() throws Exception { + IArtifactRepository repo = createVirtualArtifactRepository(); + // Bundles + IArtifactKey artifactKey = BundlesAction.createBundleArtifactKey("bundle", "1.0.0"); + IArtifactDescriptor artifactDesriptor = new ArtifactDescriptor(artifactKey); + assertEquals(IStatus.ERROR, + repo.getArtifact(artifactDesriptor, new ByteArrayOutputStream(), null).getSeverity()); + + artifactKey = BundlesAction.createBundleArtifactKey("org.junit", "3.8.2.v20090203-1005"); + artifactDesriptor = new ArtifactDescriptor(artifactKey); + assertEquals(IStatus.OK, + repo.getArtifact(artifactDesriptor, new ByteArrayOutputStream(), null).getSeverity()); + // Features + artifactKey = FeaturesAction.createFeatureArtifactKey("feature", "1.0.0"); + artifactDesriptor = new ArtifactDescriptor(artifactKey); + assertEquals(IStatus.ERROR, + repo.getArtifact(artifactDesriptor, new ByteArrayOutputStream(), null).getSeverity()); + + artifactKey = FeaturesAction.createFeatureArtifactKey("org.eclipse.jdt", + "3.6.0.v20100105-0800-7z8VFR9FMTb52_pOyKHhoek1"); + artifactDesriptor = new ArtifactDescriptor(artifactKey); + assertEquals(IStatus.OK, + repo.getArtifact(artifactDesriptor, new ByteArrayOutputStream(), null).getSeverity()); + } + /** * Tests that contents should be equal. */ @@ -327,9 +397,26 @@ public void testExternalModelManagerPreferences() throws Exception { } /** - * Creates an IU bundle container with the specified IUs from the test repository. + * Creates an in-memory artifact repository over + * {@code modified-jdt-features.zip} * - * @param unitIds identifiers of IU's to add to the container + * @return in-memory artifact repository + */ + protected VirtualArtifactRepository createVirtualArtifactRepository() throws Exception { + File repoFolder = extractModifiedFeatures().toFile(); + ITargetLocation container = new DirectoryBundleContainer(repoFolder.getAbsolutePath()); + ITargetDefinition definition = getNewTarget(); + definition.setTargetLocations(new ITargetLocation[] { container }); + container.resolve(definition, null); + return new VirtualArtifactRepository(null, container); + } + + /** + * Creates an IU bundle container with the specified IUs from the test + * repository. + * + * @param unitIds + * identifiers of IU's to add to the container * @return bundle container */ protected IUBundleContainer createContainer(String[] unitIds) throws Exception {