Copilot commented on code in PR #2276:
URL: https://github.com/apache/nifi-minifi-cpp/pull/2276#discussion_r4198564532
##########
minifi_rust/extensions/minifi_tensor/src/processors/detect_object.rs:
##########
@@ -47,6 +47,7 @@ impl Schedule for DetectObject {
{
let image_to_tensor = ImageToTensor::schedule(context, logger)?;
let filter_bounding_boxes = FilterBoundingBoxes::schedule(context,
logger)?;
+ let _tract_model_service =
context.get_controller_service(&TRACT_MODEL_SERVICE)?;
Review Comment:
This makes the existing `test_missing_controller_service_errors` panic at
`DetectObject::schedule(...).expect("Expected to schedule")`, because its
`MockProcessContext` has no controller service and this lookup now returns
`MissingRequiredProperty`. Update that test to assert that scheduling fails
(rather than scheduling successfully and checking `transform`) so the Rust unit
suite remains green and covers the new behavior.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
