Bug 497368

Description Carsten Reckord 2016-07-06 04:45:03 EDT

Create an interface IFoo and implementing classes Foo, FooImpl1, FooImpl2. In Foo, define a method as follows:

	public void log(Level level, String message, String... parameters) {
		String formattedMessage = MessageFormat.format(message, (Object[])parameters);
		System.out.println("Logging "+message+" with "+Arrays.asList(parameters));
		LogManager.getLogManager().getLogger(Foo.class.getName()).log(level, formattedMessage);
	}

Use the Pull Up refactoring to pull the method up into interface IFoo. In the wizard, select "Create necessary method stubs ..."

Expected outcome:
1. The result is compile-error free
2. The method is added to IFoo along with imports needed for the method signature
3. The method is added to all non-abstract implementors along with imports needed for the method signature
4. The method in implementors looks the same as when using the "Add unimplemented methods" hotfix to create it, including
- task marker //TODO auto-generated method stub
- @Override annotation

The method stub should look something like this:

	//import for java.util.logging.Level
	
	@Override
	public void log(Level level, String message, String... parameters) {
		// TODO Auto-generated method stub
		
	}

Actual outcome:
1. All implementors (except Foo) have compile errors
2. IFoo gets additional, unnecessary imports for classes used in the method body of the original implementation in Foo
3. Imports needed for the method signature are not added to implementors
4. The method in implementors lacks both the //TODO marker and the @Override annotation

The method stub actually looks like this:

	//no import for java.util.logging.Level

	public void log(Level level, String message, String... parameters) {
	}