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

Cucumber: add short tests & refactor some steps #2382

Merged
merged 14 commits into from
Oct 4, 2024

Conversation

pet-mit
Copy link
Contributor

@pet-mit pet-mit commented Sep 9, 2024

  • added pandas requirement
  • improved short test 3 & medium test 35
  • added some short tests from Antares_Simulator_Tests_NR (4, 5, 6, 7, 8)
  • this needed adding some steps
  • added study_output_handler to make parsing output results cleaner & easier
  • simplified some steps by adding ":g" (float) or ":d" (int) flags, for behave to automatically parse parametes to the expected type
  • added messages to some asserts, so that when a step fails, the error message is clear

@pull-request-size pull-request-size bot added size/L and removed size/M labels Sep 30, 2024
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@pet-mit pet-mit changed the title [WIP] Cucumber: add short tests Cucumber: add short tests Oct 3, 2024
@pet-mit pet-mit changed the title Cucumber: add short tests Cucumber: add short tests & refactor some steps Oct 3, 2024
@pet-mit pet-mit added the tests label Oct 3, 2024
@pet-mit pet-mit marked this pull request as ready for review October 3, 2024 09:14
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Copy link

sonarcloud bot commented Oct 3, 2024

@pet-mit pet-mit merged commit da70910 into develop Oct 4, 2024
8 checks passed
@pet-mit pet-mit deleted the feature/add_short_tests branch October 4, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants