Skip to content

Conversation

@Shastick
Copy link
Contributor

@Shastick Shastick commented Jul 12, 2025

Some documented checks in subscription sync scenarios were documented but not verified.

For two of them the check is now conducted:

  • Get Subscription by ID has been added to confirm that the DSS instance can fulfill that kind of request.
  • Get subscription response content is correct and all relevant subchecks are now run

The documentation for Created Subscription is in search results is removed, it was included because the incorrect fragment was imported in the documentation. The intent of the scenario was to check for search query success, and to run synchronization-specific checks afterwards.

Progress on #975

@Shastick Shastick force-pushed the sub-sync-run-more-checks branch from aac6305 to c87de18 Compare July 14, 2025 06:40
@Shastick Shastick changed the title [uss_qualifier] subscription sync scenario: run documented checks [uss_qualifier] subscription sync scenario: run content validation checks and remove not-runned ones Jul 14, 2025
@Shastick Shastick force-pushed the sub-sync-run-more-checks branch from c87de18 to ac8bddc Compare July 14, 2025 06:55
@Shastick
Copy link
Contributor Author

For DSS0005,5, in synchronization scenarios, the report now looks like this:

Screenshot 2025-07-14 at 08 56 55

(the notification index check is handled in a separate PR)

@Shastick Shastick marked this pull request as ready for review July 14, 2025 06:57
@Shastick Shastick force-pushed the sub-sync-run-more-checks branch from ac8bddc to d77d751 Compare July 15, 2025 10:00
Copy link
Contributor

@mickmis mickmis left a comment

Choose a reason for hiding this comment

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

LGTM modulo rebase

…checks and remove not-runned ones

# Conflicts:
#	monitoring/uss_qualifier/scenarios/astm/utm/dss/fragments/sub/crud/read_correct.md
#	monitoring/uss_qualifier/scenarios/astm/utm/dss/synchronization/subscription_synchronization.md
@Shastick Shastick force-pushed the sub-sync-run-more-checks branch from d77d751 to da61bcb Compare July 15, 2025 17:01
@mickmis mickmis merged commit 11074b9 into interuss:main Jul 16, 2025
21 checks passed
@mickmis mickmis deleted the sub-sync-run-more-checks branch July 16, 2025 09:25
github-actions bot added a commit that referenced this pull request Jul 16, 2025
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