Skip to content

Instantly share code, notes, and snippets.

@ufuk
Last active June 27, 2019 13:04
Show Gist options
  • Star 4 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save ufuk/0ccb87185c22475c64d46801fa160777 to your computer and use it in GitHub Desktop.
Save ufuk/0ccb87185c22475c64d46801fa160777 to your computer and use it in GitHub Desktop.
Just another reason to why you shouldn't use Lombok, in another saying another reason to why you should write unit tests: You have two fields in your class. Fields are in the same type. You use @AllArgsConstructor to auto-generate your all args constructor. It works for a moment, until you change the order of the field.
import lombok.AllArgsConstructor;
import lombok.Data;
@Data
@AllArgsConstructor
public class LazyDeveloper {
private String firstName;
private String lastName;
}
import org.junit.Test;
import static org.hamcrest.Matchers.equalTo;
import static org.junit.Assert.assertThat;
public class LazyDeveloperTest {
@Test
public void testAllArgsConstructor() {
LazyDeveloper lazyDeveloper = new LazyDeveloper("John", "Doe");
assertThat(lazyDeveloper.getFirstName(), equalTo("John"));
assertThat(lazyDeveloper.getLastName(), equalTo("Doe"));
}
}
@koraytugay
Copy link

Hi Ufuk, what is the problem here?

@ufuk
Copy link
Author

ufuk commented Oct 7, 2016

If you change the order of fields, for example:

...
@AllArgsConstructor
public class LazyDeveloper {

    private String lastName;

    private String firstName;

}

Code will be compiled but test will fail, so your expectation will not be satisfied. Because first param of the newly auto-gerenated constructor will be "lastName".

I think that is a great risk when you use Lombok but don't write unit tests.

@Kidlike
Copy link

Kidlike commented Jun 1, 2018

sorry for the grave-digging, but I just wanted to clarify for anyone that stumbles upon this.

Your "problem" could be easily replicated with plain java:

public LazyDeveloper(String lastName, String firstName){}
public LazyDeveloper(String firstName, String lastName){}

There's no signature change; code will still compile fine; you still need unit tests.
I understand that it's simpler to do such mistakes with Lombok, but,
such constructors are dangerous anyway in large projects, they should be avoided regardless of using Lombok or not.
It also goes without saying that you need junit tests regardless of using Lombok or not :)

@ufuk
Copy link
Author

ufuk commented Aug 23, 2018

@Kidlike great addition, thank you. I encountered such a situation when using Lombok and just noted it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment