WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
233241
Convert error to string before passing to testFailed() in FileSystemAccess layout tests
https://bugs.webkit.org/show_bug.cgi?id=233241
Summary
Convert error to string before passing to testFailed() in FileSystemAccess la...
Sihui Liu
Reported
2021-11-16 23:04:13 PST
...
Attachments
Patch
(15.59 KB, patch)
2021-11-16 23:10 PST
,
Sihui Liu
no flags
Details
Formatted Diff
Diff
Patch for landing
(15.57 KB, patch)
2021-11-28 11:42 PST
,
Sihui Liu
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Sihui Liu
Comment 1
2021-11-16 23:10:24 PST
Created
attachment 444483
[details]
Patch
youenn fablet
Comment 2
2021-11-18 05:41:00 PST
Comment on
attachment 444483
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=444483&action=review
> LayoutTests/ChangeLog:8 > + testFailed() in js-test.js expects string, but finishTest() may receive Error object as parameter.
Proposed approach is fine, but why not updating testFailed to pretty print Error itself?
> LayoutTests/storage/filesystemaccess/resources/shared.js:8 > + }
We usually do not need { } for one liner if.
Radar WebKit Bug Importer
Comment 3
2021-11-23 23:05:22 PST
<
rdar://problem/85718910
>
Sihui Liu
Comment 4
2021-11-28 11:42:20 PST
Comment on
attachment 444483
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=444483&action=review
>> LayoutTests/ChangeLog:8 >> + testFailed() in js-test.js expects string, but finishTest() may receive Error object as parameter. > > Proposed approach is fine, but why not updating testFailed to pretty print Error itself?
testFailed() has always received string as parameter, and it's symmetric to testPassed(), so I didn't change it in js-test.js. But it may be worthwhile to make the change if we see other tests have the same issue.
>> LayoutTests/storage/filesystemaccess/resources/shared.js:8 >> + } > > We usually do not need { } for one liner if.
Will remove.
Sihui Liu
Comment 5
2021-11-28 11:42:48 PST
Created
attachment 445239
[details]
Patch for landing
EWS
Comment 6
2021-11-28 12:34:42 PST
Committed
r286198
(
244580@main
): <
https://commits.webkit.org/244580@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 445239
[details]
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug