This is an archive of the discontinued LLVM Phabricator instance.

[dexter] Fix source-root-dir unittests on Windows
ClosedPublic

Authored by Orlando on Dec 8 2021, 6:12 AM.

Details

Summary

These tests were spuriously failing on Windows due to path separators getting
flipped from / to \\ in various parts of dexter on Windows.

test_add_breakpoint_with_source_root_dir (debugger.DebuggerBase.TestDebuggerBase) ... FAIL
test_get_step_info (debugger.DebuggerBase.TestDebuggerBase) ... FAIL
test_get_step_info_no_source_root_dir (debugger.DebuggerBase.TestDebuggerBase) ... FAIL

Tested on Windows and Linux.

Patch written by @TWeaver.

Diff Detail

Event Timeline

Orlando requested review of this revision.Dec 8 2021, 6:12 AM
Orlando created this revision.
jmorse accepted this revision.Dec 8 2021, 6:51 AM

LGTM, and thanks for testing both platforms.

This revision is now accepted and ready to land.Dec 8 2021, 6:51 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptDec 8 2021, 7:43 AM