Skip to content
This repository was archived by the owner on Nov 8, 2023. It is now read-only.

Commit abd82e1

Browse files
chore(python): run blacken session for all directories with a noxfile (#222)
Source-Link: googleapis/synthtool@bc0de6e Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:39ad8c0570e4f5d2d3124a509de4fe975e799e2b97e0f58aed88f8880d5a8b60 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
1 parent 32245c8 commit abd82e1

24 files changed

+303
-124
lines changed

.github/.OwlBot.lock.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
docker:
22
image: gcr.io/cloud-devrel-public-resources/owlbot-python:latest
3-
digest: sha256:ec49167c606648a063d1222220b48119c912562849a0528f35bfb592a9f72737
3+
digest: sha256:39ad8c0570e4f5d2d3124a509de4fe975e799e2b97e0f58aed88f8880d5a8b60

samples/snippets/create_cluster.py

Lines changed: 20 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -42,29 +42,39 @@ def create_cluster(project_id, location, realm_id, cluster_id, gke_cluster_name)
4242
game_server_cluster=game_server_clusters.GameServerCluster(
4343
description="My Game Server Cluster",
4444
connection_info=game_server_clusters.GameServerClusterConnectionInfo(
45-
gke_cluster_reference=gke_cluster_reference,
46-
namespace="default",
45+
gke_cluster_reference=gke_cluster_reference, namespace="default",
4746
),
4847
),
4948
)
5049

5150
operation = client.create_game_server_cluster(request)
5251
print(f"Create cluster operation: {operation.operation.name}")
5352
operation.result(timeout=120)
53+
54+
5455
# [END cloud_game_servers_cluster_create]
5556

5657

5758
if __name__ == "__main__":
5859
parser = argparse.ArgumentParser()
59-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
60-
parser.add_argument('--location', help='Your realm location.', required=True)
61-
parser.add_argument('--realm-id', help='Your realm ID.', required=True)
62-
parser.add_argument('--cluster-id', help='Your game server cluster ID.', required=True)
60+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
61+
parser.add_argument("--location", help="Your realm location.", required=True)
62+
parser.add_argument("--realm-id", help="Your realm ID.", required=True)
63+
parser.add_argument(
64+
"--cluster-id", help="Your game server cluster ID.", required=True
65+
)
6366
parser.add_argument(
64-
'--gke-cluster-name',
65-
help='The name of the GKE cluster which is managed by the game server cluster being created.',
66-
required=True)
67+
"--gke-cluster-name",
68+
help="The name of the GKE cluster which is managed by the game server cluster being created.",
69+
required=True,
70+
)
6771

6872
args = parser.parse_args()
6973

70-
create_cluster(args.project_id, args.location, args.realm_id, args.cluster_id, args.gke_cluster_name)
74+
create_cluster(
75+
args.project_id,
76+
args.location,
77+
args.realm_id,
78+
args.cluster_id,
79+
args.gke_cluster_name,
80+
)

samples/snippets/create_deployment.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,17 @@ def create_deployment(project_id, deployment_id):
4646
operation = client.create_game_server_deployment(request)
4747
print(f"Create deployment operation: {operation.operation.name}")
4848
operation.result(timeout=120)
49+
50+
4951
# [END cloud_game_servers_deployment_create]
5052

5153

5254
if __name__ == "__main__":
5355
parser = argparse.ArgumentParser()
54-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
55-
parser.add_argument('--deployment-id', help='Your game server deployment ID.', required=True)
56+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
57+
parser.add_argument(
58+
"--deployment-id", help="Your game server deployment ID.", required=True
59+
)
5660

5761
args = parser.parse_args()
5862

samples/snippets/create_realm.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,23 +35,22 @@ def create_realm(project_id, location, realm_id):
3535
request = realms.CreateRealmRequest(
3636
parent=f"projects/{project_id}/locations/{location}",
3737
realm_id=realm_id,
38-
realm=realms.Realm(
39-
description="My Realm",
40-
time_zone="US/Pacific",
41-
),
38+
realm=realms.Realm(description="My Realm", time_zone="US/Pacific",),
4239
)
4340

4441
operation = client.create_realm(request)
4542
print(f"Create realm operation: {operation.operation.name}")
4643
operation.result(timeout=120)
44+
45+
4746
# [END cloud_game_servers_realm_create]
4847

4948

5049
if __name__ == "__main__":
5150
parser = argparse.ArgumentParser()
52-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
53-
parser.add_argument('--location', help='Your realm location.', required=True)
54-
parser.add_argument('--realm-id', help='Your realm ID.', required=True)
51+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
52+
parser.add_argument("--location", help="Your realm location.", required=True)
53+
parser.add_argument("--realm-id", help="Your realm ID.", required=True)
5554

5655
args = parser.parse_args()
5756

samples/snippets/delete_cluster.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,19 @@ def delete_cluster(project_id, location, realm_id, cluster_id):
3939
operation = client.delete_game_server_cluster(request)
4040
print(f"Delete cluster operation: {operation.operation.name}")
4141
operation.result(timeout=120)
42+
43+
4244
# [END cloud_game_servers_cluster_delete]
4345

4446

4547
if __name__ == "__main__":
4648
parser = argparse.ArgumentParser()
47-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
48-
parser.add_argument('--location', help='Your realm location.', required=True)
49-
parser.add_argument('--realm-id', help='Your realm ID.', required=True)
50-
parser.add_argument('--cluster-id', help='Your game server cluster ID.', required=True)
49+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
50+
parser.add_argument("--location", help="Your realm location.", required=True)
51+
parser.add_argument("--realm-id", help="Your realm ID.", required=True)
52+
parser.add_argument(
53+
"--cluster-id", help="Your game server cluster ID.", required=True
54+
)
5155

5256
args = parser.parse_args()
5357

samples/snippets/delete_config.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,14 +42,20 @@ def delete_config(project_id, deployment_id, config_id):
4242
operation = client.delete_game_server_config(request)
4343
print(f"Delete config operation: {operation.operation.name}")
4444
operation.result(timeout=120)
45+
46+
4547
# [END cloud_game_servers_config_delete]
4648

4749

4850
if __name__ == "__main__":
4951
parser = argparse.ArgumentParser()
50-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
51-
parser.add_argument('--deployment-id', help='Your game server deployment ID.', required=True)
52-
parser.add_argument('--config-id', help='Your game server config ID.', required=True)
52+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
53+
parser.add_argument(
54+
"--deployment-id", help="Your game server deployment ID.", required=True
55+
)
56+
parser.add_argument(
57+
"--config-id", help="Your game server config ID.", required=True
58+
)
5359

5460
args = parser.parse_args()
5561

samples/snippets/delete_deployment.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,13 +42,17 @@ def delete_deployment(project_id, deployment_id):
4242
operation = client.delete_game_server_deployment(request)
4343
print(f"Delete deployment operation: {operation.operation.name}")
4444
operation.result(timeout=120)
45+
46+
4547
# [END cloud_game_servers_deployment_delete]
4648

4749

4850
if __name__ == "__main__":
4951
parser = argparse.ArgumentParser()
50-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
51-
parser.add_argument('--deployment-id', help='Your game server deployment ID.', required=True)
52+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
53+
parser.add_argument(
54+
"--deployment-id", help="Your game server deployment ID.", required=True
55+
)
5256

5357
args = parser.parse_args()
5458

samples/snippets/delete_realm.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,16 @@ def delete_realm(project_id, location, realm_id):
3939
operation = client.delete_realm(request)
4040
print(f"Delete realm operation: {operation.operation.name}")
4141
operation.result(timeout=120)
42+
43+
4244
# [END cloud_game_servers_realm_delete]
4345

4446

4547
if __name__ == "__main__":
4648
parser = argparse.ArgumentParser()
47-
parser.add_argument('--project-id', help='Your cloud project ID.', required=True)
48-
parser.add_argument('--location', help='Your realm location.', required=True)
49-
parser.add_argument('--realm-id', help='Your realm ID.', required=True)
49+
parser.add_argument("--project-id", help="Your cloud project ID.", required=True)
50+
parser.add_argument("--location", help="Your realm location.", required=True)
51+
parser.add_argument("--realm-id", help="Your realm ID.", required=True)
5052

5153
args = parser.parse_args()
5254

samples/snippets/deployment_and_config_test.py

Lines changed: 66 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -41,28 +41,32 @@
4141
REALM_LOCATION = "global"
4242

4343
# The format of realm ID. This is used in the unit tests and cleanup below.
44-
realm_id_format = 'test-realm-{}-{}'
44+
realm_id_format = "test-realm-{}-{}"
4545

4646
# The format of deployment ID. This is used in the unit tests and cleanup below.
47-
deployment_id_format = 'test-deployment-{}-{}'
47+
deployment_id_format = "test-deployment-{}-{}"
4848

