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

Change test 12.15.6 #969

Merged
merged 2 commits into from
Apr 22, 2024
Merged

Change test 12.15.6 #969

merged 2 commits into from
Apr 22, 2024

Conversation

pet-mit
Copy link
Collaborator

@pet-mit pet-mit commented Apr 12, 2024

In test 12.15.6, the FLOW RaoResult is unsecure.
Sometimes the CNE exporter checks the FLOW security status before ANGLE, sometimes it's the other way around.
When FLOW status is unsecure, if it is checked before ANGLE, then the CNE exporter won't assess the angles.
This makes the test cover less lines.

This change makes the FLOW secure so ANGLE status check is always covered by the test.

Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
@pet-mit pet-mit added the tests This issue or pull request only concerns automated tests label Apr 12, 2024
@pet-mit pet-mit marked this pull request as ready for review April 12, 2024 07:40
@pet-mit pet-mit requested a review from phiedw April 12, 2024 07:40
@pet-mit pet-mit added the PR: waiting-for-review This PR is waiting to be reviewed label Apr 22, 2024
@bqth29 bqth29 merged commit ad1c379 into main Apr 22, 2024
9 checks passed
MartinBelthle pushed a commit that referenced this pull request Apr 25, 2024
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Co-authored-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: belthlemar <martin.belthle@rte-france.com>
@pet-mit pet-mit deleted the change_swe_cne_test branch May 6, 2024 12:53
MartinBelthle pushed a commit that referenced this pull request May 28, 2024
Signed-off-by: Peter Mitri <peter.mitri@rte-france.com>
Co-authored-by: Thomas Bouquet <thomas.bouquet@rte-france.com>
Signed-off-by: belthlemar <martin.belthle@rte-france.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: waiting-for-review This PR is waiting to be reviewed tests This issue or pull request only concerns automated tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants