fix: host count for remediation#1505
Conversation
swadeley
force-pushed
the
swadeley/fix_host_count_planned_remediations
branch
from
eef7060 to
4bdcd0c
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1505 +/- ##
=======================================
Coverage 62.53% 62.53%
=======================================
Files 127 127
Lines 3310 3310
Branches 892 899 +7
=======================================
Hits 2070 2070
Misses 1120 1120
Partials 120 120 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| 'true', | ||
| ); | ||
| await waitForTableLoad(page); | ||
| await filterByName(page, system.name); |
There was a problem hiding this comment.
do you have a link handy to an example where this part of the test was failing? (nvm, i see your previous comment with a link in the other PR now 😃). the system filter is only on the systems tab, so that locator won't be found here 🤔
we are filtering by the system name already in the systems tab. there might be more than one system listed on that tab (maybe if they weren't cleaned up properly?), but since we're not checking the count anymore that should be fine
4 participants
Description
Fix test to filter on system created for the test.
How to test the PR
'Check actions, systems, and execution history tabs' test passes.
Before the change
'Check actions, systems, and execution history tabs' test fails because many other systems from other tests are present in the table.
After the change
'Check actions, systems, and execution history tabs' test passes.