[Webkit-unassigned] [Bug 53756] Refactor and/or split and/or rename the extend-selection tests to be clearer what they're doing, and able to be run manually

Thu Feb 3 22:41:08 PST 2011

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

Ryosuke Niwa <rniwa at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1

--- Comment #6 from Ryosuke Niwa <rniwa at webkit.org>  2011-02-03 22:41:08 PST ---
(In reply to comment #4)
> > Making tests manually runnable is good but all these rewrites of script isn't that great.  It's really hard to see that new tests are testing the same things the old tests were testing.  In general, we must be careful when modifying tests because subtle differences may affect what tests do.  At least, we should separate refactoring and changes to make it manually runnable.
> 
> If you look at the changes to the expected results it's pretty clear that it's testing the same thing.

Sure expected tests are same but extend-selection-* tests relies heavily on scripts so I wouldn't modify like this all at once.

> > I don't think separating the line boundary tests into LTR/RTL tests is a good idea because having both results on one test has helped us catching regressions in either result since extend-selection-* tests are written so that LTR/RTL results should match.
> 
> The reason I split those into LTR and RTL tests is because it's impossible to run them manually without doing that.  The transition from LTR to RTL actually modifies the dom mid-test.

We need to find a better way of testing both LTR and RTL cases then.  I don't think it's acceptable to separate LTR/RTL tests; we'll have regressions in no time if we did that.

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