Time and time again, we've seen regressions being introduced to Quick Edit. I don't want to know how many hours of manual testing Jesse Beach, Théodore (nod) Biadala and I have done.
Back when Quick Edit was written, Drupal 8 (and its testing infrastructure) weren't capable of executing JS. So we could not write tests. Earlier this year, a few months ago, support for that was added.
was committed yesterday. It introduced a new in-place editor plugin. It also introduced JS test coverage for that particular in-place editor. It's time we also add JS test coverage for the rest of Quick Edit. That'll allow us to add features with confidence, but also fix bugs with confidence.
For example,had an RTBC patch, which I un-RTBC'd. Because it should have test coverage. But without a body of tests to extend, that's difficult and even unreasonable. So I postponed it on this issue.
19 of the 36 open Quick Edit issues are bug reports. That's more than 50%. Adding test coverage for each (or at least most) of them would make Quick Edit infinitely more maintainable. This issue makes that much more feasible.
|#45||test-only-clean.patch||38.13 KB||Wim Leers|
|#45||test-only_18.patch||38.4 KB||Wim Leers|
|#37||2828528-37-8.2.x.patch||28.06 KB||Wim Leers|
|#37||2828528-37-8.3.x.patch||38.68 KB||Wim Leers|