Bug 195557

Summary: [ews-build] unit-tests fail when passwords.json is missing
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ap, commit-queue, lforschler, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=180390
Attachments:
Description Flags
Proposed patch none

Description Aakash Jain 2019-03-11 08:25:06 PDT
unit-tests in loadConfig_unittest.py fail when passwords.json is missing. This file is not checked-in to the repository. So the unit-tests should be able to run without this file.

Exception:


[~/OpenSource/Tools/BuildSlaveSupport/ews-build]$python runUnittests.py 
......................................................E....................
======================================================================
ERROR: test_configuration (loadConfig_unittest.ConfigDotJSONTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/jain/code/OpenSource/Tools/BuildSlaveSupport/ews-build/loadConfig_unittest.py", line 36, in test_configuration
    loadConfig.loadBuilderConfig({}, master_prefix_path=cwd)
  File "/Users/jain/code/OpenSource/Tools/BuildSlaveSupport/ews-build/loadConfig.py", line 42, in loadBuilderConfig
    passwords = json.load(open(os.path.join(master_prefix_path, 'passwords.json')))
IOError: [Errno 2] No such file or directory: 'passwords.json'
Comment 1 Aakash Jain 2019-03-11 08:29:34 PDT
Created attachment 364253 [details]
Proposed patch
Comment 2 WebKit Commit Bot 2019-03-11 09:38:11 PDT
Comment on attachment 364253 [details]
Proposed patch

Clearing flags on attachment: 364253

Committed r242707: <https://trac.webkit.org/changeset/242707>
Comment 3 WebKit Commit Bot 2019-03-11 09:38:13 PDT
All reviewed patches have been landed.  Closing bug.
Comment 4 Radar WebKit Bug Importer 2019-03-11 09:39:23 PDT
<rdar://problem/48769818>