Skip to content

Commit 1eaba29

Browse files
renaming
1 parent 4826fa8 commit 1eaba29

File tree

8 files changed

+61
-61
lines changed

8 files changed

+61
-61
lines changed

packages/service-library/src/servicelib/rabbitmq/rpc_interfaces/resource_usage_tracker/licensed_items_checkouts.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,13 @@ async def get_licensed_item_checkout(
3939
rabbitmq_rpc_client: RabbitMQRPCClient,
4040
*,
4141
product_name: ProductName,
42-
licensed_item_usage_id: LicensedItemCheckoutID,
42+
licensed_item_checkout_id: LicensedItemCheckoutID,
4343
) -> LicensedItemCheckoutGet:
4444
result = await rabbitmq_rpc_client.request(
4545
RESOURCE_USAGE_TRACKER_RPC_NAMESPACE,
4646
_RPC_METHOD_NAME_ADAPTER.validate_python("get_licensed_item_checkout"),
4747
product_name=product_name,
48-
licensed_item_usage_id=licensed_item_usage_id,
48+
licensed_item_checkout_id=licensed_item_checkout_id,
4949
timeout_s=_DEFAULT_TIMEOUT_S,
5050
)
5151
assert isinstance(result, LicensedItemCheckoutGet) # nosec

services/resource-usage-tracker/src/simcore_service_resource_usage_tracker/api/rpc/_licensed_items_checkouts.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,16 @@
2222

2323

