update test plans to include template-id instead of regex (infra) #1127
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1127 +/- ##
=======================================
Coverage 43.05% 43.05%
=======================================
Files 355 355
Lines 38575 38575
Branches 6549 6549
=======================================
Hits 16607 16607
Misses 21302 21302
Partials 666 666
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
tang-mm
changed the title
|
Pending for discussion: |
| wireless/wifi_ap_open_b_no_sta_interface_manual | ||
| wireless/wifi_ap_open_g_no_sta_interface_manual | ||
| wireless/wifi_ap_wpa_b_no_sta_interface_manual | ||
| wireless/wifi_ap_wpa_g_no_sta_interface_manual | ||
| wireless/wifi_ap_wpa_b_with_sta_.*_manual |
There was a problem hiding this comment.
Need review: regex in this and the following line might not match any test jobs
| @@ -300,7 +300,7 @@ unit: test plan | |||
| _name: Automated iperf3 tests | |||
| _description: Automated iperf3 performance test | |||
| include: | |||
| ethernet/iperf3_.* | |||
There was a problem hiding this comment.
The original regex matches not only iperf3_interface but also subsets with longer segments, such as iperf3_reverse_interface. This results in missing jobs after switching to template-id.
Regex like this one need to be matched with more than one template
|
Actions needed for this PR following the discussion:
|
Description
includefields with a corresponding template-idPending issues:
"audio/1_playback_thunderbolt_.*wireless/wifi_ap_wpa_b_with_sta_.*_manualResolved issues
Resolves CHECKBOX-1295
Documentation
Tests