1 of 4 - refactor(config): sw-625 generate routes

[cdcabrera]

What's included

• refactor(config): sw-625 generate routes
• refactor(redux): sw-625 remove connectRouter
• refactor(useRouter): sw-625 combine routerContext

Notes

• THIS AFFECTS ALL PRODUCT ROUTING. RHEL, OpenShift, RHODS, RHACS, RHOSAK/Streams, Satellite, etc
  • This update, because its generating routes from aspects of product configurations potentially allows one-off names to load the correct product.
• no visual changes associated with this update
• refactor prep for moving over to router 6, generate routes from available product configs
  • we're temporarily adding more code than we're removing in an effort to create fallbacks and history. the route generator will be removed in PR 3 of 4 in favor of a :productParameter along with some other removals.
  • The primary work is around generating a single product config via Webpack under ./src/config/products.js. This is prep for moving towards the API configuration endpoint
• appears to be a known route double load issue (manifests/displays as history having 2 entries if you hover over the browser back button, and potentially causes loading of api calls twice, for every route that's loaded. Curiosity cancels dup calls that have not resolved) when using the left navigation. this is being correlated by the consoledot ui team with the diff between react router 6 vs 5
  • this issue is currently in env. this pr does not resolve the issue

How to test

Coverage and basic unit test check

1. update the NPM packages with $ yarn
2. $ yarn test
3. confirm tests come back clean

Local run check

1. update the NPM packages with $ yarn
2. $ yarn start
3. confirm products can still be navigated towards locally

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. confirm
   • left navigation brings up the appropriate product for all products
   • optin is still activated
   • manual url updates still navigate towards a product, ie. [consoledot]/subscriptions/[some random path]/rhel

Example

...

Updates issue/story

sw-625

[codecov-commenter]

Codecov Report

Merging #1045 (14c288f) into dev (f8a813f) will increase coverage by 0.22%.
The diff coverage is 93.84%.

Additional details and impacted files

@@            Coverage Diff             @@
##              dev    #1045      +/-   ##
==========================================
+ Coverage   93.31%   93.54%   +0.22%     
==========================================
  Files         122      122              
  Lines        4071     4075       +4     
  Branches     1662     1640      -22     
==========================================
+ Hits         3799     3812      +13     
+ Misses        255      246       -9     
  Partials       17       17              

Impacted Files	Coverage Δ
src/components/router/router.js	95.23% <ø> (-1.32%)	⬇️
src/config/index.js	100.00% <ø> (ø)
src/config/product.openshiftDedicated.js	80.76% <ø> (ø)
src/config/product.rhacs.js	85.71% <ø> (ø)
src/config/product.rhods.js	85.71% <ø> (ø)
src/config/product.rhosak.js	83.72% <ø> (ø)
src/components/productView/productViewMissing.js	87.50% <66.66%> (-1.39%)	⬇️
src/components/router/redirect.js	94.11% <90.00%> (-5.89%)	⬇️
src/components/router/routerContext.js	94.44% <90.90%> (-5.56%)	⬇️
src/config/products.js	90.90% <90.90%> (ø)
... and 14 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 f8a813f...14c288f. Read the comment docs.