[Webkit-unassigned] [Bug 44920] Add layout tests for FileSystem API

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Sep 29 13:46:13 PDT 2010


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





--- Comment #9 from Kinuko Yasuda <kinuko at chromium.org>  2010-09-29 13:46:12 PST ---
(In reply to comment #7)
> (From update of attachment 67987 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=67987&action=review
> 
> All the results generally look like "8" is "8"... can these have a little more context in them. right now they're impossible for a human to parse
> tests should be as such
> shouldBe("controller.operation.result", "true");
> so the results are understandable when something breaks

Updated the tests and expectations to make them contain more informative messages.

> > LayoutTests/fast/filesystem/script-tests/simple-persistent.js:13
> > +    shouldBeTrue("true");
> this looks meaningless

Fixed.

> > LayoutTests/fast/filesystem/simple-temporary-expected.txt:7
> > +WARN: shouldBe() expects string arguments
> this looks wrong

Fixed.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list