Skip to content

Modified regression tests to run in process, added test cases for parametric test #875

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

Merged

Conversation

ChristineZh0u
Copy link
Contributor

  • Changed regression tests from out of process runs to in process run to support regression test for the new parametric test generation feature.
  • Added test case for parametric test.
    • To test the right combination of tests are generated, add a "ExpectedParametricTests.txt" file in the same directory as the ".p" file.
    • To test the correctness of the .p program, make sure the test only generates one test case.

@ankushdesai
Copy link
Member

I thought we have moved and merged param_tests into the major 3.0 branch already. Can you please create a PR into that branch?

@ChristineZh0u
Copy link
Contributor Author

I thought we have moved and merged param_tests into the major 3.0 branch already. Can you please create a PR into that branch?

Just checked the main 3.0 branch — the param_tests branch hasn’t been merged in yet. I can create a pull request to merge current branch directly into major-3.0, but it’s probably better to merge it into param_tests first, and then merge param_tests into major-3.0.

@ankushdesai ankushdesai merged commit a4a6195 into dev_p3.0/param_testcases Jun 29, 2025
8 checks passed
@ankushdesai ankushdesai deleted the dev_p3.0/param_tests_regression_test branch June 29, 2025 06:19
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.

2 participants