[regression] "Add parentheses around cast" doesn't work as expected

[fbricon]

There are 3 unit tests that pass (when they shouldn't, but that's another story), where "Add parentheses around cast" should actually add parentheses, but instead remove the cast in:

    public void foo(Object o) {
        String x= (String) o.substring(1);
    }

The tests are being rewritten and will be ignored for now, but I'm not sure what the actual issue is. The title is correct, the textedit is not. Stock Eclipse behaves properly, so it could be something we do in jdt.ls specifically.

[fbricon]

Tests to re-enable: ac29343#diff-525ef54ce68546beb0b4262b89013756