Skip to content

Commit e88bbc4

Browse files
committed
Fix gemini suggestions and lint more import order.
1 parent 757f877 commit e88bbc4

2 files changed

Lines changed: 10 additions & 11 deletions

File tree

logging/samples/snippets/snippets.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,14 @@
2424
import argparse
2525
import time
2626

27-
import logging
28-
2927
import google.cloud.logging
3028

29+
import logging
30+
3131

3232
# [START logging_write_log_entry]
3333
def write_entry():
34-
"""Writes log entries to the given logger."""
34+
"""Writes log entries to the default logger."""
3535
logging_client = google.cloud.logging.Client()
3636

3737
# By default, all logs sent through setup_logging()

logging/samples/snippets/snippets_test.py

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
TEST_LOGGER_NAME = "example_log_{}".format(uuid.uuid4().hex)
2626
TEST_TEXT = "Hello, world."
2727
GOOGLE_CLOUD_PROJECT = os.getenv("GOOGLE_CLOUD_PROJECT")
28-
DEFAULT_LOG_ID = f"projects/{GOOGLE_CLOUD_PROJECT}/logs/python"
28+
DEFAULT_LOGGER = f"projects/{GOOGLE_CLOUD_PROJECT}/logs/python"
2929

3030

3131
@pytest.fixture
@@ -56,18 +56,17 @@ def eventually_consistent_test():
5656
# retrieve logs
5757
client = logging.Client()
5858

59-
log_filter = DEFAULT_LOG_ID
59+
log_filter = DEFAULT_LOGGER
6060

6161
entries = client.list_entries(
6262
filter_=log_filter, order_by=logging.DESCENDING, max_results=3
6363
)
6464

65-
entry_1 = next(entries)
66-
assert entry_1.payload["message"] == "This is a JSON log."
67-
entry_2 = next(entries)
68-
assert entry_2.payload == "Goodbye, world!"
69-
entry_3 = next(entries)
70-
assert entry_3.payload == "Hello, world!"
65+
retrieved_entries = list(entries)
66+
67+
assert retrieved_entries[0].payload["message"] == "This is a JSON log."
68+
assert retrieved_entries[1].payload == "Goodbye, world!"
69+
assert retrieved_entries[2].payload == "Hello, world!"
7170

7271
eventually_consistent_test()
7372

0 commit comments

Comments
 (0)