197195 – [ews-build] Add unit tests for CleanWorkingDirectory

RESOLVED FIXED 197195

[ews-build] Add unit tests for CleanWorkingDirectory

https://bugs.webkit.org/show_bug.cgi?id=197195

Summary [ews-build] Add unit tests for CleanWorkingDirectory

Aakash Jain

Reported 2019-04-23 07:01:37 PDT

Add unit tests for CleanWorkingDirectory build step.

Attachments
Patch (2.27 KB, patch) 2019-04-23 07:02 PDT, Aakash Jain	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Aakash Jain

Comment 1 2019-04-23 07:02:59 PDT

Created attachment 368026 [details] Patch

EWS Watchlist

Comment 2 2019-04-23 07:05:46 PDT Comment hidden (obsolete)

Attachment 368026 [details] did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:839: [TestCleanWorkingDirectory.test_success] Undefined variable 'CleanWorkingDirectory' [pylint/E0602] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:846: [TestCleanWorkingDirectory.test_success] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:846: [TestCleanWorkingDirectory.test_success] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:850: [TestCleanWorkingDirectory.test_failure] Undefined variable 'CleanWorkingDirectory' [pylint/E0602] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:858: [TestCleanWorkingDirectory.test_failure] Passing unexpected keyword argument 'state_string' in function call [pylint/E1123] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps_unittest.py:858: [TestCleanWorkingDirectory.test_failure] No value passed for parameter 'status_text' in function call [pylint/E1120] [5] Total errors found: 6 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

WebKit Commit Bot

Comment 3 2019-04-23 10:58:26 PDT

Comment on attachment 368026 [details] Patch Clearing flags on attachment: 368026 Committed r244550: <https://trac.webkit.org/changeset/244550>

WebKit Commit Bot

Comment 4 2019-04-23 10:58:28 PDT

All reviewed patches have been landed. Closing bug.

Radar WebKit Bug Importer

Comment 5 2019-04-23 10:59:18 PDT

<rdar://problem/50136092>

Note You need to log in before you can comment on or make changes to this bug.

Status RESOLVED

Resolution FIXED

Priority P2

Severity Normal

Classification Unclassified

Version Other

Hardware Unspecified

OS Unspecified

Product WebKit

Component Tools / Tests

Assignee

Aakash Jain

Reported

2019-04-23 07:01 PDT

Modified

2019-04-23 10:59 PDT History

CC List

7 users Show

URL

Keywords InRadar

Depends on

Blocks