You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
251 lines
9.1 KiB
251 lines
9.1 KiB
import capnp
|
|
import contextlib
|
|
import io
|
|
import shutil
|
|
import tempfile
|
|
import os
|
|
import pytest
|
|
import requests
|
|
|
|
from parameterized import parameterized
|
|
|
|
from cereal import log as capnp_log
|
|
from openpilot.tools.lib.logreader import LogIterable, LogReader, comma_api_source, parse_indirect, ReadMode, InternalUnavailableException
|
|
from openpilot.tools.lib.route import SegmentRange
|
|
from openpilot.tools.lib.url_file import URLFileException
|
|
|
|
NUM_SEGS = 17 # number of segments in the test route
|
|
ALL_SEGS = list(range(NUM_SEGS))
|
|
TEST_ROUTE = "344c5c15b34f2d8a/2024-01-03--09-37-12"
|
|
QLOG_FILE = "https://commadataci.blob.core.windows.net/openpilotci/0375fdf7b1ce594d/2019-06-13--08-32-25/3/qlog.bz2"
|
|
|
|
|
|
def noop(segment: LogIterable):
|
|
return segment
|
|
|
|
|
|
@contextlib.contextmanager
|
|
def setup_source_scenario(mocker, is_internal=False):
|
|
internal_source_mock = mocker.patch("openpilot.tools.lib.logreader.internal_source")
|
|
openpilotci_source_mock = mocker.patch("openpilot.tools.lib.logreader.openpilotci_source")
|
|
comma_api_source_mock = mocker.patch("openpilot.tools.lib.logreader.comma_api_source")
|
|
if is_internal:
|
|
internal_source_mock.return_value = [QLOG_FILE]
|
|
else:
|
|
internal_source_mock.side_effect = InternalUnavailableException
|
|
|
|
openpilotci_source_mock.return_value = [None]
|
|
comma_api_source_mock.return_value = [QLOG_FILE]
|
|
|
|
yield
|
|
|
|
|
|
class TestLogReader:
|
|
@parameterized.expand([
|
|
(f"{TEST_ROUTE}", ALL_SEGS),
|
|
(f"{TEST_ROUTE.replace('/', '|')}", ALL_SEGS),
|
|
(f"{TEST_ROUTE}--0", [0]),
|
|
(f"{TEST_ROUTE}--5", [5]),
|
|
(f"{TEST_ROUTE}/0", [0]),
|
|
(f"{TEST_ROUTE}/5", [5]),
|
|
(f"{TEST_ROUTE}/0:10", ALL_SEGS[0:10]),
|
|
(f"{TEST_ROUTE}/0:0", []),
|
|
(f"{TEST_ROUTE}/4:6", ALL_SEGS[4:6]),
|
|
(f"{TEST_ROUTE}/0:-1", ALL_SEGS[0:-1]),
|
|
(f"{TEST_ROUTE}/:5", ALL_SEGS[:5]),
|
|
(f"{TEST_ROUTE}/2:", ALL_SEGS[2:]),
|
|
(f"{TEST_ROUTE}/2:-1", ALL_SEGS[2:-1]),
|
|
(f"{TEST_ROUTE}/-1", [ALL_SEGS[-1]]),
|
|
(f"{TEST_ROUTE}/-2", [ALL_SEGS[-2]]),
|
|
(f"{TEST_ROUTE}/-2:-1", ALL_SEGS[-2:-1]),
|
|
(f"{TEST_ROUTE}/-4:-2", ALL_SEGS[-4:-2]),
|
|
(f"{TEST_ROUTE}/:10:2", ALL_SEGS[:10:2]),
|
|
(f"{TEST_ROUTE}/5::2", ALL_SEGS[5::2]),
|
|
(f"https://useradmin.comma.ai/?onebox={TEST_ROUTE}", ALL_SEGS),
|
|
(f"https://useradmin.comma.ai/?onebox={TEST_ROUTE.replace('/', '|')}", ALL_SEGS),
|
|
(f"https://useradmin.comma.ai/?onebox={TEST_ROUTE.replace('/', '%7C')}", ALL_SEGS),
|
|
(f"https://cabana.comma.ai/?route={TEST_ROUTE}", ALL_SEGS),
|
|
])
|
|
def test_indirect_parsing(self, identifier, expected):
|
|
parsed, _, _ = parse_indirect(identifier)
|
|
sr = SegmentRange(parsed)
|
|
assert list(sr.seg_idxs) == expected, identifier
|
|
|
|
@parameterized.expand([
|
|
(f"{TEST_ROUTE}", f"{TEST_ROUTE}"),
|
|
(f"{TEST_ROUTE.replace('/', '|')}", f"{TEST_ROUTE}"),
|
|
(f"{TEST_ROUTE}--5", f"{TEST_ROUTE}/5"),
|
|
(f"{TEST_ROUTE}/0/q", f"{TEST_ROUTE}/0/q"),
|
|
(f"{TEST_ROUTE}/5:6/r", f"{TEST_ROUTE}/5:6/r"),
|
|
(f"{TEST_ROUTE}/5", f"{TEST_ROUTE}/5"),
|
|
])
|
|
def test_canonical_name(self, identifier, expected):
|
|
sr = SegmentRange(identifier)
|
|
assert str(sr) == expected
|
|
|
|
@pytest.mark.parametrize("cache_enabled", [True, False])
|
|
def test_direct_parsing(self, mocker, cache_enabled):
|
|
file_exists_mock = mocker.patch("openpilot.tools.lib.logreader.file_exists")
|
|
os.environ["FILEREADER_CACHE"] = "1" if cache_enabled else "0"
|
|
qlog = tempfile.NamedTemporaryFile(mode='wb', delete=False)
|
|
|
|
with requests.get(QLOG_FILE, stream=True) as r:
|
|
with qlog as f:
|
|
shutil.copyfileobj(r.raw, f)
|
|
|
|
for f in [QLOG_FILE, qlog.name]:
|
|
l = len(list(LogReader(f)))
|
|
assert l > 100
|
|
|
|
with pytest.raises(URLFileException) if not cache_enabled else pytest.raises(AssertionError):
|
|
l = len(list(LogReader(QLOG_FILE.replace("/3/", "/200/"))))
|
|
|
|
# file_exists should not be called for direct files
|
|
assert file_exists_mock.call_count == 0
|
|
|
|
@parameterized.expand([
|
|
(f"{TEST_ROUTE}///",),
|
|
(f"{TEST_ROUTE}---",),
|
|
(f"{TEST_ROUTE}/-4:--2",),
|
|
(f"{TEST_ROUTE}/-a",),
|
|
(f"{TEST_ROUTE}/j",),
|
|
(f"{TEST_ROUTE}/0:1:2:3",),
|
|
(f"{TEST_ROUTE}/:::3",),
|
|
(f"{TEST_ROUTE}3",),
|
|
(f"{TEST_ROUTE}-3",),
|
|
(f"{TEST_ROUTE}--3a",),
|
|
])
|
|
def test_bad_ranges(self, segment_range):
|
|
with pytest.raises(AssertionError):
|
|
_ = SegmentRange(segment_range).seg_idxs
|
|
|
|
@pytest.mark.parametrize("segment_range, api_call", [
|
|
(f"{TEST_ROUTE}/0", False),
|
|
(f"{TEST_ROUTE}/:2", False),
|
|
(f"{TEST_ROUTE}/0:", True),
|
|
(f"{TEST_ROUTE}/-1", True),
|
|
(f"{TEST_ROUTE}", True),
|
|
])
|
|
def test_slicing_api_call(self, mocker, segment_range, api_call):
|
|
max_seg_mock = mocker.patch("openpilot.tools.lib.route.get_max_seg_number_cached")
|
|
max_seg_mock.return_value = NUM_SEGS
|
|
_ = SegmentRange(segment_range).seg_idxs
|
|
assert api_call == max_seg_mock.called
|
|
|
|
@pytest.mark.slow
|
|
def test_modes(self):
|
|
qlog_len = len(list(LogReader(f"{TEST_ROUTE}/0", ReadMode.QLOG)))
|
|
rlog_len = len(list(LogReader(f"{TEST_ROUTE}/0", ReadMode.RLOG)))
|
|
|
|
assert qlog_len * 6 < rlog_len
|
|
|
|
@pytest.mark.slow
|
|
def test_modes_from_name(self):
|
|
qlog_len = len(list(LogReader(f"{TEST_ROUTE}/0/q")))
|
|
rlog_len = len(list(LogReader(f"{TEST_ROUTE}/0/r")))
|
|
|
|
assert qlog_len * 6 < rlog_len
|
|
|
|
@pytest.mark.slow
|
|
def test_list(self):
|
|
qlog_len = len(list(LogReader(f"{TEST_ROUTE}/0/q")))
|
|
qlog_len_2 = len(list(LogReader([f"{TEST_ROUTE}/0/q", f"{TEST_ROUTE}/0/q"])))
|
|
|
|
assert qlog_len * 2 == qlog_len_2
|
|
|
|
@pytest.mark.slow
|
|
def test_multiple_iterations(self, mocker):
|
|
init_mock = mocker.patch("openpilot.tools.lib.logreader._LogFileReader")
|
|
lr = LogReader(f"{TEST_ROUTE}/0/q")
|
|
qlog_len1 = len(list(lr))
|
|
qlog_len2 = len(list(lr))
|
|
|
|
# ensure we don't create multiple instances of _LogFileReader, which means downloading the files twice
|
|
assert init_mock.call_count == 1
|
|
|
|
assert qlog_len1 == qlog_len2
|
|
|
|
@pytest.mark.slow
|
|
def test_helpers(self):
|
|
lr = LogReader(f"{TEST_ROUTE}/0/q")
|
|
assert lr.first("carParams").carFingerprint == "SUBARU OUTBACK 6TH GEN"
|
|
assert 0 < len(list(lr.filter("carParams"))) < len(list(lr))
|
|
|
|
@parameterized.expand([(True,), (False,)])
|
|
@pytest.mark.slow
|
|
def test_run_across_segments(self, cache_enabled):
|
|
os.environ["FILEREADER_CACHE"] = "1" if cache_enabled else "0"
|
|
lr = LogReader(f"{TEST_ROUTE}/0:4")
|
|
assert len(lr.run_across_segments(4, noop)) == len(list(lr))
|
|
|
|
@pytest.mark.slow
|
|
def test_auto_mode(self, subtests, mocker):
|
|
lr = LogReader(f"{TEST_ROUTE}/0/q")
|
|
qlog_len = len(list(lr))
|
|
log_paths_mock = mocker.patch("openpilot.tools.lib.route.Route.log_paths")
|
|
log_paths_mock.return_value = [None] * NUM_SEGS
|
|
# Should fall back to qlogs since rlogs are not available
|
|
|
|
with subtests.test("interactive_yes"):
|
|
mocker.patch("sys.stdin", new=io.StringIO("y\n"))
|
|
lr = LogReader(f"{TEST_ROUTE}/0", default_mode=ReadMode.AUTO_INTERACTIVE, default_source=comma_api_source)
|
|
log_len = len(list(lr))
|
|
assert qlog_len == log_len
|
|
|
|
with subtests.test("interactive_no"):
|
|
mocker.patch("sys.stdin", new=io.StringIO("n\n"))
|
|
with pytest.raises(AssertionError):
|
|
lr = LogReader(f"{TEST_ROUTE}/0", default_mode=ReadMode.AUTO_INTERACTIVE, default_source=comma_api_source)
|
|
|
|
with subtests.test("non_interactive"):
|
|
lr = LogReader(f"{TEST_ROUTE}/0", default_mode=ReadMode.AUTO, default_source=comma_api_source)
|
|
log_len = len(list(lr))
|
|
assert qlog_len == log_len
|
|
|
|
@pytest.mark.parametrize("is_internal", [True, False])
|
|
@pytest.mark.slow
|
|
def test_auto_source_scenarios(self, mocker, is_internal):
|
|
lr = LogReader(QLOG_FILE)
|
|
qlog_len = len(list(lr))
|
|
|
|
with setup_source_scenario(mocker, is_internal=is_internal):
|
|
lr = LogReader(f"{TEST_ROUTE}/0/q")
|
|
log_len = len(list(lr))
|
|
assert qlog_len == log_len
|
|
|
|
@pytest.mark.slow
|
|
def test_sort_by_time(self):
|
|
msgs = list(LogReader(f"{TEST_ROUTE}/0/q"))
|
|
assert msgs != sorted(msgs, key=lambda m: m.logMonoTime)
|
|
|
|
msgs = list(LogReader(f"{TEST_ROUTE}/0/q", sort_by_time=True))
|
|
assert msgs == sorted(msgs, key=lambda m: m.logMonoTime)
|
|
|
|
def test_only_union_types(self):
|
|
with tempfile.NamedTemporaryFile() as qlog:
|
|
# write valid Event messages
|
|
num_msgs = 100
|
|
with open(qlog.name, "wb") as f:
|
|
f.write(b"".join(capnp_log.Event.new_message().to_bytes() for _ in range(num_msgs)))
|
|
|
|
msgs = list(LogReader(qlog.name))
|
|
assert len(msgs) == num_msgs
|
|
[m.which() for m in msgs]
|
|
|
|
# append non-union Event message
|
|
event_msg = capnp_log.Event.new_message()
|
|
non_union_bytes = bytearray(event_msg.to_bytes())
|
|
non_union_bytes[event_msg.total_size.word_count * 8] = 0xff # set discriminant value out of range using Event word offset
|
|
with open(qlog.name, "ab") as f:
|
|
f.write(non_union_bytes)
|
|
|
|
# ensure new message is added, but is not a union type
|
|
msgs = list(LogReader(qlog.name))
|
|
assert len(msgs) == num_msgs + 1
|
|
with pytest.raises(capnp.KjException):
|
|
[m.which() for m in msgs]
|
|
|
|
# should not be added when only_union_types=True
|
|
msgs = list(LogReader(qlog.name, only_union_types=True))
|
|
assert len(msgs) == num_msgs
|
|
[m.which() for m in msgs]
|
|
|