Skip to content

[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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Shastick
Copy link
Contributor

@Shastick Shastick commented Jun 3, 2025

This covers the last scenario in interuss/dss#1088

The PR is stacked on top of #1036, please only consider the two last commit.

@Shastick Shastick force-pushed the oir-implicit-expand-8 branch from 51a6844 to bec9cd8 Compare June 3, 2025 16:12
@Shastick Shastick requested a review from mickmis June 3, 2025 16:14
@Shastick Shastick marked this pull request as ready for review June 3, 2025 16:14
@Shastick Shastick force-pushed the oir-implicit-expand-8 branch from bec9cd8 to 3d1e56f Compare June 4, 2025 08:20
@@ -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,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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,

@@ -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)
Copy link
Contributor

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?

Copy link
Contributor Author

@Shastick Shastick Jun 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is/should be in #1036 but probably got caught in here due to a git mistake. The PR has been updated

(the branch has been rebased, the two last commits on this PR are accurately reflecting the change this introduces compared to #1036)

@Shastick Shastick force-pushed the oir-implicit-expand-8 branch from 3d1e56f to 3d66900 Compare June 5, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants