View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005920 | CTT UA Scripts | 1 - Script Issue | public | 2020-08-28 23:05 | 2022-10-07 14:33 |
Reporter | Kevin Herron (Inductive Automation) | Assigned To | Alexander Allmendinger | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Fixed in Version | 1.03.501 | ||||
Summary | 0005920: Subscription Transfer - Test 015 | ||||
Description | This test case does not issue a PublishRequest in session 1 after the subscription is transferred to session 2 to consume the Good_SubscriptionTransferred. When the subscription is finally transferred back to session 1 the first PublishRequest issued to it returns this status change notification instead of the data change the script expects and it fails. Not only is this script defective, but that this script passes for other servers is probably a bad indication for that server because it should be sending the status change notification ASAP. | ||||
Tags | No tags attached. | ||||
Files Affected | /maintree/Subscription Services/Subscription Transfer/Test Cases/015.js | ||||
related to | 0006159 | closed | Alexander Allmendinger | Compliance Test Tool (CTT) Unified Architecture | Subscription Transfer - Test 015 |
|
After extensive review and checking of the spec, this is is not a problem. When a subscription is transferred, the objective of the message is to notify any client that are currently listening that the subscription has transferred. (i.e. there is a publish in the queue it should be dequeue and sent to the client - so the client is aware the subscription transferred.) we will clone this issue and use it as a base for a new issue that requires a new test case that includes sending publish commands down (like a well behaved client) that are just returned . The issue you are reporting is actually a misbehaving server - when you transfer a subscription to a new client it should be the queue that was on the previous client - it should not be picking up old messages that were in the old queue i.e. I transfer to from session 1 to session two message go to session 2 - queue for session 1 goes away, transfer back to session 1 I should not see anything that was left in the old queue (it could be hours old depending on the subscription time out. It should effectively be a new queue). The issue as stated is a No Fix |
|
The behavior of the test is correct - there server behavior is incorrect. |
|
review in call |
Kevin Herron (Inductive Automation) 2022-05-05 21:44 reporter ~0016676 Last edited: 2022-05-05 21:45 |
Disagree with your analysis Paul. The queues exist per session, and the CTT does not close the first session, and the server does not reasonably know there is not still a user waiting for a notification the subscription was transferred on the original session. When the subscription is finally transferred back and the original session+queue receives the PublishRequest it was starved for the first thing to go out should be the Good_SubscriptionTransferred status notification. If the CTT does not want to receive this notification it should close the original session and open a new one to transfer back to. |
|
The correct test should be to send a publish request on the first session after the transfer - and look for the the transferred message. |
|
After a longer discussion we came to the conclusion, that the specification is silent regarding what is happening with the existing queue, when the Subscription is transferred back to Session 1. Because of that we have decided to check for the StatusChangeNotification after pushing the subscription to Session 2. |
|
review changes in call agreed to updates and closed issue |
Date Modified | Username | Field | Change |
---|---|---|---|
2020-08-28 23:05 | Kevin Herron (Inductive Automation) | New Issue | |
2020-08-28 23:07 | Kevin Herron (Inductive Automation) | Summary | Subscription Transfer 015 => Subscription Transfer - Test 015 |
2020-10-16 00:11 | Paul Hunkar | Note Added: 0013062 | |
2020-10-16 00:11 | Paul Hunkar | Assigned To | => Paul Hunkar |
2020-10-16 00:11 | Paul Hunkar | Status | new => resolved |
2020-10-16 00:11 | Paul Hunkar | Resolution | open => no change required |
2020-10-16 00:11 | Paul Hunkar | Note Added: 0013063 | |
2020-10-16 00:15 | Paul Hunkar | Issue cloned: 0006159 | |
2020-10-16 00:17 | Paul Hunkar | Relationship added | related to 0006159 |
2021-02-18 16:43 | Paul Hunkar | Status | resolved => closed |
2021-02-18 16:43 | Paul Hunkar | Note Added: 0013743 | |
2022-05-05 21:44 | Kevin Herron (Inductive Automation) | Status | closed => feedback |
2022-05-05 21:44 | Kevin Herron (Inductive Automation) | Resolution | no change required => reopened |
2022-05-05 21:44 | Kevin Herron (Inductive Automation) | Note Added: 0016676 | |
2022-05-05 21:45 | Kevin Herron (Inductive Automation) | Note Edited: 0016676 | |
2022-05-05 22:18 | Paul Hunkar | Assigned To | Paul Hunkar => Alexander Allmendinger |
2022-05-05 22:20 | Paul Hunkar | Status | feedback => assigned |
2022-05-05 22:20 | Paul Hunkar | Note Added: 0016677 | |
2022-05-05 22:42 | Paul Hunkar | Status | assigned => feedback |
2022-05-05 22:43 | Paul Hunkar | Status | feedback => assigned |
2022-05-09 14:33 | Alexander Allmendinger | Status | assigned => resolved |
2022-05-09 14:33 | Alexander Allmendinger | Resolution | reopened => fixed |
2022-05-09 14:33 | Alexander Allmendinger | Note Added: 0016686 | |
2022-08-02 20:03 | Paul Hunkar | Project | Compliance Test Tool (CTT) Unified Architecture => CTT UA Scripts |
2022-08-02 20:23 | Paul Hunkar | Project | CTT UA Scripts => .NET API |
2022-08-02 20:23 | Paul Hunkar | Category | 1 - Script Issue => Api Change |
2022-08-04 14:24 | Paul Hunkar | Project | .NET API => Compliance Test Tool (CTT) Unified Architecture |
2022-10-07 14:32 | Paul Hunkar | Category | Api Change => 1 - Script Issue |
2022-10-07 14:32 | Paul Hunkar | Files Affected | => /maintree/Subscription Services/Subscription Transfer/Test Cases/015.js |
2022-10-07 14:32 | Paul Hunkar | Project | Compliance Test Tool (CTT) Unified Architecture => CTT UA Scripts |
2022-10-07 14:33 | Paul Hunkar | Status | resolved => closed |
2022-10-07 14:33 | Paul Hunkar | Fixed in Version | => 1.03.501 |
2022-10-07 14:33 | Paul Hunkar | Note Added: 0017976 |