From a8ebb74702f07acead84edcdaf083d78a84d00a3 Mon Sep 17 00:00:00 2001 From: Ittay Stern Date: Tue, 29 Oct 2019 17:54:54 +0200 Subject: Instantiation-Status: default on unexpected statuses Issue-ID: VID-692 Change-Id: I1a81a71d32aa36fb1f0dc3b349733db11d4e579d Signed-off-by: Ittay Stern --- .../instantiationStatus.component.service.spec.ts | 45 +++++++++++----------- .../instantiationStatus.component.service.ts | 9 ++++- 2 files changed, 29 insertions(+), 25 deletions(-) (limited to 'vid-webpack-master/src/app') diff --git a/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.spec.ts b/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.spec.ts index aeaa05780..d115984f6 100644 --- a/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.spec.ts +++ b/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.spec.ts @@ -6,8 +6,9 @@ import { PAUSE, PENDING, ServiceStatus, - STOPED, + STOPPED, SUCCESS_CIRCLE, + UNKNOWN, X_O } from './instantiationStatus.component.service'; import {ServiceInfoModel} from '../shared/server/serviceInfo/serviceInfo.model'; @@ -129,28 +130,23 @@ describe('Instantiation Status Service', () => { expect (suffix + serviceModelUrl).toEqual(result); }); - test('getStatusTooltip should return status popover', () => { - let result : ServiceStatus = service.getStatus('pending'); - expect(result.tooltip).toEqual('Pending: The action required will be sent as soon as possible.'); - - result = service.getStatus('IN_PROGRESS'); - expect(result.tooltip).toEqual('In-progress: the service is in process of the action required.'); - - result = service.getStatus('PAUSED'); - expect(result.tooltip).toEqual('Paused: Service has paused and waiting for your action.\n Select actions from the menu to the right.'); - - result = service.getStatus('FAILED'); - expect(result.tooltip).toEqual('Failed: All planned actions have failed.'); - - result = service.getStatus('COMPLETED'); - expect(result.tooltip).toEqual('Completed successfully: Service is successfully instantiated, updated or deleted.'); - - result = service.getStatus('STOPPED'); - expect(result.tooltip).toEqual('Stopped: Due to previous failure, will not be instantiated.'); + for (let [status, tooltip] of Object.entries({ + 'pending': 'Pending: The action required will be sent as soon as possible.', + 'IN_PROGRESS': 'In-progress: the service is in process of the action required.', + 'PAUSED': 'Paused: Service has paused and waiting for your action.\n Select actions from the menu to the right.', + 'FAILED': 'Failed: All planned actions have failed.', + 'COMPLETED': 'Completed successfully: Service is successfully instantiated, updated or deleted.', + 'STOPPED': 'Stopped: Due to previous failure, will not be instantiated.', + 'StOpPeD': 'Stopped: Due to previous failure, will not be instantiated.', + 'COMPLETED_WITH_ERRORS': 'Completed with errors: some of the planned actions where successfully committed while other have not.\n Open the service to check it out.', + 'UNEXPECTED_STATUS': 'Unexpected status: "UNEXPECTED_RANDOM_STATUS"', + })) { + + test(`getStatusTooltip should return status popover: status=${status}`, () => { + expect(service.getStatus(status).tooltip).toEqual(tooltip); + }); - result = service.getStatus('COMPLETED_WITH_ERRORS'); - expect(result.tooltip).toEqual('Completed with errors: some of the planned actions where successfully committed while other have not.\n Open the service to check it out.'); - }); + } test('getStatusTooltip should return correct icon per job status', () => { let result : ServiceStatus = service.getStatus('pending'); @@ -169,10 +165,13 @@ describe('Instantiation Status Service', () => { expect(result.iconClassName).toEqual(SUCCESS_CIRCLE); result = service.getStatus('STOPPED'); - expect(result.iconClassName).toEqual(STOPED); + expect(result.iconClassName).toEqual(STOPPED); result = service.getStatus('COMPLETED_WITH_ERRORS'); expect(result.iconClassName).toEqual(COMPLETED_WITH_ERRORS); + + result = service.getStatus('UNEXPECTED_RANDOM_STATUS'); + expect(result.iconClassName).toEqual(UNKNOWN); }); function generateServiceInfoData(){ diff --git a/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.ts b/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.ts index 0e4451ca8..6a85d9da1 100644 --- a/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.ts +++ b/vid-webpack-master/src/app/instantiationStatus/instantiationStatus.component.service.ts @@ -12,13 +12,15 @@ import {updateDrawingBoardStatus} from "../shared/storeUtil/utils/global/global. import {Router, UrlTree} from "@angular/router"; import {of} from "rxjs"; import {MsoService} from "../shared/services/msoService/mso.service"; + export let PENDING : string = "pending"; export let INPROGRESS : string = "in_progress"; export let PAUSE : string = "pause"; export let X_O : string = "x-circle-o"; export let SUCCESS_CIRCLE : string = "success-circle-o"; -export let STOPED : string = "stop"; +export let STOPPED : string = "stop"; export let COMPLETED_WITH_ERRORS : string = "success_with_warning"; +export let UNKNOWN : string = "question-mark-circle-o"; @Injectable() @@ -148,9 +150,12 @@ export class InstantiationStatusComponentService { case 'COMPLETED' : return new ServiceStatus(SUCCESS_CIRCLE, 'success', 'Completed successfully: Service is successfully instantiated, updated or deleted.'); case 'STOPPED' : - return new ServiceStatus(STOPED, 'error', 'Stopped: Due to previous failure, will not be instantiated.'); + return new ServiceStatus(STOPPED, 'error', 'Stopped: Due to previous failure, will not be instantiated.'); case 'COMPLETED_WITH_ERRORS' : return new ServiceStatus(COMPLETED_WITH_ERRORS, 'success', 'Completed with errors: some of the planned actions where successfully committed while other have not.\n Open the service to check it out.'); + + default: + return new ServiceStatus(UNKNOWN, 'primary', `Unexpected status: "${status}"`); } } -- cgit 1.2.3-korg