tests/test_shortcuts: Fix check for success on submit_results return

[katieworton]

Since submit_results now returns two values (success and testrun ID), the test_basic for submit_results needs to be updated to reflect this, otherwise the assertion will always return True.

[katieworton]

@chaws just noticed a small change I missed with the unit test updates for changes to submit_results.

[chaws]

Good catch!

Just for future references, the test still managed to go through even when submit_results returned two values, instead of a single value. It turns out that when you invoke self.assertTrue(arg), it's the same as saying bool(arg) is True.

In [1]: bool(False) # would've caught test failures
Out[1]: False

In [2]: bool((False, "test-run-id")) # Python evaluates non-empty tuples as true, thus masking `False`
Out[2]: True