Skip to content

Fix test_suite overwriting local Webots configuration#5965

Merged
stefaniapedrazzi merged 5 commits intomasterfrom
fix-test-suite-settings
Mar 6, 2023
Merged

Fix test_suite overwriting local Webots configuration#5965
stefaniapedrazzi merged 5 commits intomasterfrom
fix-test-suite-settings

Conversation

@stefaniapedrazzi
Copy link
Copy Markdown
Member

Fix #5960:
start directly the empty world instead of using the WEBOTS_SAFE_MODE to avoid overwriting the local Webots configuration.

@stefaniapedrazzi stefaniapedrazzi added bug Something isn't working test suite issue Test failure in the test suite CI test suite Start the test suite labels Mar 6, 2023
@stefaniapedrazzi stefaniapedrazzi added this to the R2023a-rev1 milestone Mar 6, 2023
@stefaniapedrazzi stefaniapedrazzi self-assigned this Mar 6, 2023
@stefaniapedrazzi stefaniapedrazzi changed the title Fix test_suite overwriting local Webots conficuration Fix test_suite overwriting local Webots configuration Mar 6, 2023
@stefaniapedrazzi stefaniapedrazzi marked this pull request as ready for review March 6, 2023 10:17
@stefaniapedrazzi stefaniapedrazzi requested a review from a team as a code owner March 6, 2023 10:17
Copy link
Copy Markdown
Member

@omichel omichel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove the --minimize option? This is pretty convenient when running the test suite locally, otherwise Webots occupies your screen while making tests, which is a bit annoying.

@stefaniapedrazzi
Copy link
Copy Markdown
Member Author

Oops, you're are right. I removed it for debugging and forget to readd it.

Copy link
Copy Markdown
Member

@omichel omichel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

@stefaniapedrazzi stefaniapedrazzi merged commit 5240fc8 into master Mar 6, 2023
@stefaniapedrazzi stefaniapedrazzi deleted the fix-test-suite-settings branch March 6, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working test suite issue Test failure in the test suite CI test suite Start the test suite

Development

Successfully merging this pull request may close these issues.

Test suite changes the graphical options to minimum

2 participants