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

Test all of selfdrive/ #33575

Merged
merged 11 commits into from
Oct 4, 2024
Merged

Test all of selfdrive/ #33575

merged 11 commits into from
Oct 4, 2024

Conversation

sshane
Copy link
Contributor

@sshane sshane commented Sep 17, 2024

selfdrived wasn't getting tested as it was left out of testpaths, we should really remove that.

Fix in: #33571

@sshane
Copy link
Contributor Author

sshane commented Sep 17, 2024

We should just remove the hard-coded selfdrive paths, but that relies on pytest-dev/pytest-cpp#147 for some catch2 tests

Copy link
Contributor

github-actions bot commented Sep 17, 2024

UI Preview

All Screenshots

@sshane sshane changed the title Test selfdrive/selfdrived Test all of selfdrive/ Sep 17, 2024
Copy link
Contributor

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@sshane sshane merged commit 66ec788 into master Oct 4, 2024
17 of 18 checks passed
@sshane sshane deleted the test-selfdrived branch October 4, 2024 06:39
cydia2020 pushed a commit to cydia2020/dodgypilot that referenced this pull request Oct 6, 2024
* test selfdrived!

* exit() is for interactive sessions

* fix

* comments

* more

* test all of selfdrive/

* ignore what we used to

* fix test_alerts

* fix test_alertmanager.py
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