Weston test suite

[Pekka Paalanen]

On Sat, 22 Aug 2015 00:00:58 -0700
Bryce Harrington <bryce at osg.samsung.com> wrote:

> On Fri, Aug 21, 2015 at 10:35:34AM +0300, Pekka Paalanen wrote:
> > On Thu, 20 Aug 2015 11:05:39 -0700
> > Bryce Harrington <bryce at osg.samsung.com> wrote:
> > 

> > > (I wish we had some tests with coverage of this chunk of code, so I could
> > > give my usual plug for adding test cases; if we did, it'd probably
> > > pretty easy to add a couple test cases here and give us a bit extra
> > > confidence.)
> > 
> > It's a bit tough in this case, I'm not sure if we can affect what
> > outputs the kernel exposes, so it'd be tied to the test machine.
> 
> No, the way you'd do it is to construct a "mock kernel", and then
> whereever the functionality expects data from the kernel (or is making
> kernel calls), you abstract that so you can slip your mock in instead.
> Sort of the same idea as how we replaced the graphics card / monitor
> with synthetic ones in the headless testing.
> 
> I know it probably sounds like a pain in the ass (and it is), but if
> you're serious about wanting stronger automated testing of Weston, then
> that's the sort of stuff that needs done.  And it's certainly
> understandable to want to put off spending time writing tests, but it's
> the same thing with documentation and look how horribly undocumented
> Wayland is - unless you document and write tests as you go, it ain't
> gonna get done.

Hi,

it's actually good to bring this up right now. We have the old test
suite which is painful to work with, and we have the new test suite
which is seriously lacking to become on par even with the old suite in
terms of helpers.

Jon, what are your plans in continuing with the test suite?

I haven't seen a new patch series in quite a while, and I don't see any
pending in Patchwork either.


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150824/ff441037/attachment-0001.sig>