Fix bug using AddOrUpdateAnnotationAttribute on annotation with empty brackets

[nielsdebruin]

What's changed?

Fixed bug that caused incorrect code when AddOrUpdateAnnotationAttribute on annotation with empty bracket e.g., @Foo().

What's your motivation?

The recipe AddOrUpdateAnnotationAttribute can be used to modify the attribute values on annotation. When no attribute values have been specified, the recipe will work fine if the syntax @Foo is used. However, if @Foo() is used, which is also valid java. Then, the recipe breaks.

Example

public @interface Foo {
    String[] array() default {};
}

If we run AddOrUpdateAnnotationAttribute on A annotated with @Foo, configured to set the value of Foo.array to "newTest1,newTest2" we correctly get:

@Foo
public class A {}
@Foo(array = {"newTest1", "newTest2"})
public class A {
}

Yet applying it to @Foo(), incorrectly yields

@Foo()
public class A {}
@Foo(array = "newTest1,newTest2", )
public class A {
}

Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context

Checklist

• I've added unit tests to cover both positive and negative cases
• I've read and applied the recipe conventions and best practices
• I've used the IntelliJ IDEA auto-formatter on affected files