Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: added regional endpoint sample for datastore #1043

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@
import com.google.cloud.datastore.Key;

public class QuickstartSample {

kolea2 marked this conversation as resolved.
Show resolved Hide resolved

public static void main(String... args) throws Exception {
// Instantiates a client
Datastore datastore = DatastoreOptions.getDefaultInstance().getService();
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
/*
* Copyright 2023 Google Inc.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package com.example.datastore;

// [START datastore_regional_endpoint]
// Imports the Google Cloud client library
import com.google.cloud.datastore.Datastore;
import com.google.cloud.datastore.DatastoreOptions;
import com.google.cloud.datastore.Entity;
import com.google.cloud.datastore.Key;

public class RegionalEndpoint {


public static void main(String... args) throws Exception {
// Instantiates a client
DatastoreOptions options = DatastoreOptions.newBuilder()
.setProjectId("datastore-project-382616")
.setHost("https://nam5-firestore.googleapis.com")
kolea2 marked this conversation as resolved.
Show resolved Hide resolved
.build();
Datastore datastore = options.getService();

// The kind for the new entity
String kind = "Task";
// The name/ID for the new entity
String name = "sampletask1";
// The Cloud Datastore key for the new entity
Key taskKey = datastore.newKeyFactory().setKind(kind).newKey(name);

// Prepares the new entity
Entity task = Entity.newBuilder(taskKey).set("description", "Buy milk").build();

// Saves the entity
datastore.put(task);

System.out.printf("Saved %s: %s%n", task.getKey().getName(), task.getString("description"));

// Retrieve entity
Entity retrieved = datastore.get(taskKey);

System.out.printf("Retrieved %s: %s%n", taskKey.getName(), retrieved.getString("description"));

}
}
// [END datastore_regional_endpoint]
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
/*
* Copyright 2023 Google Inc.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/

package com.example.datastore;

import com.google.cloud.datastore.Datastore;
import com.google.cloud.datastore.DatastoreOptions;
import com.google.cloud.datastore.Key;
import com.rule.SystemsOutRule;
import org.junit.After;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;

/** Tests for quickstart sample. */
@RunWith(JUnit4.class)
@SuppressWarnings("checkstyle:abbreviationaswordinname")
public class RegionalEndpointIT {

@Rule public final SystemsOutRule systemsOutRule = new SystemsOutRule();

private static final void deleteTestEntity() {
Datastore datastore = DatastoreOptions.getDefaultInstance().getService();
kolea2 marked this conversation as resolved.
Show resolved Hide resolved
String kind = "Task";
String name = "sampletask1";
Key taskKey = datastore.newKeyFactory().setKind(kind).newKey(name);
datastore.delete(taskKey);
}

@Before
public void setUp() {
deleteTestEntity();
}

@After
public void tearDown() {
System.setOut(null);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue: Remove this. If you use the SystemsOutRule, you don't need to change or restore what goes pipes to stdout.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

deleteTestEntity();
}

@Test
public void testQuickstart() throws Exception {
QuickstartSample.main();
kolea2 marked this conversation as resolved.
Show resolved Hide resolved
systemsOutRule.assertContains("Saved sampletask1: Buy milk");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue: it looks like this test is upserting an entity and then querying the Datastore for the same entity. You don't need to do a string comparison here. Instead just compare the Entity you upsert to the Entity you receive.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

systemsOutRule.assertContains("Retrieved sampletask1: Buy milk");
}
}