refactor(inventory): ent 4572 class to functions

[cdcabrera]

What's included

• fix(productViews): product configs, test timestamps
• refactor(inventoryCard): ent-4572 rename from inventoryList
• refactor(inventoryCard): ent-4572 class, function conversion
  • locale, disabled tab string
  • guestsList, class to func, deprecated guests
  • guestsListContext, hooks for selectors, action
  • inventoryCard, class to func
  • inventoryCardContext, hooks for selectors, action
  • inventorySubscriptions, class to func, alias inventoryCard
  • inventorySubscriptionsContext, hooks for selectors, action
  • productViews, component prop drill, config clean up
  • productViewContext, activate guests config
  • useReactRedux, expand base redux hooks
  • viewReducer, guests query and types
  • selectors, remove unused instances, subscriptions
  • rhsmConstants, expand for guests, number_of_guests
  • rhsmSchemas, guests, subscriptions added
  • rhsmTransformers, instances number_of_guests added
• refactor(inventory): ent-4572 rename components

Notes

• There are no visual updates associated with these refactors
• In order to retain the git history associated with the components we busted the renames into their own commits
• THESE INVENTORY UPDATES AFFECT THE RHOSAK PRODUCT DISPLAY (a precursor to the inventory updates that will happen for other products) and Subscription inventory table displays across all products
  • The prior class versions of guestsList and inventoryList/Card have been retained as deprecated versions, along with the refactors, since they access the hosts API endpoint. RHOSAK utilizes the newer instances API endpoint ergo it has been updated. The intention is to migrate our other products towards the function components used under RHOSAK when the number_of_guests facet is provided via the instances API, ENT-4580
• During the refactor we removed a couple of key attributes being used to cause refreshes on inventory. Everything appeared to be refreshing without them during testing, we'll monitor them, if caching issues pop up we'll reimplement them for instances, subscriptions, and the guests display

How to test

Local run check

1. update the NPM packages with $ yarn
2. $ yarn start
3. navigate to the RHOSAK product facet and confirm the inventory, guestsList, and subscriptions load without issue

Proxy run check

1. update the NPM packages with $ yarn
2. make sure Docker is running, plus on network, then
3. $ yarn start:proxy
4. 1. navigate to the RHOSAK product facet and confirm the inventory, guestsList, and subscriptions load without issue

Example

...

Updates issue/story

ent-4572

[codecov-commenter]

Codecov Report

Merging #857 (365b37d) into ci (de19628) will decrease coverage by 0.17%.
The diff coverage is 93.98%.

@@            Coverage Diff             @@
##               ci     #857      +/-   ##
==========================================
- Coverage   94.05%   93.88%   -0.18%     
==========================================
  Files         123      125       +2     
  Lines        3548     3661     +113     
  Branches     1377     1433      +56     
==========================================
+ Hits         3337     3437     +100     
- Misses        194      204      +10     
- Partials       17       20       +3     

Impacted Files	Coverage Δ
src/config/product.satellite.js	88.57% <ø> (ø)
src/types/rhsmApiTypes.js	100.00% <ø> (ø)
...torySubscriptions/inventorySubscriptionsContext.js	73.46% <73.46%> (ø)
src/components/guestsList/guestsList.js	83.72% <91.66%> (-14.75%)	⬇️
...c/components/inventoryList/inventoryCardContext.js	91.83% <91.83%> (ø)
src/components/guestsList/guestsListContext.js	96.07% <96.07%> (ø)
src/components/guestsList/guestsList.deprecated.js	98.46% <98.46%> (ø)
src/components/inventoryList/inventoryCard.js	100.00% <100.00%> (ø)
...c/components/inventoryList/inventoryCardHelpers.js	93.40% <100.00%> (ø)
...mponents/inventoryList/inventoryList.deprecated.js	90.54% <100.00%> (ø)
... and 15 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de19628...365b37d. Read the comment docs.