Test modules CONFIG support

[sazzad16]

Redis 8 now includes module capabilities, which means that module-specific configuration parameters should be configurable via the the Redis CONFIG command. Enhance integration tests to validate that we can get/set module-specific settings

• You have read the contribution guidelines.
• You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
• You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
• You submit test cases (unit or integration tests) that back your changes.

[tishun]

(similar to the other issue)
Perhaps this needs to wait for #3145 to get merged so the CI could pass?
We may need to address the matrix build too.

[ggivo]

@sazzad16 @tishun

PR 3145 will not help in this case. 8.0 env is still bootstrapped by building it locally. When building locally with make Redis server 8.0 will not include modules by default. We can base those tests on io.lettuce.core.RedisContainerIntegrationTests which bring up test env from pre-build docker containers including modules

[tishun]

Same, about the conflicts

[sazzad16]

@tishun Done!

[tishun]

Hey, seems there are also some test failures, could you please take a look at that?

[sazzad16]

@tishun The tests are failing because Build and Test (unstable) is not running with updated RediSearch. For example, Redis 8.0-M04-pre image.

cc @ggivo

[ggivo]

@tishun The tests are failing because Build and Test (unstable) is not running with updated RediSearch. For example, Redis 8.0-M04-pre image.

cc @ggivo

Let me take a look. #3145 has been merged and now it should be using 8.0-M04-pre.

[sazzad16]

Hey, seems there are also some test failures, could you please take a look at that?

@tishun Fixed!

cc @ggivo

[tishun]

To expand on my thoughts on Ivo's suggestion for using a parametrized test:

Advantages

• (generally) if we want to change the steps of the tests we would make the change only in one place and only rotate parameters
• adding and removing test parameters (scenarios) is easy

Disadvantages

• (generally) we would assume that all these tests require the same steps always and if this changes we would need to extract them again

No clear winner for me, so I leave this to you folks to agree on

[sazzad16]

Thank you @tishun @ggivo