Refactor : Removed some Implementation smells

[Patel-Savan]

No description provided.

[joshua-roberts]

Looks good except for the small logic fix. Can you rebase on master? Thanks!

[joshua-roberts]

This looks good but the logic is slightly different. We don't want to call getAssociationsWithSource if the ua node does not exist. The previous logic relied on the if statement short circuiting if the node did not exist so getAssociationsWithSource was never called. You could change it to something like this:

boolean nodesNotExist = (!nodeExists(ua) || !nodeExists(target));
if (nodesNotExist) {
    return;
}

boolean pathNotExist = (!getAssociationsWithSource(ua).contains(new Association(ua, target)));
if (pathNotExist) {
    return;
}

to get the same logic as the short circuit.