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 Eclipse Jetty #18

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

Conversation

Moonlor
Copy link

@Moonlor Moonlor commented Dec 1, 2022

What's the purpose of this PR

Adding submodule jetty-servlet of Eclipse Jetty into the openctest framework

Brief changelog

  • Modified the submodule jetty-servletof and forked own ctest-jetty repository. Created two clean branches, ctest-logging and ctest-injection, in this repo, which can be used for values logging and injection.

  • Modified core/add_project.sh and core/identify_param/add_project.sh so that the project can be automatically added by running the script.

  • Added core/default_configs/jetty-servlet-default.tsv for default values to facilitate generating ctests.

  • Modified core/identify_param submodule to support identifying jetty-servlet parmas exercised in tests.

  • Modified core/generate_ctests submodule to support generating parameter sets for Ctests of jetty-servlet.

  • Modified core/generate_value: support generating valid values for jetty-servlet conf params.

  • Modified core/run_ctest submodule to support running generated ctests of jetty-servlet against configuration files.

A demo program_input could be:

p_input = {
    "run_mode": "run_ctest",
    "project": "jetty-servlet",
    "mapping_path": "../../data/ctest_mapping/opensource-jetty-servlet.json",
    "conf_file_dir": "sample-jetty-servlet",
    "display_mode": False,
    "use_surefire": True,
    "maven_args": [],
    "cmd_timeout": None,
}

Checklist:

  • Read the README of original repo.
  • Rebase all the commits to make the commits history clear.
  • Remove unnecessary files and code.
  • Run identify_param, generate_ctests and run_ctest to make sure the code is correct.

feat: add ctest jetty git patch
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.

None yet

1 participant