CIF-1135 - Random concurrency error in ResourceMapper

[cjelger]

• make sure the lock was acquired before releasing it

Types of changes

• [x] 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.
• [x] All unit tests pass on CircleCi.
• [x] I ran all tests locally and they pass.

[codecov[bot]]

Codecov Report

Merging #71 into master will increase coverage by 0.15%. The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #71      +/-   ##
============================================
+ Coverage     85.89%   86.05%   +0.15%     
- Complexity      363      365       +2     
============================================
  Files            23       23              
  Lines          1347     1348       +1     
  Branches        209      210       +1     
============================================
+ Hits           1157     1160       +3     
+ Misses           87       86       -1     
+ Partials        103      102       -1

Flag	Coverage Δ	Complexity Δ
#integration	54.22% <50%> (-0.05%)	228 <0> (ø)
#unittests	81.14% <100%> (+0.16%)	344 <0> (+2)	:arrow_up:

Impacted Files	Coverage Δ	Complexity Δ
...e/cq/commerce/graphql/resource/ResourceMapper.java	77.09% <100%> (+1.25%)	37 <0> (+2)	:arrow_up:

---

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 0651ae4...de43808. Read the comment docs.

[mhaack]

Verified by navigation product tree in the console, no errors in the logs.