[Piglit] [PATCH] arb_direct_state_access: fix the execution of two tests
Martin Peres
martin.peres at linux.intel.com
Mon Mar 2 02:48:08 PST 2015
On 02/03/15 12:44, Ilia Mirkin wrote:
> On Mon, Mar 2, 2015 at 5:33 AM, Martin Peres
> <martin.peres at linux.intel.com> wrote:
>> The problem was in all.py where the argument was not specified as an array
>> but rather in the same string as the binary's path.
>>
>> Problem spotted by Laura Ekstrand.
>>
>> Signed-off-by: Martin Peres <martin.peres at linux.intel.com>
>> ---
>> framework/test/base.py | 2 +-
>> tests/all.py | 4 ++--
>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/framework/test/base.py b/framework/test/base.py
>> index efc20cb..49d5eff 100644
>> --- a/framework/test/base.py
>> +++ b/framework/test/base.py
>> @@ -310,7 +310,7 @@ class Test(object):
>> # failed.
>> if e.errno == errno.ENOENT:
>> self.result['result'] = 'skip'
>> - out = "Test executable not found.\n"
>> + out = "Test executable '%s' not found: %s\n" % (e.filename, e.strerror)
> Is the strerror really useful here? Seems like it would just be "File
> not found", which is already in the error message. The filename is a
> nice addition though.
What the heck? This was debugging information. I must have screwed up
when I rebased. The whole hunk should be deleted.
>
>> err = ""
>> returncode = None
>> error = True
>> diff --git a/tests/all.py b/tests/all.py
>> index 0738de7..a2de35c 100644
>> --- a/tests/all.py
>> +++ b/tests/all.py
>> @@ -4008,11 +4008,11 @@ spec['ARB_direct_state_access']['copytexturesubimage'] = PiglitGLTest(['arb_dire
>> spec['ARB_direct_state_access']['texture-errors'] = PiglitGLTest(['arb_direct_state_access-texture-errors'], run_concurrent=True)
>> spec['ARB_direct_state_access']['get-textures'] = PiglitGLTest(['arb_direct_state_access-get-textures'], run_concurrent=True)
>> spec['ARB_direct_state_access']['gettextureimage-formats'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-formats'], run_concurrent=True)
>> -spec['ARB_direct_state_access']['gettextureimage-formats'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-formats init-by-rendering'], run_concurrent=True)
>> +spec['ARB_direct_state_access']['gettextureimage-formats'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-formats', 'init-by-rendering'], run_concurrent=True)
>> spec['ARB_direct_state_access']['gettextureimage-luminance'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-luminance'], run_concurrent=True)
>> spec['ARB_direct_state_access']['gettextureimage-simple'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-simple'], run_concurrent=True)
>> spec['ARB_direct_state_access']['gettextureimage-targets'] = PiglitGLTest(['arb_direct_state_access-gettextureimage-targets'], run_concurrent=True)
>> -spec['ARB_direct_state_access']['compressedtextureimage'] = PiglitGLTest(['arb_direct_state_access-compressedtextureimage GL_COMPRESSED_RGBA_FXT1_3DFX'], run_concurrent=True)
>> +spec['ARB_direct_state_access']['compressedtextureimage'] = PiglitGLTest(['arb_direct_state_access-compressedtextureimage', 'GL_COMPRESSED_RGBA_FXT1_3DFX'], run_concurrent=True)
> Just one space after the , . Also, only the intel driver supports
> FXT1; if possible, it'd be nicer to pick a format that more drivers
> support. But you can do that later.
>
>> spec['ARB_direct_state_access']['getcompressedtextureimage'] = PiglitGLTest(['arb_direct_state_access-getcompressedtextureimage'], run_concurrent=True)
>> spec['ARB_direct_state_access']['texture-storage-multisample'] = PiglitGLTest(['arb_direct_state_access-texture-storage-multisample'], run_concurrent=True)
>> spec['ARB_direct_state_access']['texture-buffer'] = PiglitGLTest(['arb_direct_state_access-texture-buffer'], run_concurrent=True)
>> --
>> 2.3.1
>>
>> _______________________________________________
>> Piglit mailing list
>> Piglit at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/piglit
More information about the Piglit
mailing list