Skip to content

[utm] Add SCD0090 checks #1156

Merged
BenjaminPelletier merged 1 commit intointeruss:mainfrom
Orbitalize:implements_1144_2
Sep 15, 2025
Merged

[utm] Add SCD0090 checks #1156
BenjaminPelletier merged 1 commit intointeruss:mainfrom
Orbitalize:implements_1144_2

Conversation

@the-glu
Copy link
Copy Markdown
Contributor

@the-glu the-glu commented Sep 9, 2025

Follow #1155

Implements #1144

Add SCD0090 checks.

Comment thread monitoring/uss_qualifier/scenarios/astm/utm/check_for_conflict_notification.md Outdated
Comment thread monitoring/uss_qualifier/scenarios/astm/utm/check_for_conflict_notification.md Outdated
Comment thread monitoring/uss_qualifier/scenarios/astm/utm/check_for_conflict_notification.md Outdated
@the-glu the-glu force-pushed the implements_1144_2 branch 2 times, most recently from bec050d to c87fd65 Compare September 15, 2025 08:21
Copy link
Copy Markdown
Contributor

@barroco barroco left a comment

Choose a reason for hiding this comment

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

First pass

Comment thread monitoring/uss_qualifier/scenarios/astm/utm/check_for_conflict_notification.md Outdated
Comment thread monitoring/uss_qualifier/scenarios/astm/utm/record_current_notification.md Outdated
Comment thread monitoring/uss_qualifier/scenarios/astm/utm/record_current_notification.md Outdated
@the-glu the-glu force-pushed the implements_1144_2 branch 3 times, most recently from 936f19b to 2b33e53 Compare September 15, 2025 16:00
@the-glu
Copy link
Copy Markdown
Contributor Author

the-glu commented Sep 15, 2025

The PR should be ready, as discussed I changed the behavior to generate notification on successful creation that generated conflict, not on any creation, meaning only conflict_higher_priority tests generate those.

I kept the generic class for notifications tests since it's already done should there be a need in the future.

I merged that one with #1159 (who is closed now) to simply changes, since that one is smaller now.

@BenjaminPelletier BenjaminPelletier merged commit ae5429f into interuss:main Sep 15, 2025
37 of 38 checks passed
github-actions Bot added a commit that referenced this pull request Sep 15, 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.

3 participants