-
Notifications
You must be signed in to change notification settings - Fork 234
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support to generate random Java Records #397
Comments
Since records are classes, EasyRandom is actually able to randomize them. Here is an example that works as expected with ER 4.3 and Java 14: record Person(int id, String name) {}
public static void main(String[] args) {
EasyRandom easyRandom = new EasyRandom();
Person person = easyRandom.nextObject(Person.class);
System.out.println("person.id = " + person.id());
System.out.println("person.name = " + person.name());
// prints:
// person.id = -1188957731
// person.name = eOMtThyhVNLWUZNRcBaQKxI
} Minimal complete example: test-easy-random-records.zip |
Hi @benas , it doesn't work with java 15.
https://mail.openjdk.java.net/pipermail/amber-dev/2020-June/006241.html |
Hi @mmaggioni-wise , Thank you for raising this. Indeed, things seem to have changed in Java 15. The fact that records are immutable is actually incompatible with the way Easy Random works which is creating an a new ("empty") instance of the target type and then populating its fields afterwards. With records, it should be the other way around: generate random values for record components (ie fields) then create an instance with those values. So I think this feature should not be implemented in Easy Random itself (given the impact this has on the current code/approach), but using a custom package org.jeasy.random;
import java.lang.reflect.Constructor;
import java.lang.reflect.RecordComponent;
import org.jeasy.random.api.RandomizerContext;
public class RecordFactory extends ObjenesisObjectFactory {
private EasyRandom easyRandom;
@Override
public <T> T createInstance(Class<T> type, RandomizerContext context) {
if (easyRandom == null) {
easyRandom = new EasyRandom(context.getParameters());
}
if (type.isRecord()) {
return createRandomRecord(type);
} else {
return super.createInstance(type, context);
}
}
private <T> T createRandomRecord(Class<T> recordType) {
// generate random values for record components
RecordComponent[] recordComponents = recordType.getRecordComponents();
Object[] randomValues = new Object[recordComponents.length];
for (int i = 0; i < recordComponents.length; i++) {
randomValues[i] = easyRandom.nextObject(recordComponents[i].getType());
}
// create a random instance with random values
try {
return getCanonicalConstructor(recordType).newInstance(randomValues);
} catch (Exception e) {
throw new ObjectCreationException("Unable to create a random instance of recordType " + recordType, e);
}
}
private <T> Constructor<T> getCanonicalConstructor(Class<T> recordType) {
RecordComponent[] recordComponents = recordType.getRecordComponents();
Class<?>[] componentTypes = new Class<?>[recordComponents.length];
for (int i = 0; i < recordComponents.length; i++) {
// recordComponents are ordered, see javadoc:
// "The components are returned in the same order that they are declared in the record header"
componentTypes[i] = recordComponents[i].getType();
}
try {
return recordType.getDeclaredConstructor(componentTypes);
} catch (NoSuchMethodException e) {
// should not happen, from Record javadoc:
// "A record class has the following mandated members: a public canonical constructor ,
// whose descriptor is the same as the record descriptor;"
throw new RuntimeException("Invalid record definition", e);
}
}
} With this custom factory, the previous example works with Java 15 (assuming that the custom factory is passed to ER through parameters): EasyRandomParameters parameters = new EasyRandomParameters()
.objectFactory(new RecordFactory());
EasyRandom easyRandom = new EasyRandom(parameters);
// ... Minimal complete example for reference: test-easy-random-records-custom-factory.zip |
thank you @benas for your answer and your example! |
Thank you @benas. Are you sure that this should not be enabled by default for records (seeing as they will become a standard feature should easy-random not support them out of the box)? What would be the negatives of doing so? |
@vab2048 It should, this is what I had in mind when I first created this issue. The idea was that easy random should be able to handle both regular classes and records in a transparent way. This was true with Java 14 (as shown in #397 (comment)), but unfortunately not anymore with Java 15 as reported by @mmaggioni-wise (it's ok that things break since java records are still in preview). Records will be out of preview in Java 16. This means adding record support in ER in a transparent way (ie without a custom object factory) requires a major release that is based on Java 16 (there are techniques to do conditional compilation &co but I'm not ready to deal with that here). Even though that's fine in my opinion, it is a quite aggressive requirement, I remember some users were reluctant to making ER v5 based on Java 11 already, see dfbab94#r43985189). But the real issue is not about the minimum required Java version. The issue is that since java records are immutable, they should be randomized in the opposite way of randomizing java beans (ie first generate random values then create a record instance with them vs create an empty bean instance then set random values on it). This requires a substantial amount of work to implement and test. I think it's unfortunate that ER does not support records OOTB after being declared in maintenance mode (this decision was made based on the assumption that records are already supported, which, I must admit, is a mistake since I should have expected things to break before this feature goes out of preview..). So I think it's wise to re-consider records support in ER v6 which would be based on Java 16 (in which records will be stable and we can expect no breaking changes as the ones introduced between 14 and 15). wdyt? |
@benas your idea of waiting until records become a standard feature makes perfect sense. With Java 16 due to be released in March 2021 and records becoming a standard feature (https://openjdk.java.net/projects/jdk/16/) perhaps this issue should remain open as a reminder for the future once it is released? Until then the code example you provided works wonderfully with Java 15 and I suspect would also work with Java 16 - so we have an easy workaround. |
An additional issue I have found when using records: For example consider: public interface Dog {
void bark();
} with a single implementation: public record Rottweiler(String name) implements Dog {
@Override
public void bark() {
System.out.println("WOOF WOOF");
}
} And we have the following record which has an interface ( public record DogOwner(String ownerName, Dog dog) {} Now with the following test: /**
* Fails... when it should succeed.
*/
@Test
void easyRandom_DogOwner() {
var easyRandom = getInstance();
// Should not throw.... but does...
var dogOwner = easyRandom.nextObject(DogOwner.class);
System.out.println(dogOwner.toString());
}
private EasyRandom getInstance() {
EasyRandomParameters parameters = new EasyRandomParameters()
.objectFactory(new EasyRandomRecordFactory())
.scanClasspathForConcreteTypes(true)
;
return new EasyRandom(parameters);
} fails with the exception:
I think its because it still attempts to create the record like it was a normal class. I've attached an example project which replicates this (which is attached). |
Hi @benas |
Additional Information: The Lombok Here are is an examples that work fine for me with ER 5. @Value
public class Point {
int x, y;
}
@Test
void shouldRandomizePoint() {
// Given
var easyRandom = new EasyRandom();
// When
var randomizedPoint = easyRandom.nextObject(Point.class);
// Then
assertThat(randomizedPoint.getX()).isNotNull();
assertThat(randomizedPoint.getY()).isNotNull();
} Even if I'm "immitating" the java record by implementing an immutable class by myself ER works fine. Maybe I'm mistaking something but I don't see the difference to records. public final class Point {
private final int x;
private final int y;
public Point(int x, int y) {
this.x = x;
this.y = y;
}
public int x() {
return x;
}
public int y() {
return y;
}
@Override
public boolean equals(Object o) {
if (this == o) {
return true;
}
if (o == null || getClass() != o.getClass()) {
return false;
}
Point point = (Point) o;
return x == point.x && y == point.y;
}
@Override
public int hashCode() {
return Objects.hash(x, y);
}
@Override
public String toString() {
return "Point{" +
"x=" + x +
", y=" + y +
'}';
}
} |
Any advance in this feature? Java 16 was released in March 2020 which made records official part of the language. Is this going to be implemented or is Easy Random dead for good? |
It would be a must to be able to support @record since in 2020 became records official part of the language. will it be available in easy random roadmap? |
This comment was marked as off-topic.
This comment was marked as off-topic.
Since Java 15 EasyRandom cannot handle Java Records. j-easy/easy-random#397
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Initial support for records has been added and will be released in v6. As mentioned in my previous comment, the idea is to make ER handle both beans and records in a transparent way. This is now possible, and there is no need to register a custom object factory for records. v6.0.0 snapshots are available now, and I would be grateful to anyone interested in giving this a try and reporting any problem in a separate issue. There might be a few edge cases that should be tested in depth (recursion, composition, etc), but the goal here is iterate on the feature. @vab2048 thank you for reporting this issue, I created #496 for further investigation. For those wondering why this was not addressed when Java 16 was out, here is the reason: while I could have released ER6 right after Java 16 was released (in which records were finalized), I thought basing ER 6 on an LTS Java release made more sense. Hence, ER 6 will be based on Java 17 (#495). As a side note, it is great to see other people working on different/similar solutions in the same space! However, this is not the place to do issue management on forks or to promote other projects. I have hidden all comments that are off-topic. If anyone is interested in promoting a fork or a similar project, please reach out to me and I would be more than happy to mention it in the README. Thank you all! |
Hello ! I haven't tested the v6.0.0 snapshot but I am using your workaround posted above. I found an issue with that workaround. I have a record class with contains a list of another record object (besides other fields). When I tried to generate a random object, that list is null. Not sure if this is already addressed in the v6.0.0. I tried to find another workaround for this, and here is my solution (if you will add it in v6.0.0 the code will be much simpler).
|
Hi, I seem to have run into an issue generating random record objects. It seems that generation fails when the record object has a field of type Optional. The error occurs in the v6 snapshot and the factory mentioned in the comment. Using the factory and v5 version of the library the error is: Caused by: java.lang.IllegalAccessException: class org.jeasy.random.util.ReflectionUtils cannot access a member of class java.util.Optional (in module java.base) with modifiers "private final"
at java.base/jdk.internal.reflect.Reflection.newIllegalAccessException(Reflection.java:392)
at java.base/java.lang.reflect.AccessibleObject.checkAccess(AccessibleObject.java:674)
at java.base/java.lang.reflect.Field.checkAccess(Field.java:1102)
at java.base/java.lang.reflect.Field.get(Field.java:423)
at org.jeasy.random.util.ReflectionUtils.getFieldValue(ReflectionUtils.java:167)
at org.jeasy.random.EasyRandom.populateField(EasyRandom.java:206)
at org.jeasy.random.EasyRandom.populateFields(EasyRandom.java:198)
at org.jeasy.random.EasyRandom.doPopulateBean(EasyRandom.java:165) The error does not occur when trying to create a random instance of a class with a field of type Optional. I think the problem comes from using |
@johan-miller you should not use Optional as a class field type. Instead, use a getter for that nullable value. Something like that
|
Java 14 introduced Records. It would be great if Easy Random can generate random instances of such types.
The text was updated successfully, but these errors were encountered: