---

mderka · mderka · commit 1d77ee0bc839 · 2016-02-01T16:14:09.000-08:00
yaml --- r: 6811 b: refs/heads/tswast-patch-1 c: 6ecde35 h: refs/heads/master i: 6809: 738deda 6807: 13cc542
diff --git a/[refs] b/[refs]
@@ -57,5 +57,5 @@ refs/tags/v0.18.0: 9d193c4c4b9d1c6f21515dd8e50836b9194ec9bb
 refs/tags/v0.19.0: e67b56e4d8dad5f9a7b38c9b2107c23c828f2ed5
 refs/tags/v0.20.0: 839f7fb7156535146aa1cb2c5aadd8d375d854e8
 refs/tags/v0.20.1: 370471f437f1f4f68a11e068df5cd6bf39edb1fa
-refs/heads/tswast-patch-1: a9cc927e002277face656af9f059175192d31c3d
+refs/heads/tswast-patch-1: 6ecde35c355ca8e38e726044e9a0012f5a3935ab
 refs/heads/pubsub-streaming-pull: 19262b752ee874eb2ca3b950eb2aef44d5a5267b
diff --git a/branches/tswast-patch-1/gcloud-java-dns/src/main/java/com/google/gcloud/dns/Dns.java b/branches/tswast-patch-1/gcloud-java-dns/src/main/java/com/google/gcloud/dns/Dns.java
@@ -24,15 +24,15 @@
 import java.io.Serializable;
 import java.util.Set;
 
+import static com.google.gcloud.dns.Dns.ZoneField.selector;
+
 /**
  * An interface for the Google Cloud DNS service.
  *
  * @see <a href="https://cloud.google.com/dns/docs">Google Cloud DNS</a>
  */
 public interface Dns extends Service<DnsOptions> {
 
-
-
   /**
    * The fields of a project.
    *
@@ -284,7 +284,9 @@ class ZoneListOption extends AbstractOption implements Serializable {
      * specified. {@link ZoneField} provides a list of fields that can be used.
      */
     public static ZoneListOption fields(ZoneField... fields) {
-      return new ZoneListOption(DnsRpc.Option.FIELDS, ZoneField.selector(fields));
+      StringBuilder builder = new StringBuilder();
+      builder.append("managedZones(").append(selector(fields)).append(')');
+      return new ZoneListOption(DnsRpc.Option.FIELDS, builder.toString());
     }
 
     /**
@@ -381,10 +383,9 @@ class ChangeRequestListOption extends AbstractOption implements Serializable {
      * a list of fields that can be used.
      */
     public static ChangeRequestListOption fields(ChangeRequestField... fields) {
-      return new ChangeRequestListOption(
-          DnsRpc.Option.FIELDS,
-          ChangeRequestField.selector(fields)
-      );
+      StringBuilder builder = new StringBuilder();
+      builder.append("changes(").append(ChangeRequestField.selector(fields)).append(')');
+      return new ChangeRequestListOption(DnsRpc.Option.FIELDS, builder.toString());
     }
 
     /**
diff --git a/branches/tswast-patch-1/gcloud-java-dns/src/test/java/com/google/gcloud/dns/AbstractOptionTest.java b/branches/tswast-patch-1/gcloud-java-dns/src/test/java/com/google/gcloud/dns/AbstractOptionTest.java
@@ -16,30 +16,26 @@
 
 package com.google.gcloud.dns;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.fail;
-
 import com.google.gcloud.spi.DnsRpc;
 
 import org.junit.Test;
 
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotEquals;
+import static org.junit.Assert.fail;
+
 public class AbstractOptionTest {
 
   private static final DnsRpc.Option RPC_OPTION = DnsRpc.Option.DNS_TYPE;
   private static final DnsRpc.Option ANOTHER_RPC_OPTION = DnsRpc.Option.DNS_NAME;
   private static final String VALUE = "some value";
   private static final String OTHER_VALUE = "another value";
-  private static final AbstractOption OPTION = new AbstractOption(RPC_OPTION, VALUE) {
-  };
-  private static final AbstractOption OPTION_EQUALS = new AbstractOption(RPC_OPTION, VALUE) {
-  };
+  private static final AbstractOption OPTION = new AbstractOption(RPC_OPTION, VALUE) {};
+  private static final AbstractOption OPTION_EQUALS = new AbstractOption(RPC_OPTION, VALUE) {};
   private static final AbstractOption OPTION_NOT_EQUALS1 =
-      new AbstractOption(RPC_OPTION, OTHER_VALUE) {
-      };
+      new AbstractOption(RPC_OPTION, OTHER_VALUE) {};
   private static final AbstractOption OPTION_NOT_EQUALS2 =
-      new AbstractOption(ANOTHER_RPC_OPTION, VALUE) {
-      };
+      new AbstractOption(ANOTHER_RPC_OPTION, VALUE) {};
 
   @Test
   public void testEquals() {
@@ -58,13 +54,11 @@ public void testConstructor() {
     assertEquals(RPC_OPTION, OPTION.rpcOption());
     assertEquals(VALUE, OPTION.value());
     try {
-      new AbstractOption(null, VALUE) {
-      };
+      new AbstractOption(null, VALUE) {};
       fail("Cannot build with empty option.");
     } catch (NullPointerException e) {
       // expected
     }
-    new AbstractOption(RPC_OPTION, null) {
-    }; // null value is ok
+    new AbstractOption(RPC_OPTION, null) {}; // null value is ok
   }
 }
