Skip to content
Snippets Groups Projects

[test] Fix Recording Test

Merged MelanieCarriere requested to merge FixRecordingTest into master
All threads resolved!

The recording test used a single LidarStream (so a single Interpreter)
The recorded pcap was tested by the same LidarStream (so the same LidarInterpreter) than the one used to generate the recorded file.
We now use 2 different Streams/Interpreters. One that test the original pcap, the second test the recorded pcap.

long type does not have the same contained capacity on windows that on linux.
This leads to a different behavior on theses OS.
There is no real reason why we should let pass value that are equal up to 3600 * 1e6.

https://gitlab.kitware.com/LidarView/velodyneplugin-private/-/merge_requests/36

Edited by MelanieCarriere

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Nicolas Cadart
  • Nicolas Cadart approved this merge request

    approved this merge request

  • MelanieCarriere added 5 commits

    added 5 commits

    Compare with previous version

  • MelanieCarriere added 2 commits

    added 2 commits

    • a734ec41 - [test] Fix Recording Test
    • c2411971 - [test] Fix `TestPointDataArray` function in `testHelper`

    Compare with previous version

  • MelanieCarriere resolved all threads

    resolved all threads

  • MelanieCarriere added 17 commits

    added 17 commits

    Compare with previous version

  • MelanieCarriere mentioned in commit 87f9b1e4

    mentioned in commit 87f9b1e4

  • Please register or sign in to reply
    Loading