[Piglit] [PATCH 1/2] xts: Add a profile for just the rendering tests.

[Dylan Baker]

On Wed, Jul 08, 2015 at 02:38:50PM -0700, Eric Anholt wrote:
> Many X developers work on rendering code, and don't really care about
> the rest of xts.  This is our equivalent of GL's quick.py.
> ---
>  tests/xts-render.py | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>  create mode 100644 tests/xts-render.py
> 
> diff --git a/tests/xts-render.py b/tests/xts-render.py
> new file mode 100644
> index 0000000..c9adece
> --- /dev/null
> +++ b/tests/xts-render.py
> @@ -0,0 +1,32 @@
> +# Copyright © 2015 Broadcom
> +#
> +# Permission is hereby granted, free of charge, to any person obtaining a
> +# copy of this software and associated documentation files (the "Software"),
> +# to deal in the Software without restriction, including without limitation
> +# the rights to use, copy, modify, merge, publish, distribute, sublicense,
> +# and/or sell copies of the Software, and to permit persons to whom the
> +# Software is furnished to do so, subject to the following conditions:
> +#
> +# The above copyright notice and this permission notice (including the next
> +# paragraph) shall be included in all copies or substantial portions of the
> +# Software.
> +#
> +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
> +# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> +# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> +# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> +# IN THE SOFTWARE.
> +
> +from framework import core
> +from framework.profile import load_test_profile
> +
> +def xts_render_filter(path, test):
> +    # All of Xlib9 is for rendering.
> +    return 'xlib9' in path

Two spaces before and after top level functions and classes please

> +
> +profile = load_test_profile("xts")
> +profile.filter_tests(xts_render_filter)
> +
> +__all__ = ['profile']

Generally __all__ comes right after any imports

> -- 
> 2.1.4
> 
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20150708/8b4316a9/attachment.sig>