[TRNSPRTPCE-121] Unreachable catch in ServicehandlerTxRxCheck Created: 16/May/19  Updated: 13/Sep/21

Status: Open
Project: transportpce
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Bug Priority: Lowest
Reporter: Rami Mohamed Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Epic Link: Code cleaning

 Description   

In the method check in class ServicehandlerTxRxCheck the NullPointerException catch is unreachable

if (serviceEnd != null) {
    try {
        Long serviceRate = serviceEnd.getServiceRate();
        ServiceFormat serviceformat = serviceEnd.getServiceFormat();
        String clli = serviceEnd.getClli();
        if ((serviceRate == null) || (serviceRate <= 0)) {
            result = false;
            message = "Service " + endpointType + " rate is not set";
            LOG.debug(message);
        } else if (serviceformat == null) {
            result = false;
            message = "Service " + endpointType + " format is not set";
            LOG.debug(message);
        } else if (!checkString(clli)) {
            result = false;
            message = "Service" + endpointType + " clli format is not set";
            LOG.debug(message);
        } else {
            ComplianceCheckResult complianceCheckResult
                    = checkTxOrRxInfo(serviceEnd.getTxDirection(), serviceEnd.getRxDirection());
            if (!complianceCheckResult.hasPassed()) {
                result = false;
                message = complianceCheckResult.getMessage();
            }
        }
    } catch (NullPointerException e) {
        message = "Service " + endpointType + " rate, format or clli is not set";
        LOG.error(message, e);
        return new ComplianceCheckResult(false, message);
    }
} else {
    result = false;
    message = endpointType + " is not set";
    LOG.debug(message);
}

Generated at Wed Feb 07 20:42:18 UTC 2024 using Jira 8.20.10#820010-sha1:ace47f9899e9ee25d7157d59aa17ab06aee30d3d.