Bug 235534

Description kiril mitov 2008-06-04 02:40:31 EDT

Created attachment 103512 [details]
Test case for the problem

Hi,
This problem was found in our eclipse base product. We have actions for automatically managing annotations of a different j2ee artifacts. We are using AST to add/remove annotations from the type.

I have noticed that if the class looks like this:

//some comment
@SomeAnnotation
public class MyClass {}

And I try to remove the annotation using

List modifiers = bodyDeclaration.modifiers();
Annotation annotation = findAnnotation(bodyDeclaration);
modifiers.remove(annotation);
TextEdit edit = unit.rewrite(doc, null);
edit.apply(doc);

Then the result will be

public class MyClass{}

and it should be 

//some comment 
public class MyClass()

The shown code is also removing the comment. The returned from unit.rewrite() method TextEdit is containing a DeleteEdit for deleting the comment. 

I am attaching a test case that is doing exactly this.

It is creating a java project. Then a working copy in the java project. Then it is finding the annotation and is trying to remove this annotation.

I have captured the logic for removing the annotation in ComplexAnnotation class.
I have tried to make the test case as simple as possible. The test case is:

final String content = "//@ApplicationException(rollback=false)\r\n"
			+ "@ApplicationException(rollback=false)\r\n"
			+ "public class SessionBean1{}";
IType type = getWorkingCopyType("SessionBean1.java", content);
new ComplexAnnotation().removeFromMember(type, type.getElementName());
assertEquals(		removeWhitespaces("//@ApplicationException(rollback=false)\r\n"
		+ "public class SessionBean1{}"),
removeWhitespaces(type.getSource()));


Best Regards,
Kiril