fix: add some missing annotations and fix equals/hashcode for DatastoreOptions (#1106)

* chore(multi db): add some missing annotations and fix equals/hashcode on DatastoreOptions

* add some tests

Author: kolea2

datastore-v1-proto-client/src/main/java/com/google/datastore/v1/client/DatastoreOptions.java

@@ -188,6 +188,7 @@ public String getProjectId() {
     return projectId;
   }
 
+  @BetaApi
   public String getDatabaseId() {
     return databaseId;
   }

google-cloud-datastore/src/main/java/com/google/cloud/datastore/BaseKey.java

@@ -148,6 +148,7 @@ public String getNamespace() {
     return namespace;
   }
 
+  @BetaApi
   public String getDatabaseId() {
     return databaseId;
   }

google-cloud-datastore/src/main/java/com/google/cloud/datastore/DatastoreOptions.java

@@ -155,6 +155,7 @@ public String getNamespace() {
     return namespace;
   }
 
+  @BetaApi
   public String getDatabaseId() {
     return this.databaseId;
   }
@@ -193,7 +194,7 @@ public Builder toBuilder() {
 
   @Override
   public int hashCode() {
-    return Objects.hash(baseHashCode(), namespace);
+    return Objects.hash(baseHashCode(), namespace, databaseId);
   }
 
   @Override
@@ -202,7 +203,9 @@ public boolean equals(Object obj) {
       return false;
     }
     DatastoreOptions other = (DatastoreOptions) obj;
-    return baseEquals(other) && Objects.equals(namespace, other.namespace);
+    return baseEquals(other)
+        && Objects.equals(namespace, other.namespace)
+        && Objects.equals(databaseId, other.databaseId);
   }
 
   public static Builder newBuilder() {

google-cloud-datastore/src/test/java/com/google/cloud/datastore/DatastoreOptionsTest.java

@@ -17,6 +17,7 @@
 package com.google.cloud.datastore;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertSame;
 import static org.junit.Assert.assertTrue;
@@ -38,7 +39,6 @@ public class DatastoreOptionsTest {
   private DatastoreRpc datastoreRpc;
   private DatastoreOptions.Builder options;
 
-  // todo parameterize
   @Before
   public void setUp() {
     datastoreRpcFactory = EasyMock.createMock(DatastoreRpcFactory.class);
@@ -90,6 +90,12 @@ public void testToBuilder() {
     assertEquals(original.getHost(), copy.getHost());
     assertEquals(original.getRetrySettings(), copy.getRetrySettings());
     assertEquals(original.getCredentials(), copy.getCredentials());
+    assertEquals(original, copy);
+    assertEquals(original.hashCode(), copy.hashCode());
+
+    DatastoreOptions newOptions = options.setDatabaseId("new-database-id").build();
+    assertNotEquals(original, newOptions);
+    assertNotEquals(original.hashCode(), newOptions.hashCode());
   }
 
   @Test