Hi, Well after further investigation that change does actually seem like the right fix otherwise the method can exit with addLocation and removeLocation being identical, however I think I've come up with another fix.
I run own only unit test (90 odd test), the standing collection tests and the test that failed and all pass so I'm going to check it in and see if all tests pass. Thanks, Justin