resolve SonarScan issues

[zacharyburnett]

Before the regression tests can be run in spacetelescope/RegressionTests, these issues must be resolved (ITSD requires that all code running on on-prem hardware pass SonarScan), or marked as non-issues. Can I mark any of these as "wontfix" or "false positive"?

https://plsonarqube.stsci.edu/project/issues?resolved=false&types=BUG&inNewCodePeriod=false&id=crds_client

• [x] Delete this unreachable code or refactor the code to make it reachable. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/crds/core/config.py#L883-L889
• [x] Group parts of the regex together to make the intended operator precedence explicit. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/crds/jwst/locate.py#L102
• [x] Introduce a new variable or use its initial value before reassigning 'rmapping'. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/crds/refactoring/refactor2.py#L260
• [x] Group parts of the regex together to make the intended operator precedence explicit. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/crds/tobs/locate.py#L72
• [x] The return value of "repr" must be used. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/test/certify/test_certify.py#L2393-L2398
• [x] Remove or refactor this statement; it has no side effects. https://github.com/spacetelescope/crds/blob/2d7c93bc2c86e3d5ed82a4b2d0c64c615f49a4f0/test/core/test_rmap.py#L983-L994