ScopedThread's join() method should throw if an exception was thrown in the thread

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Duplicate
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Shell
    • None
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      Almost all tests using ScopedThreads call join() on the thread, but few if any call hasFailed(). Without checking the result of hasFailed() on the thread, it's possible that an uncaught exception was thrown from the thread but the test can still pass. This may be hiding existing test failures.

      If we don't do this, we should at least audit all users of ScopedThread and make sure they're handing errors from within the thread appropriately.

            Assignee:
            DO NOT USE - Backlog - Platform Team
            Reporter:
            Spencer Brody (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: