mirror of https://github.com/apache/jclouds.git
remove addl whitespace pointed out in JCLOUDS-331
This commit is contained in:
parent
9a7336bb0e
commit
3ecbf90847
|
@ -86,7 +86,7 @@ public class EC2TemplateBuilderTest {
|
||||||
.parent(provider).build();
|
.parent(provider).build();
|
||||||
|
|
||||||
public static final Hardware CC1_4XLARGE = cc1_4xlarge().supportsImageIds(ImmutableSet.of("us-east-1/cc-image"))
|
public static final Hardware CC1_4XLARGE = cc1_4xlarge().supportsImageIds(ImmutableSet.of("us-east-1/cc-image"))
|
||||||
.build();
|
.build();
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Verifies that {@link TemplateBuilderImpl} would choose the correct size of the instance, based
|
* Verifies that {@link TemplateBuilderImpl} would choose the correct size of the instance, based
|
||||||
|
|
|
@ -95,7 +95,7 @@ public class EC2TemplateBuilderImplTest extends TemplateBuilderImplTest {
|
||||||
protected String getProviderFormatId(String uniqueLabel) {
|
protected String getProviderFormatId(String uniqueLabel) {
|
||||||
return "us-east-1/"+uniqueLabel;
|
return "us-east-1/"+uniqueLabel;
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
@Test
|
@Test
|
||||||
public void testHardwareWithImageIdPredicateOnlyAcceptsImageWhenLocationNull() {
|
public void testHardwareWithImageIdPredicateOnlyAcceptsImageWhenLocationNull() {
|
||||||
|
|
|
@ -178,7 +178,7 @@ public interface TemplateBuilder {
|
||||||
* This strategy may change from version to version.
|
* This strategy may change from version to version.
|
||||||
*/
|
*/
|
||||||
TemplateBuilder imageChooser(Function<Iterable<? extends Image>, Image> imageChooser);
|
TemplateBuilder imageChooser(Function<Iterable<? extends Image>, Image> imageChooser);
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Configure this template to require the minimum cores below
|
* Configure this template to require the minimum cores below
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -505,7 +505,7 @@ public class TemplateBuilderImpl implements TemplateBuilder {
|
||||||
Ordering.<String> natural().nullsLast()).result();
|
Ordering.<String> natural().nullsLast()).result();
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
@VisibleForTesting
|
@VisibleForTesting
|
||||||
// non-static for logging
|
// non-static for logging
|
||||||
final Function<Iterable<? extends Image>, Image> imageChooserFromOrdering(final Ordering<Image> ordering) {
|
final Function<Iterable<? extends Image>, Image> imageChooserFromOrdering(final Ordering<Image> ordering) {
|
||||||
|
|
|
@ -88,12 +88,12 @@ public class TemplateBuilderImplTest {
|
||||||
Iterable<String> values = Arrays.asList("1", "3", null, "2", "2");
|
Iterable<String> values = Arrays.asList("1", "3", null, "2", "2");
|
||||||
assertEquals(TemplateBuilderImpl.multiMax(Ordering.natural().nullsLast(), values), Arrays.asList((Object)null));
|
assertEquals(TemplateBuilderImpl.multiMax(Ordering.natural().nullsLast(), values), Arrays.asList((Object)null));
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testMultiMaxNulls() {
|
public void testMultiMaxNulls() {
|
||||||
Iterable<String> values = Arrays.asList("1", "3", null, "2", "2", null);
|
Iterable<String> values = Arrays.asList("1", "3", null, "2", "2", null);
|
||||||
assertEquals(TemplateBuilderImpl.multiMax(Ordering.natural().nullsLast(), values), Arrays.asList((Object)null, null));
|
assertEquals(TemplateBuilderImpl.multiMax(Ordering.natural().nullsLast(), values), Arrays.asList((Object)null, null));
|
||||||
}
|
}
|
||||||
|
|
||||||
protected Location provider = new LocationBuilder().scope(LocationScope.PROVIDER).id("aws-ec2").description("aws-ec2").build();
|
protected Location provider = new LocationBuilder().scope(LocationScope.PROVIDER).id("aws-ec2").description("aws-ec2").build();
|
||||||
|
|
||||||
protected Location region = new LocationBuilder().scope(LocationScope.REGION).id("us-east-1")
|
protected Location region = new LocationBuilder().scope(LocationScope.REGION).id("us-east-1")
|
||||||
|
@ -115,13 +115,13 @@ public class TemplateBuilderImplTest {
|
||||||
|
|
||||||
protected Image image64bit = ImageBuilder.fromImage(image).id(getProviderFormatId("image64bId")).providerId("image64bId")
|
protected Image image64bit = ImageBuilder.fromImage(image).id(getProviderFormatId("image64bId")).providerId("image64bId")
|
||||||
.operatingSystem(os.toBuilder().arch("X86_64").build()).build();
|
.operatingSystem(os.toBuilder().arch("X86_64").build()).build();
|
||||||
|
|
||||||
protected Image imageArchNull = ImageBuilder.fromImage(image).id(getProviderFormatId("imageArchNullId")).providerId("imageArchNullId")
|
protected Image imageArchNull = ImageBuilder.fromImage(image).id(getProviderFormatId("imageArchNullId")).providerId("imageArchNullId")
|
||||||
.operatingSystem(os.toBuilder().arch(null).build()).build();
|
.operatingSystem(os.toBuilder().arch(null).build()).build();
|
||||||
|
|
||||||
protected Image imageNameAlt = ImageBuilder.fromImage(image).id(getProviderFormatId("imageNameAltId")).providerId("imageNameAltId")
|
protected Image imageNameAlt = ImageBuilder.fromImage(image).id(getProviderFormatId("imageNameAltId")).providerId("imageNameAltId")
|
||||||
.name("alternateImageName").build();
|
.name("alternateImageName").build();
|
||||||
|
|
||||||
@SuppressWarnings("unchecked")
|
@SuppressWarnings("unchecked")
|
||||||
public void testLocationPredicateWhenComputeMetadataIsNotLocationBound() {
|
public void testLocationPredicateWhenComputeMetadataIsNotLocationBound() {
|
||||||
|
|
||||||
|
@ -189,7 +189,7 @@ public class TemplateBuilderImplTest {
|
||||||
Suppliers.<Set<? extends Image>> ofInstance(ImmutableSet.<Image> of(image, image64bit)),
|
Suppliers.<Set<? extends Image>> ofInstance(ImmutableSet.<Image> of(image, image64bit)),
|
||||||
image64bit);
|
image64bit);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testResolveImagesPrefersNull() {
|
public void testResolveImagesPrefersNull() {
|
||||||
// preferring null has been the default behaviour; not sure if this is ideal
|
// preferring null has been the default behaviour; not sure if this is ideal
|
||||||
// (would make more sense to prefer nonNull) but don't change behaviour by default
|
// (would make more sense to prefer nonNull) but don't change behaviour by default
|
||||||
|
@ -214,7 +214,7 @@ public class TemplateBuilderImplTest {
|
||||||
};
|
};
|
||||||
assertTrue(TemplateBuilderImpl.DEFAULT_IMAGE_ORDERING.compare(image64bit, imageArchNull) < 0, "wrong default image ordering");
|
assertTrue(TemplateBuilderImpl.DEFAULT_IMAGE_ORDERING.compare(image64bit, imageArchNull) < 0, "wrong default image ordering");
|
||||||
assertTrue(sorterPreferringNonNullArch.compare(image64bit, imageArchNull) > 0, "wrong custom image ordering");
|
assertTrue(sorterPreferringNonNullArch.compare(image64bit, imageArchNull) > 0, "wrong custom image ordering");
|
||||||
|
|
||||||
Function<TemplateBuilderImpl, TemplateBuilderImpl> tbCustomiser = new Function<TemplateBuilderImpl, TemplateBuilderImpl>() {
|
Function<TemplateBuilderImpl, TemplateBuilderImpl> tbCustomiser = new Function<TemplateBuilderImpl, TemplateBuilderImpl>() {
|
||||||
@Override
|
@Override
|
||||||
public TemplateBuilderImpl apply(TemplateBuilderImpl input) {
|
public TemplateBuilderImpl apply(TemplateBuilderImpl input) {
|
||||||
|
|
Loading…
Reference in New Issue