christopherjreid

Results 12 comments of christopherjreid

@nnmm Is this package structure what you envisioned for #237?

Based on previous feedback, I'm wondering about structuring this as - Setup a graph with a couple of nodes, publishers, subscribers, services and clients - Run tests for all the...

Hrm, looks like adding the `test_msgs` dependency also pulls lots of other depedencies in... `nav_msgs, sensor_msgs, diagnostic_msgs, ` and more ``` error: failed to load source for dependency `diagnostic_msgs` Caused...

After rebasing and adding all the msg dependencies, it's building and running now. I'm getting a spurious failure with `count_publishers` and `count_subscribers` - after constructing 2 publishers/subscribers, they occasionally return...

Updated to combine everything into a single test - constructing the graph, then calling extracted functions for checking publishers, subscriptions, etc. I know that's a bit of an anti-pattern, but...

I can't reproduce that error locally - looks like a problem with the export dependencies of `test_msgs`? Not sure how to proceed...

Thanks for the tips! That, and some test changes (apparently Galactic/cyclonedds do not allow publishing to the same topics with different types) got all the tests to pass! So this...

I just added a new commit that creates a new test case for publishers, and one for subscribers. I feel like this is a lot noisier than the previous method,...

Haven't looked at this in a couple days, but noticed CI had a test failure. Running this locally, I notice there's about a 1/15 chance or so that it randomly...

I would like to do it, just have been very busy (renovating our house). Hopefully I'll have some time this week to take a stab at it!