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.

Sign up for GitHub

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

Jump to bottom

Feature pytest #385

Closed
wants to merge 4 commits into from
Closed

Feature pytest #385

wants to merge 4 commits into from

Conversation

julienthevenoz
Copy link

@julienthevenoz julienthevenoz commented Dec 14, 2023 •
edited
Loading

  • test_flights.py (using unittest framework) replaces run_test.py (github action changed accordingly)
  • plotter class now corrects the 0.3 m X-axis offset in the pdf of the multitrajectory test (and other details)

@julienthevenoz
pytest integration first draft
acec690
@julienthevenoz
replaced run_test.py with test_flights.py ; changed the github action…
9852d0d 
… accordingly; corrected 0.3m x-offset in plotter for multi_trajectory (and other details)
whoenig
whoenig requested changes Dec 14, 2023
View reviewed changes
systemtests/test_flights.py Outdated Show resolved Hide resolved
@julienthevenoz
Copy link
Author

Just realized that the current github action will ONLY download the results folder if the tests pass, which is not what we want. So I am trying to find out how to make step6 of the job (download) execute even when step5 (fligh test) failed because of the assert statement. In the meantime I don't think it should be merged so I am closing the pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@whoenig whoenig whoenig requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@julienthevenoz @whoenig