From 0b83fc95f4d93e5c4cba24522f9ebff5187b0d57 Mon Sep 17 00:00:00 2001 From: "Janet L. Goldstein" <jgoldste@nrao.edu> Date: Mon, 10 Jan 2022 16:41:51 -0700 Subject: [PATCH] WS-858: lay groundwork for fixing `create_follow_on_capability_request`. CapabilityRequest.current_execution removed to match CapabilityRequest ORM. Tests pass. --- .../request-operations/request-operations.component.html | 2 +- apps/web/src/app/workspaces/model/capability-request.ts | 1 - .../app/workspaces/services/capability-launcher.service.ts | 3 ++- services/capability/capability/routes.py | 5 ++++- services/capability/test/test_capability_version_views.py | 1 - 5 files changed, 7 insertions(+), 5 deletions(-) diff --git a/apps/web/src/app/workspaces/components/capability-request/components/request-operations/request-operations.component.html b/apps/web/src/app/workspaces/components/capability-request/components/request-operations/request-operations.component.html index d5eb25972..8d7bc76a5 100644 --- a/apps/web/src/app/workspaces/components/capability-request/components/request-operations/request-operations.component.html +++ b/apps/web/src/app/workspaces/components/capability-request/components/request-operations/request-operations.component.html @@ -21,7 +21,7 @@ class="btn btn-warning" (click)=" capabilityLauncherService - .createImageRequestFromPreviousCal('std_cms_imaging', capabilityRequest.id) + .createImageRequestFromPreviousCal('std_cms_imaging', capabilityRequest.id, selectedVersion.version_number) .subscribe(followonRequestObserver) " > diff --git a/apps/web/src/app/workspaces/model/capability-request.ts b/apps/web/src/app/workspaces/model/capability-request.ts index 2488048d1..1528d27f2 100644 --- a/apps/web/src/app/workspaces/model/capability-request.ts +++ b/apps/web/src/app/workspaces/model/capability-request.ts @@ -27,7 +27,6 @@ export interface CapabilityRequest { capability: Capability; state: string; versions: Array<CapabilityVersion>; - current_execution: CapabilityExecution; ingested: boolean; created_at: string; updated_at: string; diff --git a/apps/web/src/app/workspaces/services/capability-launcher.service.ts b/apps/web/src/app/workspaces/services/capability-launcher.service.ts index 63abf5a55..bbf79060b 100644 --- a/apps/web/src/app/workspaces/services/capability-launcher.service.ts +++ b/apps/web/src/app/workspaces/services/capability-launcher.service.ts @@ -49,8 +49,9 @@ export class CapabilityLauncherService { createImageRequestFromPreviousCal( followonType: string, requestId: string, + versionNumber: number, ): Observable<CapabilityRequest> { - const url = this.endpoint + "request/" + requestId + "/followon/" + followonType; + const url = this.endpoint + "request/" + requestId + + "/version/" + versionNumber + "/followon/" + followonType; return this.httpClient.post<CapabilityRequest>(url, null); } diff --git a/services/capability/capability/routes.py b/services/capability/capability/routes.py index ba4d5f055..231dfda5b 100644 --- a/services/capability/capability/routes.py +++ b/services/capability/capability/routes.py @@ -93,9 +93,12 @@ def capability_request_routes(config: Configurator): pattern="capability/{capability_name}/request/calibration-from-ingestion-details", request_method="POST", ) + + version_url = request_url + "/version/{version}" + followon_pattern = f"{version_url}" + "/followon/{followon_type}" config.add_route( name="create_follow_on_capability_request", - pattern=f"{request_url}" + "/followon/{followon_type}", + pattern=followon_pattern, request_method="POST", ) config.add_route( diff --git a/services/capability/test/test_capability_version_views.py b/services/capability/test/test_capability_version_views.py index 4969dc36c..547dc2e8f 100644 --- a/services/capability/test/test_capability_version_views.py +++ b/services/capability/test/test_capability_version_views.py @@ -27,7 +27,6 @@ from pyramid.testing import DummyRequest # pylint: disable=C0415, E0401 -# @pytest.mark.skip("Broken by addition of current_execution to version json") def test_view_latest_version(request_null_capability: DummyRequest): from capability.views.capability_version import view_latest_version -- GitLab