This is an archive of the discontinued LLVM Phabricator instance.

Convert TestCommandRegex to lit (from expect)
ClosedPublic

Authored by davide on Feb 6 2019, 12:53 PM.

Diff Detail

Event Timeline

davide created this revision.Feb 6 2019, 12:53 PM
Herald added a project: Restricted Project. · View Herald Transcript
davide marked an inline comment as done.Feb 6 2019, 12:54 PM
davide added inline comments.
lldb/packages/Python/lldbsuite/test/tools/lldb-server/TestAppleSimulatorOSType.py
18 ↗(On Diff #185620)

Ignore this bit, I already pushed it.

davide updated this revision to Diff 185621.Feb 6 2019, 12:55 PM

update with the correct patch.

davide retitled this revision from [Py3/TestAppleOSSimulator] Another byte<->str interoperability issue. to Convert TestCommandRegex to lit (from expect).Feb 6 2019, 12:55 PM
zturner accepted this revision.Feb 6 2019, 1:42 PM

Can you rename these two tests to command-regex-delete.test and command-regex-unalias.test? Otherwise, lgtm

This revision is now accepted and ready to land.Feb 6 2019, 1:42 PM
davide updated this revision to Diff 185633.Feb 6 2019, 1:46 PM

zturner's update.

davide updated this revision to Diff 185634.Feb 6 2019, 1:47 PM

jonas was right

JDevlieghere accepted this revision.Feb 6 2019, 1:47 PM

LGTM, thanks!

This revision was automatically updated to reflect the committed changes.