4949

5050
@pytest.fixture(scope="session", autouse=True)
5151
def clean_up_old_deployments():
5252
all_deployments = list_deployments.list_deployments(PROJECT_ID)
5353
for deployment in all_deployments:
5454
deployment_name = deployment.name
55-
deployment_id = deployment_name[deployment_name.rfind('/') + 1: len(deployment_name)]
56-
if deployment_id.find('test-deployment-') == 0:
57-
time_str = deployment_id[deployment_id.rfind('-') + 1: len(deployment_id)]
55+
deployment_id = deployment_name[
56+
deployment_name.rfind("/") + 1 : len(deployment_name)
57+
]
58+
if deployment_id.find("test-deployment-") == 0:
59+
time_str = deployment_id[deployment_id.rfind("-") + 1 : len(deployment_id)]
5860
test_date = datetime.datetime.utcfromtimestamp(int(time_str))
5961
now_date = datetime.datetime.utcfromtimestamp(int(time.time()))
6062
difftime = now_date - test_date
6163

6264
# *NOTE* Restrict to deployments used in the tests older than 2 days
6365
# to prevent thrashing in the case of async tests
64-
if (difftime.days > 2):
65-
print(f"Cleaning up old deployment {deployment_id} and its configs, difftime: {difftime}")
66+
if difftime.days > 2:
67+
print(
68+
f"Cleaning up old deployment {deployment_id} and its configs, difftime: {difftime}"
69+
)
6670
clean_up_deployment_and_configs(deployment_id)
6771

6872

@@ -95,7 +99,9 @@ def test_deployment_with_config():
9599
print(f"Creating deployment {deployment_id} in project {PROJECT_ID}")
96100
create_deployment.create_deployment(PROJECT_ID, deployment_id)
97101

98-
print(f"Creating config {CONFIG_ID} in deployment {deployment_id} in project {PROJECT_ID}")
102+
print(
103+
f"Creating config {CONFIG_ID} in deployment {deployment_id} in project {PROJECT_ID}"
104+
)
99105
create_config.create_config(PROJECT_ID, deployment_id, CONFIG_ID)
100106

101107
yield deployment_id
@@ -112,18 +118,22 @@ def clean_up_deployment_and_configs(deployment_id):
112118
return
113119

114120
try:
115-
update_rollout_remove_default.update_rollout_remove_default(PROJECT_ID, deployment_id)
121+
update_rollout_remove_default.update_rollout_remove_default(
122+
PROJECT_ID, deployment_id
123+
)
116124
except exceptions.NotFound: # Ignore the non-existent deployment
117125
return
118126

119127
try:
120-
update_rollout_remove_override.update_rollout_remove_override(PROJECT_ID, deployment_id)
128+
update_rollout_remove_override.update_rollout_remove_override(
129+
PROJECT_ID, deployment_id
130+
)
121131
except exceptions.NotFound: # Ignore the non-existent deployment
122132
return
123133

124134
configs = list_configs.list_configs(PROJECT_ID, deployment_id)
125135
for config in configs:
126-
config_id = config.name.rsplit('/', 1)[-1]
136+
config_id = config.name.rsplit("/", 1)[-1]
127137
print(f"Deleting config {config_id} in deployment {deployment_id}")
128138
try:
129139
delete_config.delete_config(PROJECT_ID, deployment_id, config_id)
@@ -141,7 +151,9 @@ def clean_up_deployment_and_configs(deployment_id):
141151
def test_realm():
142152
realm_id = realm_id_format.format(uuid.uuid4().hex, int(time.time()))
143153

144-
print(f"Creating realm {realm_id} in location {REALM_LOCATION} in project {PROJECT_ID}")
154+
print(
155+
f"Creating realm {realm_id} in location {REALM_LOCATION} in project {PROJECT_ID}"
156+
)
145157
create_realm.create_realm(PROJECT_ID, REALM_LOCATION, realm_id)
146158

147159
yield realm_id
@@ -165,7 +177,10 @@ def test_create_deployment(test_deployment):
165177

166178
def test_get_deployment(test_deployment):
167179
deployment = get_deployment.get_deployment(PROJECT_ID, test_deployment)
168-
assert deployment.name == f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment}"
180+
assert (
181+
deployment.name
182+
== f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment}"
183+
)
169184

170185

171186
def test_list_deployments(test_deployment):
@@ -182,7 +197,10 @@ def test_list_deployments(test_deployment):
182197
def test_update_deployment(test_deployment):
183198
update_deployment.update_deployment(PROJECT_ID, test_deployment)
184199
deployment = get_deployment.get_deployment(PROJECT_ID, test_deployment)
185-
assert deployment.labels == {"label-key-1": "label-value-1", "label-key-2": "label-value-2"}
200+
assert deployment.labels == {
201+
"label-key-1": "label-value-1",
202+
"label-key-2": "label-value-2",
203+
}
186204

187205

188206
def test_delete_deployment(test_deployment):
@@ -193,42 +211,66 @@ def test_delete_deployment(test_deployment):
193211

194212
def test_get_rollout(test_deployment):
195213
rollout = get_rollout.get_rollout(PROJECT_ID, test_deployment)
196-
assert rollout.name == f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment}/rollout"
214+
assert (
215+
rollout.name
216+
== f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment}/rollout"
217+
)
197218

198219

199220
def test_update_rollout_default(test_deployment_with_config):
200-
update_rollout_default.update_rollout_default(PROJECT_ID, test_deployment_with_config, CONFIG_ID)
221+
update_rollout_default.update_rollout_default(
222+
PROJECT_ID, test_deployment_with_config, CONFIG_ID
223+
)
201224
rollout = get_rollout.get_rollout(PROJECT_ID, test_deployment_with_config)
202-
assert rollout.default_game_server_config == f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
225+
assert (
226+
rollout.default_game_server_config
227+
== f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
228+
)
203229

204230

205231
def test_update_rollout_override(test_realm, test_deployment_with_config):
206-
update_rollout_override.update_rollout_override(PROJECT_ID, test_deployment_with_config, CONFIG_ID, REALM_LOCATION, test_realm)
232+
update_rollout_override.update_rollout_override(
233+
PROJECT_ID, test_deployment_with_config, CONFIG_ID, REALM_LOCATION, test_realm
234+
)
207235
rollout = get_rollout.get_rollout(PROJECT_ID, test_deployment_with_config)
208236
assert len(rollout.game_server_config_overrides) == 1
209-
assert rollout.game_server_config_overrides[0].config_version == f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
210-
assert rollout.game_server_config_overrides[0].realms_selector.realms == [f"projects/{PROJECT_ID}/locations/{REALM_LOCATION}/realms/{test_realm}"]
237+
assert (
238+
rollout.game_server_config_overrides[0].config_version
239+
== f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
240+
)
241+
assert rollout.game_server_config_overrides[0].realms_selector.realms == [
242+
f"projects/{PROJECT_ID}/locations/{REALM_LOCATION}/realms/{test_realm}"
243+
]
211244

212245

213246
def test_update_rollout_remove_default(test_deployment):
214-
update_rollout_remove_default.update_rollout_remove_default(PROJECT_ID, test_deployment)
247+
update_rollout_remove_default.update_rollout_remove_default(
248+
PROJECT_ID, test_deployment
249+
)
215250
rollout = get_rollout.get_rollout(PROJECT_ID, test_deployment)
216251
assert rollout.default_game_server_config == ""
217252

218253

219254
def test_update_rollout_remove_override(test_deployment):
220-
update_rollout_remove_override.update_rollout_remove_override(PROJECT_ID, test_deployment)
255+
update_rollout_remove_override.update_rollout_remove_override(
256+
PROJECT_ID, test_deployment
257+
)
221258
rollout = get_rollout.get_rollout(PROJECT_ID, test_deployment)
222259
assert len(rollout.game_server_config_overrides) == 0
223260

224261

225262
def test_create_conifg(test_deployment_with_config):
226-
print(f"Created config {CONFIG_ID} in deployment {test_deployment_with_config} in project {PROJECT_ID}")
263+
print(
264+
f"Created config {CONFIG_ID} in deployment {test_deployment_with_config} in project {PROJECT_ID}"
265+
)
227266

228267

229268
def test_get_config(test_deployment_with_config):
230269
config = get_config.get_config(PROJECT_ID, test_deployment_with_config, CONFIG_ID)
231-
assert config.name == f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
270+
assert (
271+
config.name
272+
== f"projects/{PROJECT_ID}/locations/global/gameServerDeployments/{test_deployment_with_config}/configs/{CONFIG_ID}"
273+
)
232274

233275

234276
def test_list_configs(test_deployment_with_config):

0 commit comments

Comments
 (0)