[Piglit] [PATCH 2/3] Merge quick-driver.tests to quick.tests

[Dylan Baker]

On Tuesday 17 September 2013 23:57:02 Marek Olšák wrote:
> Yes. I added quick-driver.tests. I was making changes to it. Now I'm
> removing it.
> 
> The difference between the two is 5 tests, which is pretty much nothing.
> 
> Marek
> 
> On Tue, Sep 17, 2013 at 5:46 PM, Dylan Baker <baker.dylan.c at gmail.com> 
wrote:
> > On Monday 16 September 2013 20:08:34 Marek Olšák wrote:
> >> ---
> >> 
> >>  tests/quick-driver.tests | 14 --------------
> >>  tests/quick.tests        |  6 ++++++
> >>  2 files changed, 6 insertions(+), 14 deletions(-)
> >>  delete mode 100644 tests/quick-driver.tests
> >> 
> >> diff --git a/tests/quick-driver.tests b/tests/quick-driver.tests
> >> deleted file mode 100644
> >> index f7c1a0f..0000000
> >> --- a/tests/quick-driver.tests
> >> +++ /dev/null
> >> @@ -1,14 +0,0 @@
> >> -# -*- coding: utf-8 -*-
> >> -
> >> -import os.path
> >> -
> >> -global profile
> >> -
> >> -execfile(os.path.dirname(__file__) + '/quick.tests')
> >> -
> >> -# These take too long
> >> -del profile.tests['shaders']['glsl-fs-inline-explosion']
> >> -del profile.tests['shaders']['glsl-fs-unroll-explosion']
> >> -del profile.tests['shaders']['glsl-vs-inline-explosion']
> >> -del profile.tests['shaders']['glsl-vs-unroll-explosion']
> >> -del profile.tests['spec']['!OpenGL 1.1']['streaming-texture-leak']
> >> diff --git a/tests/quick.tests b/tests/quick.tests
> >> index 58c5d41..f840da8 100644
> >> --- a/tests/quick.tests
> >> +++ b/tests/quick.tests
> >> @@ -11,3 +11,9 @@ from framework.gleantest import GleanTest
> >> 
> >>  GleanTest.globalParams += [ "--quick" ]
> >>  
> >>  execfile(os.path.dirname(__file__) + '/all.tests')
> >> 
> >> +
> >> +# These take too long
> >> +del profile.tests['shaders']['glsl-fs-inline-explosion']
> >> +del profile.tests['shaders']['glsl-fs-unroll-explosion']
> >> +del profile.tests['shaders']['glsl-vs-inline-explosion']
> >> +del profile.tests['shaders']['glsl-vs-unroll-explosion']
> > 
> > Could you provide some rational for this change?

Cool. I'm really just concerned that there is something in the git log so that 
we know why we're removing it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20130917/9de0e0ce/attachment.pgp>