-
Notifications
You must be signed in to change notification settings - Fork 22
[uss_qualifier] oir implicit sub scenario: oir with no subscription mutated to get implicit sub #1038
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
51a6844
to
bec9cd8
Compare
bec9cd8
to
3d1e56f
Compare
@@ -322,7 +322,38 @@ it is in violation of **[astm.f3548.v21.DSS0005,1](../../../../requirements/astm | |||
|
|||
#### 🛑 OIR is now attached to an implicit subscription check | |||
|
|||
If the DSS under test fails to attach the OIR to an implicit subscription (which may either already exist or be newly created) when it is requested to do so, | |||
If the DSS under test fails to attach the OIR to an implicit subscription (which may either already exist or be newly created) when it is requested to do, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the DSS under test fails to attach the OIR to an implicit subscription (which may either already exist or be newly created) when it is requested to do, | |
If the DSS under test fails to attach the OIR to an implicit subscription (which may either already exist or be newly created) when it is requested to do so, |
monitoring/uss_qualifier/scenarios/astm/utm/dss/oir_implicit_sub_handling.md
Show resolved
Hide resolved
@@ -134,7 +134,7 @@ or **[astm.f3548.v21.DSS0005,5](../../../../requirements/astm/f3548/v21.md)**. | |||
|
|||
#### [No implicit subscription was attached](./fragments/oir/oir_has_no_subscription.md) | |||
|
|||
### [Cleanup After Test Case test step](./cleanup_after_testcase_oir_subs.md) | |||
### [Cleanup After Test Case test step](./fragments/oir/crud/delete_query.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this change? Here and below.
Shouldn't the cleanup of subscriptions actually be implemented?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3d1e56f
to
3d66900
Compare
This covers the last scenario in interuss/dss#1088
The PR is stacked on top of #1036, please only consider the two last commit.