---

mderka · mderka · commit 25d66330eeaa · 2016-02-01T16:14:09.000-08:00
yaml --- r: 3543 b: refs/heads/pubsub-alpha c: 6ecde35 h: refs/heads/master i: 3541: 5f3defa 3539: 2b6fe61 3535: 3949e1a
diff --git a/[refs] b/[refs]
@@ -2,7 +2,7 @@
 refs/heads/master: 36a62ef856d199f8efd09501b5ba65c422c01f23
 refs/heads/travis: e21ee7b88a5edc3f3d8c71f90c3fc32abf7e8dd6
 refs/heads/gh-pages: 7406918e071dd2c5677a638ae2a06e7592b6542c
-refs/heads/pubsub-alpha: a9cc927e002277face656af9f059175192d31c3d
+refs/heads/pubsub-alpha: 6ecde35c355ca8e38e726044e9a0012f5a3935ab
 refs/heads/update-datastore: 47aae517c2cb33f1dccd909adaced73ec9d0f4df
 refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
 refs/tags/v0.0.10: 207ebd2a3472fddee69fe1298eb90429e3306efd
diff --git a/branches/pubsub-alpha/gcloud-java-dns/src/main/java/com/google/gcloud/dns/Dns.java b/branches/pubsub-alpha/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/pubsub-alpha/gcloud-java-dns/src/test/java/com/google/gcloud/dns/AbstractOptionTest.java b/branches/pubsub-alpha/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
   }
 }
