Removed DocketFacet.setDockerFile(File).Fixes #15.

forge · Jul 7, 2015 · 3b4d6d0 · 3b4d6d0
1 parent 4717c4c
commit 3b4d6d0
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 53 deletions.
diff --git a/src/main/java/org/jboss/forge/addon/docker/facets/DockerFacet.java b/src/main/java/org/jboss/forge/addon/docker/facets/DockerFacet.java
@@ -1,7 +1,5 @@
 package org.jboss.forge.addon.docker.facets;
 
-import java.io.File;
-
 import org.jboss.forge.addon.docker.resource.DockerFileResource;
 import org.jboss.forge.addon.facets.Facet;
 import org.jboss.forge.addon.projects.ProjectFacet;
@@ -20,13 +18,6 @@ public interface DockerFacet extends ProjectFacet
     */
    DockerFileResource getDockerfileResource();
 
-   /**
-    * Set the {@link DockerFileResource} installed by this facet with this {@link File}.
-    * 
-    * @param file {@link File} whose contents are set to the facet Dockerfile.
-    */
-   void setDockerfile(File file);
-
    /**
     * Set the {@link DockerFileResource} installed by this facet with the {@link DockerFileResource} passed.
     * 

diff --git a/src/main/java/org/jboss/forge/addon/docker/facets/DockerFacetImpl.java b/src/main/java/org/jboss/forge/addon/docker/facets/DockerFacetImpl.java
@@ -1,11 +1,5 @@
 package org.jboss.forge.addon.docker.facets;
 
-import java.io.File;
-import java.io.FileInputStream;
-import java.io.FileNotFoundException;
-import java.io.IOException;
-import java.io.InputStream;
-
 import org.jboss.forge.addon.docker.resource.DockerFileResource;
 import org.jboss.forge.addon.facets.AbstractFacet;
 import org.jboss.forge.addon.projects.Project;
@@ -37,33 +31,6 @@ public DockerFileResource getDockerfileResource()
       return getFaceted().getRoot().getChild("Dockerfile").reify(DockerFileResource.class);
    }
 
-   @Override
-   public void setDockerfile(File file)
-   {
-
-      DockerFileResource dockerfile = getFaceted().getRoot().getChild("Dockerfile")
-               .reify(DockerFileResource.class);
-
-      InputStream is = null;
-      try
-      {
-         is = new FileInputStream(file);
-      }
-      catch (FileNotFoundException e1)
-      {
-      }
-      dockerfile.setContents(is);
-
-      try
-      {
-         is.close();
-      }
-      catch (IOException e)
-      {
-      }
-
-   }
-
    public void setDockerFileResource(DockerFileResource dockerFileResource)
    {
       DockerFileResource dockerfile = getFaceted().getRoot().getChild("Dockerfile")

diff --git a/src/test/java/org/jboss/forge/addon/docker/facets/DockerFacetImplTest.java b/src/test/java/org/jboss/forge/addon/docker/facets/DockerFacetImplTest.java
@@ -1,7 +1,5 @@
 package org.jboss.forge.addon.docker.facets;
 
-import java.io.File;
-
 import javax.inject.Inject;
 
 import org.jboss.arquillian.container.test.api.Deployment;
@@ -10,7 +8,6 @@
 import org.jboss.forge.addon.facets.FacetFactory;
 import org.jboss.forge.addon.projects.Project;
 import org.jboss.forge.addon.projects.ProjectFactory;
-import org.jboss.forge.addon.resource.FileResource;
 import org.jboss.forge.addon.resource.ResourceFactory;
 import org.jboss.forge.arquillian.AddonDependencies;
 import org.jboss.forge.arquillian.archive.AddonArchive;
@@ -47,14 +44,6 @@ public void testDockerFacet() throws Exception
       DockerFileResource dockerfileResource = dockerFacet.getDockerfileResource();
       Assert.assertEquals("#Dockerfile for your project", dockerfileResource.getContents());
 
-      File f = File.createTempFile("Dockerfile", "");
-      @SuppressWarnings("unchecked")
-      FileResource<?> resource = rf.create(FileResource.class, f);
-      resource.setContents("#new Dockerfile");
-      dockerFacet.setDockerfile(resource.getUnderlyingResourceObject());
-      dockerfileResource = dockerFacet.getDockerfileResource();
-      Assert.assertEquals("#new Dockerfile", dockerFacet.getDockerfileResource().getContents());
-
    }
 
 }