[PATCH weston v2 0/6] Wayland/Weston Test Extension

[Pekka Paalanen]

On Tue, 11 Dec 2012 23:31:37 -0500
Kristian Høgsberg <hoegsberg at gmail.com> wrote:

> On Fri, Dec 07, 2012 at 01:50:28PM -0800, U. Artie Eoff wrote:
> > From: "U. Artie Eoff" <ullysses.a.eoff at intel.com>
> > 
> > Define a relatively generic protocol for a Wayland Test
> > Extension.  Provide a Weston implementation and use it
> > in the Weston client tests.  Support both module-style
> > and extension-style tests.
> 
> Yup, this is good now.  I've merged the series and I had a look at the
> cases where you use the yield() function.  It's just as Pekka said, in
> some cases (when we map a surface or move it) the transforms aren't
> updated until we repaint.  In that case we can just use a surface
> frame callback to wait for that.  When we receive the frame callback
> we know the compositor has repainted and updated the transform
> matrices and can expect the new geometry to be in effect.

Yes, until we want to test something about a case when frame callbacks
are not supposed to come, like a surface on a different virtual
desktop, outside of all outputs, or during compositor lock-down. But,
that can be solved later.

> Anyway, this is all upstream and we're ready to start writing a lot
> tests now :)
> 
> Kristian
> 
> > U. Artie Eoff (6):
> >   tests: Define and implement a test protocol extension.
> >   tests: Rename weston-test to weston-tests-env
> >   tests: Allow weston-tests-env to process different test types
> >   tests: Convert keyboard-test to new test extension
> >   tests: Convert event-test to new test extension
> >   tests: Convert button-test to new test extension
> > 
> >  protocol/Makefile.am              |   3 +-
> >  protocol/wayland-test.xml         |  55 ++++
> >  tests/.gitignore                  |   6 +
> >  tests/Makefile.am                 |  81 +++++-
> >  tests/button-test.c               | 128 ++-------
> >  tests/event-test.c                | 527 +++++++++++++++++++-------------------
> >  tests/keyboard-test.c             | 139 +++-------
> >  tests/weston-test                 |   4 -
> >  tests/weston-test-client-helper.c | 488 +++++++++++++++++++++++++++++++++++
> >  tests/weston-test-client-helper.h | 109 ++++++++
> >  tests/weston-test-runner.c        | 114 +++++++++
> >  tests/weston-test-runner.h        |  56 ++++
> >  tests/weston-test.c               | 248 ++++++++++++++++++
> >  tests/weston-tests-env            |  12 +
> >  14 files changed, 1485 insertions(+), 485 deletions(-)
> >  create mode 100644 protocol/wayland-test.xml
> >  delete mode 100644 tests/weston-test
> >  create mode 100644 tests/weston-test-client-helper.c
> >  create mode 100644 tests/weston-test-client-helper.h
> >  create mode 100644 tests/weston-test-runner.c
> >  create mode 100644 tests/weston-test-runner.h
> >  create mode 100644 tests/weston-test.c
> >  create mode 100755 tests/weston-tests-env

Woohoo! :-D


Thanks!
- pq