Wither and inheritance not work at all

[lombokissues]

Migrated from Google Code (issue 705)

[lombokissues]

👤 MAPADEP   🕗 Jul 15, 2014 at 20:02 UTC

Wither sould be inheritance compatible and pass that simple test.

@ Wither @ Getter
public class Foo {
    private int fooValue;

    public Foo(int fooValue) {
        this.fooValue = fooValue;
    }
}

@ Wither @ Getter
public class Bar extends Foo {
    private int barValue;

    public Bar(int fooValue, int barValue) {
        super(fooValue);
        this.barValue = barValue;
    }
}

@ Test
public void testWither() {
    final Bar bar = new Bar(3, 5);
    final Bar bar1 = bar.withBarValue(10);
    final Bar bar2 = bar.withFooValue(15);        

    assertThat(bar, instanceOf(Bar.class));
    assertThat(bar.getFooValue(), is(3));
    assertThat(bar.getBarValue(), is(5));

    assertThat(bar1, instanceOf(Bar.class));
    assertThat(bar1, is(not(bar)));
    assertThat(bar1.getFooValue(), is(3));
    assertThat(bar1.getBarValue(), is(10));

    assertThat(bar2, instanceOf(Bar.class));
    assertThat(bar2, is(not(bar)));
    assertThat(bar2.getFooValue(), is(15));
    assertThat(bar2.getBarValue(), is(5));
}

easiest way to implement - override with* methods in subclass and change return type.

[lombokissues]

👤 MAPADEP   🕗 Jul 15, 2014 at 20:12 UTC

little correction to test:
instance check should be assertThat(bar1, not(sameInstance(bar))) instead
assertThat(bar1, is(not(bar)))
same for assertThat(bar2, is(not(bar)))

[lombokissues]

👤 cesaryomismo   🕗 Apr 10, 2015 at 10:12 UTC

If we are using @ AllArgsConstructor inheritance doesn't work either...

[lombokissues]

End of migration