CIF-1313 - Product carousel no longer displays product configured wit…

[cjelger]

…h JCR path

• fixed regression bug introduced by CIF-1253 - Product Carousel does not show configurable product and pr… #202
• extended unit test to make sure this is properly covered

How Has This Been Tested?

Extended unit test + manually tested.

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes and the overall coverage did not decrease.
• All unit tests pass on CircleCi.
• I ran all tests locally and they pass.

[codecov]

Codecov Report

Merging #220 into master will increase coverage by 3.93%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #220      +/-   ##
============================================
+ Coverage     56.16%   60.09%   +3.93%     
- Complexity      537      539       +2     
============================================
  Files           138      147       +9     
  Lines          3928     4368     +440     
  Branches        744      745       +1     
============================================
+ Hits           2206     2625     +419     
- Misses         1630     1652      +22     
+ Partials         92       91       -1     

Flag	Coverage Δ	Complexity Δ
#jest	38.21% <ø> (ø)	0.00 <ø> (ø)
#karma	94.97% <ø> (?)	0.00 <ø> (?)
#unittests	84.88% <100.00%> (+0.08%)	539.00 <0.00> (+2.00)

Impacted Files	Coverage Δ	Complexity Δ
...models/v1/productcarousel/ProductCarouselImpl.java	87.27% <100.00%> (+2.36%)	21.00 <0.00> (+2.00)
...ommerce/product/v1/product/clientlib/js/gallery.js	94.87% <0.00%> (ø)	0.00% <0.00%> (?%)
...ctlist/v1/productlist/clientlibs/js/productlist.js	93.61% <0.00%> (ø)	0.00% <0.00%> (?%)
...merce/product/v1/product/clientlib/js/addToCart.js	96.42% <0.00%> (ø)	0.00% <0.00%> (?%)
...ore/cif/clientlibs/common/js/CommerceGraphqlApi.js	88.33% <0.00%> (ø)	0.00% <0.00%> (?%)
...ctteaser/v1/productteaser/clientlibs/js/actions.js	92.30% <0.00%> (ø)	0.00% <0.00%> (?%)
...vigation/v1/navigation/clientlibs/js/navigation.js	98.66% <0.00%> (ø)	0.00% <0.00%> (?%)
...product/v1/product/clientlib/js/variantSelector.js	98.30% <0.00%> (ø)	0.00% <0.00%> (?%)
...ps/core/cif/clientlibs/common/js/PriceFormatter.js	90.90% <0.00%> (ø)	0.00% <0.00%> (?%)
...ommerce/product/v1/product/clientlib/js/product.js	96.29% <0.00%> (ø)	0.00% <0.00%> (?%)
... and 1 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 55b044a...b184b7c. Read the comment docs.