Skip to content

Conversation

@seanavery
Copy link
Collaborator

@seanavery seanavery commented Nov 4, 2025

Description

Simple unit tests against the CSICamera class to sanity check diffs.

  • Revamps test_csi_camera.cpp to work with latest code
  • Adds unit test workflow runner
  • Test pipeline now works on both Pi and Jetson

Tests

  • Unit test wofklow run

Refs

Gtest Docs

if [ "$(TARGET)" = "jetson" ]; then \
apt-get -y install libgtest-dev && \
apt-get install -y gstreamer1.0-tools && \
apt-get install -y gstreamer1.0-tools gstreamer1.0-plugins-good && \
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Need this to run test piepline on Ubuntu

workflow_run:
workflows: ["Build Latest"]
types: [completed]
push:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Small fix here for integration test triggers

inited = true;
}

// Test that the camera can be created with default values

Choose a reason for hiding this comment

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

As a follow up, it would be great to be able to inject pictures and make sure to receive them.

Copy link

@SebastianMunozP SebastianMunozP left a comment

Choose a reason for hiding this comment

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

LGTM.

@seanavery seanavery merged commit 7dfdc8b into viam-modules:main Nov 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants