Skip to content

Fix utitiles readiness for ce-oem-mipi-camera tests (Bugfix)#2361

Merged
stanley31huang merged 6 commits intomainfrom
fix-camera-requires
Mar 11, 2026
Merged

Fix utitiles readiness for ce-oem-mipi-camera tests (Bugfix)#2361
stanley31huang merged 6 commits intomainfrom
fix-camera-requires

Conversation

@stanley31huang
Copy link
Collaborator

Description

there are some gstreamer and v4l2-ctl utilites are needed for the camera tests, and the utilities name might be diffrent on classic image and core image, so this fixed will remove those requires fileds from the functional tests, and add a new uitilities readiness test.

Resolved issues

Documentation

Tests

========[ Bootstrap com.canonical.contrib::mipi_camera_resource (1/1) ]=========
WARNING:checkbox_ng.user_utils:Using `ceqa` user
Reports will be saved to: /home/ceqa/snap/checkbox-ce-oem/5797/.local/share/checkbox-ng
==============[ Running job 1 / 1. Estimated time left: 0:00:02 ]===============
--------------[ Verify installation of V4L2 and GStreamer tools ]---------------
ID: com.canonical.contrib::ce-oem-mipi-camera/readiness
Category: com.canonical.plainbox::camera
... 8< -------------------------------------------------------------------------
INFO     - camera_test: readiness_test 480  - the location of gst-launch-1.0 is /snap/bin/oem-imx-gstreamer.gst-launch
INFO     - camera_test: readiness_test 480  - the location of media-ctl is /snap/checkbox-ce-oem/5797/usr/bin/media-ctl
INFO     - camera_test: readiness_test 480  - the location of v4l2-ctl is /snap/checkbox-ce-oem/5797/usr/bin/v4l2-ctl
------------------------------------------------------------------------- >8 ---
Outcome: job passed

add camera related utilities check, all capture and recording tests will be skipped if camera related utilities are not available.
fix issues
@stanley31huang stanley31huang added the ce-oem The PR only contains to the ce-oem provider under the contrib area label Mar 4, 2026
@stanley31huang stanley31huang changed the title Fix utitiles readiness for ce-oem-mipi-camera tests Fix utitiles readiness for ce-oem-mipi-camera tests (Bugfix) Mar 4, 2026
revised argument
Copy link
Contributor

@seankingyang seankingyang left a comment

Choose a reason for hiding this comment

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

Leave a suggestion in line.
The others are good.

Co-authored-by: Isaac Yang <47034756+seankingyang@users.noreply.github.com>
…jobs.pxu

Co-authored-by: stanley31huang <stanley.huang@canonical.com>
@seankingyang seankingyang self-requested a review March 9, 2026 08:38
seankingyang
seankingyang previously approved these changes Mar 9, 2026
Copy link
Contributor

@seankingyang seankingyang left a comment

Choose a reason for hiding this comment

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

LGTM~

fix pep8 issue
Copy link
Contributor

@seankingyang seankingyang left a comment

Choose a reason for hiding this comment

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

LGTM~

@stanley31huang stanley31huang merged commit d0a2e17 into main Mar 11, 2026
16 checks passed
@stanley31huang stanley31huang deleted the fix-camera-requires branch March 11, 2026 09:39
@codecov
Copy link

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.63%. Comparing base (0022f15) to head (691b5c0).
⚠️ Report is 18 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2361   +/-   ##
=======================================
  Coverage   57.63%   57.63%           
=======================================
  Files         465      465           
  Lines       47174    47174           
  Branches     8399     8399           
=======================================
  Hits        27191    27191           
  Misses      19105    19105           
  Partials      878      878           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

ce-oem The PR only contains to the ce-oem provider under the contrib area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants