Proper template config in the image extension live tests

Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/cce44958
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/cce44958
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/cce44958

Branch: refs/heads/master
Commit: cce449587289d7333afe4f8e095bb64367255adb
Parents: 6cd22b0
Author: Ignasi Barrera <n...@apache.org>
Authored: Thu Oct 13 12:29:14 2016 +0200
Committer: Ignasi Barrera <n...@apache.org>
Committed: Thu Oct 13 15:06:50 2016 +0200

----------------------------------------------------------------------
 .../compute/extensions/EC2ImageExtensionLiveTest.java   |  6 +++---
 .../extensions/internal/BaseImageExtensionLiveTest.java | 12 ++++++------
 2 files changed, 9 insertions(+), 9 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/cce44958/apis/ec2/src/test/java/org/jclouds/ec2/compute/extensions/EC2ImageExtensionLiveTest.java
----------------------------------------------------------------------
diff --git 
a/apis/ec2/src/test/java/org/jclouds/ec2/compute/extensions/EC2ImageExtensionLiveTest.java
 
b/apis/ec2/src/test/java/org/jclouds/ec2/compute/extensions/EC2ImageExtensionLiveTest.java
index be8abb6..f80bcdf 100644
--- 
a/apis/ec2/src/test/java/org/jclouds/ec2/compute/extensions/EC2ImageExtensionLiveTest.java
+++ 
b/apis/ec2/src/test/java/org/jclouds/ec2/compute/extensions/EC2ImageExtensionLiveTest.java
@@ -26,7 +26,7 @@ import java.util.Set;
 
 import org.jclouds.collect.Memoized;
 import org.jclouds.compute.domain.Image;
-import org.jclouds.compute.domain.Template;
+import org.jclouds.compute.domain.TemplateBuilder;
 import org.jclouds.compute.domain.TemplateBuilderSpec;
 import org.jclouds.compute.extensions.ImageExtension;
 import org.jclouds.compute.extensions.internal.BaseImageExtensionLiveTest;
@@ -95,8 +95,8 @@ public class EC2ImageExtensionLiveTest extends 
BaseImageExtensionLiveTest {
    }
 
    @Override
-   public Template getNodeTemplate() {
-      return 
view.getComputeService().templateBuilder().from(ebsTemplate).build();
+   public TemplateBuilder getNodeTemplate() {
+      return getNodeTemplate().from(ebsTemplate);
    }
 
    @Override

http://git-wip-us.apache.org/repos/asf/jclouds/blob/cce44958/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseImageExtensionLiveTest.java
----------------------------------------------------------------------
diff --git 
a/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseImageExtensionLiveTest.java
 
b/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseImageExtensionLiveTest.java
index 878768e..0e2e77a 100644
--- 
a/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseImageExtensionLiveTest.java
+++ 
b/compute/src/test/java/org/jclouds/compute/extensions/internal/BaseImageExtensionLiveTest.java
@@ -35,6 +35,7 @@ import org.jclouds.compute.domain.Image;
 import org.jclouds.compute.domain.ImageTemplate;
 import org.jclouds.compute.domain.NodeMetadata;
 import org.jclouds.compute.domain.Template;
+import org.jclouds.compute.domain.TemplateBuilder;
 import org.jclouds.compute.extensions.ImageExtension;
 import org.jclouds.compute.internal.BaseComputeServiceContextLiveTest;
 import org.jclouds.compute.reference.ComputeServiceConstants;
@@ -66,8 +67,8 @@ public abstract class BaseImageExtensionLiveTest extends 
BaseComputeServiceConte
     * 
     * @return
     */
-   public Template getNodeTemplate() {
-      return view.getComputeService().templateBuilder().build();
+   public TemplateBuilder getNodeTemplate() {
+      return view.getComputeService().templateBuilder();
    }
 
    /**
@@ -96,7 +97,7 @@ public abstract class BaseImageExtensionLiveTest extends 
BaseComputeServiceConte
       Optional<ImageExtension> imageExtension = 
computeService.getImageExtension();
       assertTrue(imageExtension.isPresent(), "image extension was not 
present");
 
-      Template template = getNodeTemplate();
+      Template template = getNodeTemplate().build();
       NodeMetadata node = 
Iterables.getOnlyElement(computeService.createNodesInGroup(imageGroup, 1, 
template));
       checkReachable(node);
 
@@ -128,10 +129,9 @@ public abstract class BaseImageExtensionLiveTest extends 
BaseComputeServiceConte
       Optional<? extends Image> optImage = getImage();
       assertTrue(optImage.isPresent());
 
-      NodeMetadata node = 
Iterables.getOnlyElement(computeService.createNodesInGroup(imageGroup, 1, view
-               .getComputeService()
+      NodeMetadata node = 
Iterables.getOnlyElement(computeService.createNodesInGroup(imageGroup, 1, 
getNodeTemplate()
                // fromImage does not use the arg image's id (but we do need to 
set location)
-               
.templateBuilder().imageId(optImage.get().getId()).fromImage(optImage.get()).build()));
+               
.imageId(optImage.get().getId()).fromImage(optImage.get()).build()));
 
       checkReachable(node);
       view.getComputeService().destroyNode(node.getId());

Reply via email to