karellen / karellen-jdtls-kotlin / 23782633693

80%

push

github

Fix KotlinElement stub methods crashing jdtls hover handler [release] (#31)

## Summary
- `KotlinMethodElement.getDefaultValue()` threw
`UnsupportedOperationException` when jdtls
`HoverInfoProvider.computeJavadoc()` called it to check annotation
member default values
- Changed to return `null` (Kotlin methods don't have annotation
defaults)
- Added integration test exercising this code path

Discovered via the new `.whenComplete()` exception logging added to the
jdtls fork in PR #28.

## Test plan
- [x] 355 integration tests pass (`mvn clean verify`)
- [x] New test `testGetDefaultValueOnKotlinMethodDoesNotThrow` verifies
the fix

2488 of 3555 branches covered (69.99%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 1 file covered. (100.0%)

5230 of 6137 relevant lines covered (85.22%)

3.01 hits per line