rosbag2 icon indicating copy to clipboard operation
rosbag2 copied to clipboard

Remove `rosbag2_tests` direct dependence on `sqlite3`

Open emersonknapp opened this issue 2 years ago • 1 comments

Description

This is a code-decoupling issue. Currently rosbag2_tests (many of which tests are disabled on CI because they are very flaky), directly depends on the sqlite3 storage implementation, which creates a hard coupling to this choice of storage plugin. The tests should be changed to use the generic storage APIs so that they can test any storage implementation.

emersonknapp avatar Oct 04 '22 21:10 emersonknapp

It would be ideally if they will be able to test two storage implementations (sqlite3 and mcap) on CI.

MichaelOrlov avatar Oct 05 '22 01:10 MichaelOrlov