2424
@router.expose(reraise_if_error_type=())
25-
async def get_licensed_item_usage(
25+
async def get_licensed_item_checkout(
2626
app: FastAPI,
2727
*,
2828
product_name: ProductName,
29-
licensed_item_usage_id: LicensedItemCheckoutID,
29+
licensed_item_checkout_id: LicensedItemCheckoutID,
3030
) -> LicensedItemCheckoutGet:
3131
return await licensed_items_checkouts.get_licensed_item_checkout(
3232
db_engine=app.state.engine,
3333
product_name=product_name,
34-
licensed_item_checkout_id=licensed_item_usage_id,
34+
licensed_item_checkout_id=licensed_item_checkout_id,
3535
)
3636

3737

services/resource-usage-tracker/src/simcore_service_resource_usage_tracker/exceptions/errors.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,5 +80,5 @@ class LicensedItemPurchaseNotFoundError(RutNotFoundError):
8080
licensed_item_purchase_id: LicensedItemPurchaseID
8181

8282

83-
class LicensedItemUsageNotFoundError(RutNotFoundError):
84-
licensed_item_usage_id: LicensedItemCheckoutID
83+
class LicensedItemCheckoutNotFoundError(RutNotFoundError):
84+
licensed_item_checkout_id: LicensedItemCheckoutID

services/resource-usage-tracker/src/simcore_service_resource_usage_tracker/services/licensed_items_checkouts.py

Lines changed: 34 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -52,17 +52,17 @@ async def list_licensed_items_checkouts(
5252
total=total,
5353
items=[
5454
LicensedItemCheckoutGet(
55-
licensed_item_checkout_id=licensed_item_usage_db.licensed_item_checkout_id,
56-
licensed_item_id=licensed_item_usage_db.licensed_item_id,
57-
wallet_id=licensed_item_usage_db.wallet_id,
58-
user_id=licensed_item_usage_db.user_id,
59-
product_name=licensed_item_usage_db.product_name,
60-
service_run_id=licensed_item_usage_db.service_run_id,
61-
started_at=licensed_item_usage_db.started_at,
62-
stopped_at=licensed_item_usage_db.stopped_at,
63-
num_of_seats=licensed_item_usage_db.num_of_seats,
55+
licensed_item_checkout_id=licensed_item_checkout_db.licensed_item_checkout_id,
56+
licensed_item_id=licensed_item_checkout_db.licensed_item_id,
57+
wallet_id=licensed_item_checkout_db.wallet_id,
58+
user_id=licensed_item_checkout_db.user_id,
59+
product_name=licensed_item_checkout_db.product_name,
60+
service_run_id=licensed_item_checkout_db.service_run_id,
61+
started_at=licensed_item_checkout_db.started_at,
62+
stopped_at=licensed_item_checkout_db.stopped_at,
63+
num_of_seats=licensed_item_checkout_db.num_of_seats,
6464
)
65-
for licensed_item_usage_db in licensed_items_checkouts_list_db
65+
for licensed_item_checkout_db in licensed_items_checkouts_list_db
6666
],
6767
)
6868

@@ -73,24 +73,24 @@ async def get_licensed_item_checkout(
7373
product_name: ProductName,
7474
licensed_item_checkout_id: LicensedItemCheckoutID,
7575
) -> LicensedItemCheckoutGet:
76-
licensed_item_usage_db: LicensedItemCheckoutDB = (
76+
licensed_item_checkout_db: LicensedItemCheckoutDB = (
7777
await licensed_items_checkouts_db.get(
7878
db_engine,
7979
product_name=product_name,
80-
licensed_item_usage_id=licensed_item_checkout_id,
80+
licensed_item_checkout_id=licensed_item_checkout_id,
8181
)
8282
)
8383

8484
return LicensedItemCheckoutGet(
85-
licensed_item_checkout_id=licensed_item_usage_db.licensed_item_checkout_id,
86-
licensed_item_id=licensed_item_usage_db.licensed_item_id,
87-
wallet_id=licensed_item_usage_db.wallet_id,
88-
user_id=licensed_item_usage_db.user_id,
89-
product_name=licensed_item_usage_db.product_name,
90-
service_run_id=licensed_item_usage_db.service_run_id,
91-
started_at=licensed_item_usage_db.started_at,
92-
stopped_at=licensed_item_usage_db.stopped_at,
93-
num_of_seats=licensed_item_usage_db.num_of_seats,
85+
licensed_item_checkout_id=licensed_item_checkout_db.licensed_item_checkout_id,
86+
licensed_item_id=licensed_item_checkout_db.licensed_item_id,
87+
wallet_id=licensed_item_checkout_db.wallet_id,
88+
user_id=licensed_item_checkout_db.user_id,
89+
product_name=licensed_item_checkout_db.product_name,
90+
service_run_id=licensed_item_checkout_db.service_run_id,
91+
started_at=licensed_item_checkout_db.started_at,
92+
stopped_at=licensed_item_checkout_db.stopped_at,
93+
num_of_seats=licensed_item_checkout_db.num_of_seats,
9494
)
9595

9696

@@ -139,7 +139,7 @@ async def checkout_licensed_item(
139139
):
140140
raise ValueError("This should not happen")
141141

142-
_create_item_usage = CreateLicensedItemCheckoutDB(
142+
_create_item_checkout = CreateLicensedItemCheckoutDB(
143143
licensed_item_id=licensed_item_id,
144144
wallet_id=wallet_id,
145145
user_id=user_id,
@@ -150,7 +150,7 @@ async def checkout_licensed_item(
150150
num_of_seats=num_of_seats,
151151
)
152152
license_item_checkout_db = await licensed_items_checkouts_db.create(
153-
db_engine, data=_create_item_usage
153+
db_engine, data=_create_item_checkout
154154
)
155155

156156
# Return checkout ID
@@ -166,23 +166,23 @@ async def release_licensed_item(
166166
product_name: ProductName,
167167
) -> LicensedItemCheckoutGet:
168168

169-
licensed_item_usage_db: LicensedItemCheckoutDB = (
169+
licensed_item_checkout_db: LicensedItemCheckoutDB = (
170170
await licensed_items_checkouts_db.update(
171171
db_engine,
172-
licensed_item_usage_id=checkout_id,
172+
licensed_item_checkout_id=checkout_id,
173173
product_name=product_name,
174174
stopped_at=datetime.now(tz=UTC),
175175
)
176176
)
177177

178178
return LicensedItemCheckoutGet(
179-
licensed_item_checkout_id=licensed_item_usage_db.licensed_item_checkout_id,
180-
licensed_item_id=licensed_item_usage_db.licensed_item_id,
181-
wallet_id=licensed_item_usage_db.wallet_id,
182-
user_id=licensed_item_usage_db.user_id,
183-
product_name=licensed_item_usage_db.product_name,
184-
service_run_id=licensed_item_usage_db.service_run_id,
185-
started_at=licensed_item_usage_db.started_at,
186-
stopped_at=licensed_item_usage_db.stopped_at,
187-
num_of_seats=licensed_item_usage_db.num_of_seats,
179+
licensed_item_checkout_id=licensed_item_checkout_db.licensed_item_checkout_id,
180+
licensed_item_id=licensed_item_checkout_db.licensed_item_id,
181+
wallet_id=licensed_item_checkout_db.wallet_id,
182+
user_id=licensed_item_checkout_db.user_id,
183+
product_name=licensed_item_checkout_db.product_name,
184+
service_run_id=licensed_item_checkout_db.service_run_id,
185+
started_at=licensed_item_checkout_db.started_at,
186+
stopped_at=licensed_item_checkout_db.stopped_at,
187+
num_of_seats=licensed_item_checkout_db.num_of_seats,
188188
)

services/resource-usage-tracker/src/simcore_service_resource_usage_tracker/services/modules/db/licensed_items_checkouts_db.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44
import sqlalchemy as sa
55
from models_library.licensed_items import LicensedItemID
66
from models_library.products import ProductName
7-
from models_library.resource_tracker_licensed_items_purchases import (
8-
LicensedItemPurchaseID,
9-
)
107
from models_library.rest_ordering import OrderBy, OrderDirection
118
from models_library.wallets import WalletID
129
from pydantic import NonNegativeInt
@@ -17,9 +14,12 @@
1714
pass_or_acquire_connection,
1815
transaction_context,
1916
)
17+
from simcore_service_resource_usage_tracker.services.licensed_items_checkouts import (
18+
LicensedItemCheckoutID,
19+
)
2020
from sqlalchemy.ext.asyncio import AsyncConnection, AsyncEngine
2121

22-
from ....exceptions.errors import LicensedItemUsageNotFoundError
22+
from ....exceptions.errors import LicensedItemCheckoutNotFoundError
2323
from ....models.licensed_items_checkouts import (
2424
CreateLicensedItemCheckoutDB,
2525
LicensedItemCheckoutDB,
@@ -125,16 +125,16 @@ async def get(
125125
engine: AsyncEngine,
126126
connection: AsyncConnection | None = None,
127127
*,
128-
licensed_item_usage_id: LicensedItemPurchaseID,
128+
licensed_item_checkout_id: LicensedItemCheckoutID,
129129
product_name: ProductName,
130130
) -> LicensedItemCheckoutDB:
131131
base_query = (
132132
sa.select(*_SELECTION_ARGS)
133133
.select_from(resource_tracker_licensed_items_checkouts)
134134
.where(
135135
(
136-
resource_tracker_licensed_items_checkouts.c.licensed_item_usage_id
137-
== licensed_item_usage_id
136+
resource_tracker_licensed_items_checkouts.c.licensed_item_checkout_id
137+
== licensed_item_checkout_id
138138
)
139139
& (resource_tracker_licensed_items_checkouts.c.product_name == product_name)
140140
)
@@ -144,8 +144,8 @@ async def get(
144144
result = await conn.stream(base_query)
145145
row = await result.first()
146146
if row is None:
147-
raise LicensedItemUsageNotFoundError(
148-
licensed_item_usage_id=licensed_item_usage_id
147+
raise LicensedItemCheckoutNotFoundError(
148+
licensed_item_checkout_id=licensed_item_checkout_id
149149
)
150150
return LicensedItemCheckoutDB.model_validate(row)
151151

@@ -154,7 +154,7 @@ async def update(
154154
engine: AsyncEngine,
155155
connection: AsyncConnection | None = None,
156156
*,
157-
licensed_item_usage_id: LicensedItemPurchaseID,
157+
licensed_item_checkout_id: LicensedItemCheckoutID,
158158
product_name: ProductName,
159159
stopped_at: datetime,
160160
) -> LicensedItemCheckoutDB:
@@ -166,8 +166,8 @@ async def update(
166166
)
167167
.where(
168168
(
169-
resource_tracker_licensed_items_checkouts.c.licensed_item_usage_id
170-
== licensed_item_usage_id
169+
resource_tracker_licensed_items_checkouts.c.licensed_item_checkout_id
170+
== licensed_item_checkout_id
171171
)
172172
& (resource_tracker_licensed_items_checkouts.c.product_name == product_name)
173173
& (resource_tracker_licensed_items_checkouts.c.stopped_at.is_(None))
@@ -179,8 +179,8 @@ async def update(
179179
result = await conn.execute(update_stmt)
180180
row = result.first()
181181
if row is None:
182-
raise LicensedItemUsageNotFoundError(
183-
licensed_item_usage_id=licensed_item_usage_id
182+
raise LicensedItemCheckoutNotFoundError(
183+
licensed_item_checkout_id=licensed_item_checkout_id
184184
)
185185
return LicensedItemCheckoutDB.model_validate(row)
186186

services/resource-usage-tracker/tests/unit/with_dbs/test_api_licensed_items_checkouts.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ async def test_rpc_licensed_items_checkouts_workflow(
125125
output = await licensed_items_checkouts.get_licensed_item_checkout(
126126
rpc_client,
127127
product_name="osparc",
128-
licensed_item_usage_id=output.items[0].licensed_item_checkout_id,
128+
licensed_item_checkout_id=output.items[0].licensed_item_checkout_id,
129129
)
130130
assert isinstance(output, LicensedItemCheckoutGet)
131131

services/web/server/src/simcore_service_webserver/licenses/_licensed_checkouts_api.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ async def release_licensed_item_for_wallet(
6666
rpc_client = get_rabbitmq_rpc_client(app)
6767

6868
checkout_item = await licensed_items_checkouts.get_licensed_item_checkout(
69-
rpc_client, product_name=product_name, licensed_item_usage_id=checkout_id
69+
rpc_client, product_name=product_name, licensed_item_checkout_id=checkout_id
7070
)
7171

7272
# Check whether user has access to the wallet
@@ -86,7 +86,7 @@ async def release_licensed_item_for_wallet(
8686
)
8787

8888
return webserver_licensed_items_checkouts.LicensedItemCheckoutGet(
89-
licensed_item_checkout_id=licensed_item_get.licensed_item_usage_id,
89+
licensed_item_checkout_id=licensed_item_get.licensed_item_checkout_id,
9090
licensed_item_id=licensed_item_get.licensed_item_id,
9191
wallet_id=licensed_item_get.wallet_id,
9292
user_id=licensed_item_get.user_id,

services/web/server/tests/unit/with_dbs/04/licenses/test_licenses_rpc.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,9 +101,9 @@ def mock_checkout_licensed_item(mocker: MockerFixture) -> tuple:
101101

102102

103103
@pytest.fixture
104-
def mock_get_licensed_item_usage(mocker: MockerFixture) -> tuple:
104+
def mock_get_licensed_item_checkout(mocker: MockerFixture) -> tuple:
105105
return mocker.patch(
106-
"simcore_service_webserver.licenses._licensed_checkouts_api.licensed_items_checkouts.get_licensed_item_usage",
106+
"simcore_service_webserver.licenses._licensed_checkouts_api.licensed_items_checkouts.get_licensed_item_checkout",
107107
spec=True,
108108
return_value=_LICENSED_ITEM_USAGE_GET,
109109
)
@@ -130,7 +130,7 @@ async def test_license_checkout_workflow(
130130
mock_get_wallet_by_user: MockerFixture,
131131
mock_checkout_licensed_item: MockerFixture,
132132
mock_release_licensed_item: MockerFixture,
133-
mock_get_licensed_item_usage: MockerFixture,
133+
mock_get_licensed_item_checkout: MockerFixture,
134134
):
135135
assert client.app
136136

0 commit comments

Comments
 (0)