Add list/get options serialization tests + other small fixes

Author: Ajay Kannan

gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ProjectInfo.java

@@ -132,7 +132,7 @@ private Builder() {
     Builder(ProjectInfo info) {
       this.name = info.name;
       this.projectId = info.projectId;
-      this.labels = Maps.newHashMap(checkNotNull(info.labels));
+      this.labels.putAll(info.labels);
       this.projectNumber = info.projectNumber;
       this.state = info.state;
       this.createTimeMillis = info.createTimeMillis;
@@ -159,7 +159,7 @@ public Builder name(String name) {
      * project.
      */
     public Builder projectId(String projectId) {
-      this.projectId = projectId;
+      this.projectId = checkNotNull(projectId);
       return this;
     }
 
@@ -230,7 +230,7 @@ public ProjectInfo build() {
 
   ProjectInfo(Builder builder) {
     this.name = builder.name;
-    this.projectId = checkNotNull(builder.projectId);
+    this.projectId = builder.projectId;
     this.labels = ImmutableMap.copyOf(builder.labels);
     this.projectNumber = builder.projectNumber;
     this.state = builder.state;

gcloud-java-resourcemanager/src/main/java/com/google/gcloud/resourcemanager/ResourceManager.java

@@ -36,7 +36,7 @@ public interface ResourceManager extends Service<ResourceManagerOptions> {
   /**
    * The fields of a project.
    *
-   * These values can be used to specify the fields to include of in a partial response when calling
+   * These values can be used to specify the fields to include in a partial response when calling
    * {@link ResourceManager#get} or {@link ResourceManager#list}. Project ID is always returned,
    * even if not specified.
    */

gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/DefaultResourceManagerRpc.java

@@ -63,7 +63,7 @@ public void delete(String projectId) throws ResourceManagerException {
   }
 
   @Override
-  public Project get(String projectId) throws ResourceManagerException {
+  public Project get(String projectId, Map<Option, ?> options) throws ResourceManagerException {
     // TODO(ajaykannan): fix me!
     return null;
   }

gcloud-java-resourcemanager/src/main/java/com/google/gcloud/spi/ResourceManagerRpc.java

@@ -77,7 +77,7 @@ public Y y() {
 
   void delete(String projectId) throws ResourceManagerException;
 
-  Project get(String projectId) throws ResourceManagerException;
+  Project get(String projectId, Map<Option, ?> options) throws ResourceManagerException;
 
   Tuple<String, Iterable<Project>> list(Map<Option, ?> options) throws ResourceManagerException;
 

gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/ProjectTest.java

@@ -18,7 +18,6 @@
 
 import static org.easymock.EasyMock.createStrictMock;
 import static org.easymock.EasyMock.expect;
-import static org.easymock.EasyMock.expectLastCall;
 import static org.easymock.EasyMock.replay;
 import static org.easymock.EasyMock.verify;
 import static org.junit.Assert.assertEquals;
@@ -94,15 +93,13 @@ public void testResourceManager() {
   @Test
   public void testDelete() {
     resourceManager.delete(PROJECT_INFO.projectId());
-    expectLastCall();
     replay(resourceManager);
     project.delete();
   }
 
   @Test
   public void testUndelete() {
     resourceManager.undelete(PROJECT_INFO.projectId());
-    expectLastCall();
     replay(resourceManager);
     project.undelete();
   }

gcloud-java-resourcemanager/src/test/java/com/google/gcloud/resourcemanager/SerializationTest.java

@@ -46,6 +46,10 @@ public class SerializationTest {
       .build();
   private static final PageImpl<ProjectInfo> PAGE_RESULT =
       new PageImpl<>(null, "c", Collections.singletonList(PARTIAL_PROJECT_INFO));
+  private static final ResourceManager.ProjectGetOption PROJECT_GET_OPTION =
+      ResourceManager.ProjectGetOption.fields(ResourceManager.ProjectField.NAME);
+  private static final ResourceManager.ProjectListOption PROJECT_LIST_OPTION =
+      ResourceManager.ProjectListOption.filter("name:*");
 
   @Test
   public void testServiceOptions() throws Exception {
@@ -63,7 +67,8 @@ public void testServiceOptions() throws Exception {
 
   @Test
   public void testModelAndRequests() throws Exception {
-    Serializable[] objects = {PARTIAL_PROJECT_INFO, FULL_PROJECT_INFO, PAGE_RESULT};
+    Serializable[] objects = {PARTIAL_PROJECT_INFO, FULL_PROJECT_INFO, PAGE_RESULT,
+        PROJECT_GET_OPTION, PROJECT_LIST_OPTION};
     for (Serializable obj : objects) {
       Object copy = serializeAndDeserialize(obj);
       assertEquals(obj, obj);