View Issue Details

IDProjectCategoryView StatusLast Update
0003445Compliance Test Tool (CTT) Unified Architecture1 - Script Issuepublic2019-07-25 15:09
ReporterMartin Lang Assigned ToAlexander Allmendinger  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Target Version1.03Fixed in Version1.03.341.383 
Summary0003445: Data Access/DA DataItems/015.js script throws ReferenceError: Can't find variable usStatus
Description

used CTT version 1.2.336.273:

If a server does not support Subcription Service Sets the script throw the following error:
"Test.Execute() encounted an unexpected error: ReferenceError: Can't find variable uaStatus"
This error is caused by line 84.
addError( "CreateMonitoredItems(): status '" + uaStatus, uaStatus );
There is no code implemented which creates a variable 'uaStatus', probably the service result could be mean?

But if a server does not support the subscription service set the script should not indicate an error. Therefore an if-statement is missing in front of line 84.

Steps To Reproduce
  • setup server which does NOT support the Subscription Service Set
  • run test script
Additional Information
  1. fix error from line 84:
    current implementation:
    addError( "CreateMonitoredItems(): status '" + uaStatus, uaStatus );

possible implementation for fix:
addError( "CreateMonitoredItems(): status '" + subscription.ServiceResult, subscription.ServiceResult );

  1. skip script if server does not support the CreateSubscription service:
    current implementation, line 63:
    // Create subscription for this test
    subscription = new Subscription();
    CreateSubscriptionHelper.Execute( {
    Subscription: subscription
    } );

possible implementation for fix:
// Create subscription for this test
subscription = new Subscription();
if( CreateSubscriptionHelper.Execute( {
Subscription: subscription
} ) == false )
{
return( false );
}

TagsNo tags attached.
Files Affected

Activities

Martin Lang

2016-06-06 15:57

reporter   ~0007004

related to issue 3367

Nathan Pocock

2016-08-05 21:03

viewer   ~0007133

Thank you. Removed the addError line altogether since the service-helper code reports errors internally. Checked into source-control and will be in the next release which is when this issue will be closed.

Alexander Allmendinger

2018-12-19 09:11

developer   ~0009734

Removed the unnecessary addError

Paul Hunkar

2019-07-25 15:09

administrator   ~0010603

Reviewed in CMP Call

Issue History

Date Modified Username Field Change
2016-06-06 15:10 Martin Lang New Issue
2016-06-06 15:57 Martin Lang Note Added: 0007004
2016-08-05 21:03 Nathan Pocock Note Added: 0007133
2016-08-05 21:03 Nathan Pocock Status new => resolved
2016-08-05 21:03 Nathan Pocock Resolution open => fixed
2016-08-05 21:03 Nathan Pocock Assigned To => Nathan Pocock
2018-11-23 15:31 Paul Hunkar Assigned To Nathan Pocock => Alexander Allmendinger
2018-11-23 15:31 Paul Hunkar Status resolved => assigned
2018-12-19 09:11 Alexander Allmendinger Note Added: 0009734
2018-12-19 09:11 Alexander Allmendinger Status assigned => resolved
2018-12-19 09:11 Alexander Allmendinger Fixed in Version => 1.03.341.383
2019-07-25 15:09 Paul Hunkar Category Implementation Bug => 1 - Script Issue
2019-07-25 15:09 Paul Hunkar Target Version => 1.03
2019-07-25 15:09 Paul Hunkar Status resolved => closed
2019-07-25 15:09 Paul Hunkar Note Added: 0010603