test(nebula_ros): fix flaky smoke test

[mojomex]

PR Type

• Improvement

Related Links

• test(nebula_ros): add smoke tests #240 -- these turn out to be flaky in CI

Description

#240 is flaky in CI but not on my local machine (works on my machine 🤡). This leads me to believe that it's a timing issue - Nebula might still be in its startup phase when it's interrupted by the test framework, resulting in a non-zero exit code.

This PR makes the test suite wait for the Wrapper=OK output before interrupting Nebula.

Review Procedure

Hope and pray.

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Assign PR to reviewer

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• (Optional) Unit tests have been written for new behavior
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR: Required to pass before the merge.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 27.14%. Comparing base (48d9ab9) to head (4f1cf58).
Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #260      +/-   ##
==========================================
- Coverage   27.27%   27.14%   -0.14%     
==========================================
  Files         105      105              
  Lines        9465     9271     -194     
  Branches     2337     2057     -280     
==========================================
- Hits         2582     2517      -65     
+ Misses       6435     6334     -101     
+ Partials      448      420      -28     

Flag	Coverage Δ
differential	27.14% <ø> (?)
total	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mojomex]

Since CI takes ~10min per build/test cycle, there's no good way to test if flakiness is gone. Going ahead and merging.