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

Marek Olšák maraeo at gmail.com
Tue Sep 17 14:57:02 PDT 2013


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?


More information about the Piglit mailing list