Add new table layout unit tests

Authored by danders on Jan 8 2019, 12:37 PM.



Adds multiple areas capabilities and tests with merged cells

Diff Detail

R8 Calligra
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
danders created this revision.Jan 8 2019, 12:37 PM
Restricted Application added a project: Calligra: 3.0. · View Herald TranscriptJan 8 2019, 12:37 PM
Restricted Application added a subscriber: Calligra-Devel-list. · View Herald Transcript
danders requested review of this revision.Jan 8 2019, 12:37 PM

Not trivial (for me) to get data out of layout so it is hard to create robust tests.
I don't think I have been able to create a test to simulate the loop excactly.
Also, uncertain wether the MockRootAreaProvider is sufficient.
Any ideas/comments wellcome.

Looks like a good first start to me - do they pass currently? - if so then i think they should be pushed

I'll think about further tests

No, all do not pass, generally when the first row is merged, layout fails.
(I am trying to provoke problems similar to bug 381341 that we have tried to fix in D15428.)

danders updated this revision to Diff 49141.Jan 10 2019, 10:16 AM

Add a test that trigger a loop in the table layout

Even if they do not all pass, I think they should be commited if the tests/methods are ok and the reason for not passing are genuin bugs, no?

This revision was not accepted when it landed; it landed in state Needs Review.Jan 15 2019, 11:06 AM
This revision was automatically updated to reflect the committed changes.