test: Fix flaky HybridInventory.cy.js test

[gkarat]

We don't pass data-ouia-safe to InventoryTable and it's set to true by default. Though, in the flaky test, for some reason, data-ouia-safe was missing completely, which makes this attribute somewhat unreliable to use: https://app.travis-ci.com/github/RedHatInsights/insights-inventory-frontend/jobs/625131895#L6568. HybridInventory.cy.test should use other way to indicate the table readiness.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 58.69%. Comparing base (30a8d9c) to head (96d26c5).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2243   +/-   ##
=======================================
  Coverage   58.69%   58.69%           
=======================================
  Files         198      198           
  Lines        5239     5239           
  Branches     1912     1912           
=======================================
  Hits         3075     3075           
  Misses       2164     2164           

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

[mkholjuraev]

Should not we fix passing data-ouia-safe to InventoryTable or give it a default value instead?

[gkarat]

@mkholjuraev, it's not an easy task to program data-ouia-safe properly: you should account for all data to be finished fetching, consider if all the animations are complete, etc. When, in the context of the tests, we have another indicator of table loading complete like a first row (or any other from the fixtures data) to be rendered. I think it's an easier solution and not requiring additional work in the source code (not the intention of this PR).

[mkholjuraev]

LGTM!