Bug 222359 - [ews] Make unit-test compatible with latest buildbot
Summary: [ews] Make unit-test compatible with latest buildbot
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-02-24 07:50 PST by Aakash Jain
Modified: 2021-02-24 09:26 PST (History)
4 users (show)

See Also:


Attachments
Patch (2.12 KB, patch)
2021-02-24 07:55 PST, Aakash Jain
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2021-02-24 07:50:07 PST
EWS unit-tests currently work with Buildbot v1.8.2. But on newer buildbot (e.g.: v2.8.4) they fail with error:

======================================================================
ERROR: test_unexpected_failure (steps_unittest.TestWebKitPyPython3Tests)
test_unexpected_failure
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Library/Python/3.7/site-packages/twisted/internet/defer.py", line 151, in maybeDeferred
    result = f(*args, **kw)
  File "/Library/Python/3.7/site-packages/twisted/internet/utils.py", line 221, in runWithWarningsSuppressed
    reraise(exc_info[1], exc_info[2])
  File "/Library/Python/3.7/site-packages/twisted/python/compat.py", line 464, in reraise
    raise exception.with_traceback(traceback)
  File "/Library/Python/3.7/site-packages/twisted/internet/utils.py", line 217, in runWithWarningsSuppressed
    result = f(*a, **kw)
  File "Tools/CISupport/ews-build/steps_unittest.py", line 542, in setUp
    return self.setUpBuildStep()
  File "Tools/CISupport/ews-build/steps_unittest.py", line 114, in setUpBuildStep
    super(BuildStepMixinAdditions, self).setUpBuildStep()
  File "/Library/Python/3.7/site-packages/buildbot/test/util/steps.py", line 119, in setUpBuildStep
    raise Exception('Reactor has not yet been setup for step')
Exception: Reactor has not yet been setup for step


We should make the unit-tests compatible with newer version of buildbot.
Comment 1 Aakash Jain 2021-02-24 07:50:16 PST
This is because of recent change in Buildbot in https://github.com/buildbot/buildbot/commit/16c2c931a0 (in v2.8.0)
Comment 2 Aakash Jain 2021-02-24 07:55:38 PST
Created attachment 421410 [details]
Patch
Comment 3 EWS 2021-02-24 09:25:20 PST
Committed r273403: <https://commits.webkit.org/r273403>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 421410 [details].
Comment 4 Radar WebKit Bug Importer 2021-02-24 09:26:15 PST
<rdar://problem/74698904>