Skip to content

Commit 62ca995

Browse files
authored
chore(code) remove Dict (#275)
1 parent 257fe66 commit 62ca995

File tree

4 files changed

+17
-17
lines changed

4 files changed

+17
-17
lines changed

cloudfoundry_client/operations/push/push.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import shutil
66
import tempfile
77
import time
8-
from typing import Tuple, Dict
8+
from typing import Tuple
99

1010
from cloudfoundry_client.client import CloudFoundryClient
1111
from cloudfoundry_client.operations.push.cf_ignore import CfIgnore
@@ -211,7 +211,7 @@ def _resolve_new_tcp_route(self, space: Entity, domain: Entity, port: int) -> En
211211
return existing_route
212212

213213
@staticmethod
214-
def _split_route(requested_route: Dict[str, str]) -> Tuple[str, int, str]:
214+
def _split_route(requested_route: dict[str, str]) -> Tuple[str, int, str]:
215215
route_splitted = PushOperation.SPLIT_ROUTE_PATTERN.match(requested_route["route"])
216216
if route_splitted is None:
217217
raise AssertionError("Invalid route: %s" % requested_route["route"])
@@ -222,7 +222,7 @@ def _split_route(requested_route: Dict[str, str]) -> Tuple[str, int, str]:
222222

223223
@staticmethod
224224
def _resolve_domain(
225-
route: str, private_domains: Dict[str, Entity], shared_domains: Dict[str, Entity]
225+
route: str, private_domains: dict[str, Entity], shared_domains: dict[str, Entity]
226226
) -> Tuple[str, str, Entity]:
227227
for domains in [private_domains, shared_domains]:
228228
if route in domains:

cloudfoundry_client/v2/apps.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import os
44
from http import HTTPStatus
55
from time import sleep
6-
from typing import Dict, TYPE_CHECKING
6+
from typing import TYPE_CHECKING
77

88
from cloudfoundry_client.doppler.client import EnvelopeStream
99
from cloudfoundry_client.errors import InvalidStatusCode
@@ -17,7 +17,7 @@
1717

1818

1919
class Application(Entity):
20-
def instances(self) -> Dict[str, JsonObject]:
20+
def instances(self) -> dict[str, JsonObject]:
2121
return self.client.v2.apps.get_instances(self["metadata"]["guid"])
2222

2323
def start(self) -> "Application":
@@ -29,10 +29,10 @@ def stop(self) -> "Application":
2929
def restart_instance(self, instance_id: int):
3030
return self.client.v2.apps.restart_instance(self["metadata"]["guid"], instance_id)
3131

32-
def stats(self) -> Dict[str, JsonObject]:
32+
def stats(self) -> dict[str, JsonObject]:
3333
return self.client.v2.apps.get_stats(self["metadata"]["guid"])
3434

35-
def env(self) -> Dict[str, JsonObject]:
35+
def env(self) -> dict[str, JsonObject]:
3636
return self.client.v2.apps.get_env(self["metadata"]["guid"])
3737

3838
def summary(self) -> JsonObject:
@@ -80,13 +80,13 @@ def __init__(self, target_endpoint: str, client: "CloudFoundryClient"):
8080
target_endpoint, client, "/v2/apps", lambda pairs: Application(target_endpoint, client, pairs)
8181
)
8282

83-
def get_stats(self, application_guid: str) -> Dict[str, JsonObject]:
83+
def get_stats(self, application_guid: str) -> dict[str, JsonObject]:
8484
return self._get("%s/%s/stats" % (self.entity_uri, application_guid), JsonObject)
8585

86-
def get_instances(self, application_guid: str) -> Dict[str, JsonObject]:
86+
def get_instances(self, application_guid: str) -> dict[str, JsonObject]:
8787
return self._get("%s/%s/instances" % (self.entity_uri, application_guid), JsonObject)
8888

89-
def get_env(self, application_guid: str) -> Dict[str, JsonObject]:
89+
def get_env(self, application_guid: str) -> dict[str, JsonObject]:
9090
return self._get("%s/%s/env" % (self.entity_uri, application_guid), JsonObject)
9191

9292
def get_summary(self, application_guid: str) -> JsonObject:
@@ -195,7 +195,7 @@ def _wait_for_instances_in_state(
195195
sleep(check_time)
196196
sum_waiting += check_time
197197

198-
def _safe_get_instances(self, application_guid: str) -> Dict[str, JsonObject]:
198+
def _safe_get_instances(self, application_guid: str) -> dict[str, JsonObject]:
199199
try:
200200
return self.get_instances(application_guid)
201201
except InvalidStatusCode as ex:

cloudfoundry_client/v2/service_instances.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from typing import Dict, TYPE_CHECKING
1+
from typing import TYPE_CHECKING
22

33
from cloudfoundry_client.v2.entities import EntityManager, Entity
44

@@ -44,7 +44,7 @@ def update(
4444
params = None if not accepts_incomplete else dict(accepts_incomplete="true")
4545
return super(ServiceInstanceManager, self)._update(instance_guid, request, params=params)
4646

47-
def list_permissions(self, instance_guid: str) -> Dict[str, bool]:
47+
def list_permissions(self, instance_guid: str) -> dict[str, bool]:
4848
return super(ServiceInstanceManager, self)._get("%s/%s/permissions" % (self.entity_uri, instance_guid), dict)
4949

5050
def remove(self, instance_guid: str, accepts_incomplete: bool | None = False, purge: bool | None = False):

cloudfoundry_client/v3/service_plans.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from typing import Dict, TYPE_CHECKING
1+
from typing import TYPE_CHECKING
22

33
from cloudfoundry_client.v3.entities import EntityManager, Entity
44

@@ -27,12 +27,12 @@ def update(
2727
def remove(self, guid: str):
2828
super(ServicePlanManager, self)._remove(guid)
2929

30-
def get_visibility(self, service_plan_guid: str) -> Dict:
30+
def get_visibility(self, service_plan_guid: str) -> dict:
3131
return super(ServicePlanManager, self)._get(f"{self.target_endpoint}{self.entity_uri}/{service_plan_guid}/visibility")
3232

3333
# Updates a service plan visibility. It behaves similar to the POST service plan visibility endpoint but
3434
# this endpoint will REPLACE the existing list of organizations when the service plan is organization visible.
35-
def update_visibility(self, service_plan_guid: str, type: str, organizations: list[dict] | None = None) -> Dict:
35+
def update_visibility(self, service_plan_guid: str, type: str, organizations: list[dict] | None = None) -> dict:
3636
payload = {"type": type}
3737
if organizations:
3838
payload["organizations"] = organizations
@@ -42,7 +42,7 @@ def update_visibility(self, service_plan_guid: str, type: str, organizations: li
4242

4343
# Applies a service plan visibility. It behaves similar to the PATCH service plan visibility endpoint but
4444
# this endpoint will APPEND to the existing list of organizations when the service plan is organization visible.
45-
def apply_visibility_to_extra_orgs(self, service_plan_guid: str, organizations: list[dict]) -> Dict:
45+
def apply_visibility_to_extra_orgs(self, service_plan_guid: str, organizations: list[dict]) -> dict:
4646
payload = {"type": "organization", "organizations": organizations}
4747
return super(ServicePlanManager, self)._post(
4848
url=f"{self.target_endpoint}{self.entity_uri}/{service_plan_guid}/visibility", data=payload, files=None

0 commit comments

Comments
 (0)