Return a lit.Test.Result object from TestRunner's executeShTest()

[dim]

Summary

For various clang analyzer tests, which were unsupported, I got lit
exceptions, similar to the following:

Exception during script execution:
Traceback (most recent call last):
  File "utils/lit/lit/run.py", line 190, in execute_test
    result = test.config.test_format.execute(test, lit_config)
  File "tools/clang/test/Analysis/analyzer_test.py", line 11, in execute
    if result.code == lit.Test.FAIL:
AttributeError: 'tuple' object has no attribute 'code'

This is because executeShTest() in utils/lit/lit/TestRunner.py is
supposed to return a lit.Test.Result object, but in case of unsupported
tests, it returns a plain tuple.

Fix this by returning a properly initialized lit.Test.Result object
instead.

[rnk]

lgtm

[MatzeB]

Looks like an oversight in r189545. I am surprised this wasn't noticed before.

utils/lit/lit/TestRunner.py

if err:
         output += """Command Output (stderr):\n--\n%s\n--\n""" % (err,)
 
     return lit.Test.Result(status, output)
 
 
 def executeShTest(test, litConfig, useExternalSh,
                   extra_substitutions=[]):
     if test.config.unsupported:
-        return (Test.UNSUPPORTED, 'Test is unsupported')
+        return lit.Test.Result(Test.UNSUPPORTED, 'Test is unsupported')
 
     script = parseIntegratedTestScript(test)
     if isinstance(script, lit.Test.Result):
         return script
     if litConfig.noExecute:
         return lit.Test.Result(Test.PASS)
 
     tmpDir, tmpBase = getTempPaths(test)