-
Notifications
You must be signed in to change notification settings - Fork 0
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
Integration tests: Small updates & triggering in every PR. #146
Conversation
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #146 +/- ##
==========================================
- Coverage 33.40% 33.30% -0.10%
==========================================
Files 68 69 +1
Lines 3739 3837 +98
==========================================
+ Hits 1249 1278 +29
- Misses 2490 2559 +69 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
This PR adds the ability to run integration tests on all connected platforms (RVC2, RVC4). If there are multiple RVC2 devices connected it will only select one.
Additionally, there was a bug in
ImageOutputParser
because we were setting the attributetransformation
of thedai.ImgFrame
which doesn't have this attribute - fixing it with the methodsetTransformations()
. Hopefully, this bugs will be catched now:)Lastly, we updated the CI workflows for E2E and integration tests with the option to specify which depthai version we want - the specified version will then be installed in the testbed server before the tests run. CI jobs will also wait until testbed is free and wont fail the job if testbed is reserved.
Integration tests are included in the CI for every PR. They are triggered only if the affected files are from
/depthai-nodes
or/tests/integration-tests folder
.Successful CI run for integration tests is HERE. It took 10 minutes.