The reason was because two different tests from two different unittest executables need it, but we have no good way to share code across unittests like this.  Open to suggestions though<br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 20, 2017 at 4:02 AM Pavel Labath via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added inline comments.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lldb/include/lldb/Utility/TildeExpressionResolver.h:62<br class="gmail_msg">
+<br class="gmail_msg">
+class MockTildeExpressionResolver : public TildeExpressionResolver {<br class="gmail_msg">
+  llvm::StringRef CurrentUser;<br class="gmail_msg">
----------------<br class="gmail_msg">
I find it very strange to see the Mock object in a regular header file (and I also don't see a reason why we would need that). Could we get rid of this? (perhaps by declaring it in a test-only header file if necessary)<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D31129" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D31129</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>