From 47d94daffc30f51b28541094c0cad444214fbc37 Mon Sep 17 00:00:00 2001 From: agostina Date: Mon, 7 Aug 2023 12:25:00 +0200 Subject: [PATCH] LITE-28334: Saving the right marketplace id, instead of the id of the MarketplaceConfiguration --- connect_ext_ppr/service.py | 2 +- tests/api/test_deployment_requests.py | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/connect_ext_ppr/service.py b/connect_ext_ppr/service.py index 435b142..16400e4 100644 --- a/connect_ext_ppr/service.py +++ b/connect_ext_ppr/service.py @@ -316,7 +316,7 @@ def add_new_deployment_request(db, dr_data, deployment, account_id, logger): marketplaces = [m.id for m in dr_data.marketplaces.choices] if dr_data.marketplaces.all: - marketplaces = [m.id for m in deployment.marketplaces] + marketplaces = [m.marketplace for m in deployment.marketplaces] for m_id in marketplaces: mc = MarketplaceConfiguration( diff --git a/tests/api/test_deployment_requests.py b/tests/api/test_deployment_requests.py index 803f3fb..cce00e5 100644 --- a/tests/api/test_deployment_requests.py +++ b/tests/api/test_deployment_requests.py @@ -450,7 +450,10 @@ def test_create_deployment_request_with_all_marketplaces( assert dbsession.query(MarketplaceConfiguration).filter_by( deployment_request=deployment_request.id, - ).filter(MarketplaceConfiguration.deployment_id.is_(null())).count() == 2 + ).filter( + MarketplaceConfiguration.deployment_id.is_(null()), + MarketplaceConfiguration.marketplace.in_(['MP-123', 'MP-124']), + ).count() == 2 def test_create_deployment_request_invalid_deployment(