End to end tests for Redis. #1405
Conversation
# Conflicts: # .gitignore # core-tests/e2e-tests/pom.xml
| .filter(command -> command.getType().shouldCalculateHeuristic()) | ||
| .forEach(redisCommand -> { | ||
| RedisDistanceWithMetrics distanceWithMetrics = computeDistance(redisCommand, redisClient); | ||
| RedisDistanceWithMetrics distanceWithMetrics = computeDistance(redisCommand, (RedisClient) redisClient); |
There was a problem hiding this comment.
hi. thx @aszyrej i think it is best if @jgaleotti gives first round of comments, and then, after updating, i ll give my pass. but one thing here that caught my eyes is: why turning RedisClient into Object reference if anyway you do a cast to (RedisClient) here?
There was a problem hiding this comment.
Hi @arcuri82 . The reason why I'm using Object is just to be aligned with the solutions for Mongo and OpenSearch where they used Object for the clients. It's not necessary to keep it this way and since I have a dedicated class for encapsulating the client logic, I could refactor it and keep that class instead of Object.
I'll proceed with this if you agree.
There was a problem hiding this comment.
the reason for using private Object mongoClient is that cannot have dependencies to actual Mongo code, and must do access via reflection. if for Redis you encapsulate all the code for reflection in your own RedisClient that is different from io.lettuce.core.RedisClient, then no point in storing it as an Object reference. Btw, for clarity, your own RedisClient could be renamed into ReflectionBasedRedisClient (or something like that)
There was a problem hiding this comment.
Hi @arcuri82, I've renamed that class into ReflectionBasedRedisClient as we discussed, and changed the type in RedisHandler.
|
hi @jgaleotti it would be best if you look at this PR first, and then, after you approve it, i review it as well |
jgaleotti
left a comment
jgaleotti
left a comment
There was a problem hiding this comment.
Please review the use of string literals and the maximum distance value
...ler/src/main/java/org/evomaster/client/java/controller/redis/ReflectionBasedRedisClient.java
Outdated
Show resolved
Hide resolved
...ller/src/main/java/org/evomaster/client/java/controller/redis/RedisHeuristicsCalculator.java
Outdated
Show resolved
Hide resolved
jgaleotti
left a comment
jgaleotti
left a comment
There was a problem hiding this comment.
changes asked in pom.xml
jgaleotti
left a comment
jgaleotti
left a comment
There was a problem hiding this comment.
No further changes from my side. Please request Andrea's revision.
|
@arcuri82 would you review this PR and let me know if we're ready to merge? Thanks. |
| ReflectionBasedRedisClient connection = getRedisConnection(); | ||
| redisHandler.setRedisClient(connection); | ||
|
|
||
| List<AdditionalInfo> list = getAdditionalInfoList(); |
There was a problem hiding this comment.
@jgaleotti i see this was done for Mongo and OpenSearch as well... but why? can you leave some comments to explain it? i m a bit confused of what is this handle on the init of the handler based on additional info of currently executed test case
There was a problem hiding this comment.
Hi, thanks for pointing this out.
For Redis/OpenSearch, this is not right since there should be no handling for Redis/OpenSearch commands.
For Mongo, this is done for handling the expected classes to parse the document into. In other words, the call is to handle(MongoCollectionSchema) instead of handle(MongoFindCommand)
Could you remove this code @aszyrej ?
There was a problem hiding this comment.
I also pushed a change to this branch adding a comment in SutController.initMongoHandler()
| <parent> | ||
| <artifactId>evomaster-e2e-tests-spring</artifactId> | ||
| <groupId>org.evomaster</groupId> | ||
| <version>5.0.1-SNAPSHOT</version> |
There was a problem hiding this comment.
new relese was made yesterday. make sure to update all these values to latest 5.0.3-SNAPSHOT
…n Mongo handler initialization
First e2e-test for Redis.
Includes GET and SET commands, applying Redis heuristics.