Skip to content
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

Adding support for Redisson #23

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

kfadillah
Copy link

What does this pull request do?

Add ctest support for Redisson.

What changes were included in this pull request?
Because Redisson uses YAML to for its configuration setting import, some changes were made to support YAML configuration file.

For identify_params module:

  • add conf_params.txt and test_method_list.json for Redisson.
  • modify add_project.sh, identify_param.sh, and constant.py to include command and configuration value for setting up and running the Redisson.

for run_ctest module:

  • for inject.py, modify inject_config() function. Because Redisson uses yaml file to store configuration values, which is different from the xml file used in other maven-based projects.
  • for program_input.py, -DredisBinary needs to be added in the maven command to run redisson tests

for generate_ctest module:

  • modified value_generation.py for redisson.
  • due to the YAML support and the project structure in redisson, it can only inject the configuration values that exist in Config.java. Hence, I put all the testable parameters in redisson-default.tsv

for generate_ctest module:

  • modified inject.py, main.py, program_input.py for Redisson.
  • for inject.py, modify the function to support yaml file to store configuration values
  • for main.py, skipped parameters that are not in the mapping.

Others

  • Added patch for redisson ctest logging and injection
  • Added redis install in setup_ubuntu.sh
  • Added mapping for redisson in opensource-redisson.json

@kfadillah kfadillah changed the title Adding support for Cassandra Adding support for Redisson Dec 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant