RESOLVED FIXED 183171
Rename samplePendingBuild to samplePendingBuildRequest 
https://bugs.webkit.org/show_bug.cgi?id=183171
Summary Rename samplePendingBuild to samplePendingBuildRequest
Aakash Jain
Reported 2018-02-27 12:29:28 PST
Websites/perf.webkit.org/unit-tests/buildbot-syncer-tests.js contains methods samplePendingBuild and samplePendingBuildDeprecated. These are poorly named functions as they actually represent a pending buildrequest. We should rename them to samplePendingBuildRequest and samplePendingBuildRequestDeprecated.
Attachments
Proposed patch (6.62 KB, patch)
2018-02-27 12:31 PST, Aakash Jain 		rniwa: review+
DetailsFormatted DiffDiff
Patch for landing (6.63 KB, patch)
2018-02-28 12:16 PST, Aakash Jain 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2018-02-27 12:31:30 PST
Created attachment 334700 [details] Proposed patch
Ryosuke Niwa
Comment 2 2018-02-28 01:50:25 PST
Comment on attachment 334700 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=334700&action=review > Websites/perf.webkit.org/unit-tests/buildbot-syncer-tests.js:318 > +function samplePendingBuildRequest(buildRequestId, buildTime, builderName) Should be samplePendingBuildRequests since this function returns a list of build requests even though it always contains one.
Aakash Jain
Comment 3 2018-02-28 12:16:42 PST
Created attachment 334757 [details] Patch for landing
Aakash Jain
Comment 4 2018-02-28 12:17:39 PST
> Should be samplePendingBuildRequests since this function returns a list of > build requests even though it always contains one. ok. Renamed.
WebKit Commit Bot
Comment 5 2018-02-28 13:55:01 PST
Comment on attachment 334757 [details] Patch for landing Clearing flags on attachment: 334757 Committed r229101: <https://trac.webkit.org/changeset/229101>
WebKit Commit Bot
Comment 6 2018-02-28 13:55:03 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 7 2018-02-28 13:55:42 PST
<rdar://problem/37999819>
Note You need to log in before you can comment on or make changes to this bug.