From 8849eeaf05b447b4f88b3e90b564d2f2defd8649 Mon Sep 17 00:00:00 2001
From: "Edwin L. Punzalan" <epunzalan@apache.org>
Date: Tue, 14 Mar 2006 02:43:06 +0000
Subject: [PATCH] PR: MRM-101

Removed ProxyManagerFactory

git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@385717 13f79535-47bb-0310-9956-ffa450edef68
---
 .../repository/proxy/ProxyManagerFactory.java | 66 -------------------
 .../proxy/DefaultProxyManagerTest.java        |  5 +-
 .../proxy/LegacyProxyManagerTest.java         |  5 +-
 3 files changed, 6 insertions(+), 70 deletions(-)
 delete mode 100644 maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/ProxyManagerFactory.java

diff --git a/maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/ProxyManagerFactory.java b/maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/ProxyManagerFactory.java
deleted file mode 100644
index 2a52fcda6..000000000
--- a/maven-repository-proxy/src/main/java/org/apache/maven/repository/proxy/ProxyManagerFactory.java
+++ /dev/null
@@ -1,66 +0,0 @@
-package org.apache.maven.repository.proxy;
-
-/*
- * Copyright 2005-2006 The Apache Software Foundation.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-import org.apache.maven.repository.proxy.configuration.ProxyConfiguration;
-import org.codehaus.plexus.PlexusConstants;
-import org.codehaus.plexus.PlexusContainer;
-import org.codehaus.plexus.component.repository.exception.ComponentLookupException;
-import org.codehaus.plexus.context.Context;
-import org.codehaus.plexus.context.ContextException;
-import org.codehaus.plexus.personality.plexus.lifecycle.phase.Contextualizable;
-
-/**
- * Factory class for creating ProxyManager instances.  The usage of a factory ensures that the created instance will
- * have the necessary configuration
- *
- * @author Edwin Punzalan
- * @plexus.component role="org.apache.maven.repository.proxy.ProxyManagerFactory"
- */
-public class ProxyManagerFactory
-    implements Contextualizable
-{
-    public static String ROLE = "org.apache.maven.repository.proxy.ProxyManagerFactory";
-
-    private PlexusContainer container;
-
-    /**
-     * Used to create a ProxyManager instance of a certain type with a configuration to base its behavior
-     *
-     * @param proxy_type The ProxyManager repository type
-     * @param config     The ProxyConfiguration to describe the behavior of the proxy instance
-     * @return The ProxyManager instance of type proxy_type with ProxyConfiguration config
-     * @throws ComponentLookupException when the factory fails to create the ProxyManager instance
-     */
-    public ProxyManager getProxyManager( String proxy_type, ProxyConfiguration config )
-        throws ComponentLookupException
-    {
-        ProxyManager proxy = (ProxyManager) container.lookup( ProxyManager.ROLE );
-        config.setLayout( proxy_type );
-        proxy.setConfiguration( config );
-        return proxy;
-    }
-
-    /**
-     * @see org.codehaus.plexus.personality.plexus.lifecycle.phase.Contextualizable#contextualize(org.codehaus.plexus.context.Context)
-     */
-    public void contextualize( Context context )
-        throws ContextException
-    {
-        container = (PlexusContainer) context.get( PlexusConstants.PLEXUS_KEY );
-    }
-}
diff --git a/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/DefaultProxyManagerTest.java b/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/DefaultProxyManagerTest.java
index 859580070..f3f8b27ba 100644
--- a/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/DefaultProxyManagerTest.java
+++ b/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/DefaultProxyManagerTest.java
@@ -39,8 +39,9 @@ public class DefaultProxyManagerTest
     {
         super.setUp();
 
-        ProxyManagerFactory factory = (ProxyManagerFactory) container.lookup( ProxyManagerFactory.ROLE );
-        proxy = factory.getProxyManager( "default", getTestConfiguration() );
+        proxy = (ProxyManager) container.lookup( ProxyManager.ROLE );
+
+        proxy.setConfiguration( getTestConfiguration() );
     }
 
     public void testExceptions()
diff --git a/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/LegacyProxyManagerTest.java b/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/LegacyProxyManagerTest.java
index 5e2268908..768e505b8 100644
--- a/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/LegacyProxyManagerTest.java
+++ b/maven-repository-proxy/src/test/java/org/apache/maven/repository/proxy/LegacyProxyManagerTest.java
@@ -39,8 +39,9 @@ public class LegacyProxyManagerTest
     {
         super.setUp();
 
-        ProxyManagerFactory factory = (ProxyManagerFactory) container.lookup( ProxyManagerFactory.ROLE );
-        proxy = factory.getProxyManager( "legacy", getTestConfiguration() );
+        proxy = (ProxyManager) container.lookup( ProxyManager.ROLE );
+
+        proxy.setConfiguration( getTestConfiguration() );
     }
 
     public void testExceptions()
-- 
2.39.5