glean: Remove the getString test.

This test always passes, so all it does is make sure glGetString doesn't
crash.  But essentially every Piglit test that does extension checks
already does that.
glean/glsl1: Delete duplicated built-in function tests.

I've less "cross() function, in-place" because we don't have an
equivalent test for it elsewhere.
glean/glsl1: Delete duplicated logic operator tests.
glean/glsl1: Delete duplicated basic arithmetic tests.
glean/glsl1: Delete duplicated equality/inequality tests.
glean/glsl1: Delete duplicated function tests.
glean/glsl1: Delete duplicate vector relational tests.
glean/glsl1: Delete duplicated comment tests.
glean/glsl1: Delete duplicated matrix tests.
glean/glsl1: Delete duplicated array tests.
glsl-1.50: Add shader tests to verify gl_FragCoord redeclarations

V2: Add few more shader tests and do minor changes in tests added in V1.

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Chris Forbes <chrisf@ijw.co.nz>
glsl-1.50: Add compiler tests to verify gl_FragCoord redeclarations

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Add test to verify overlapping locations of vertex input attributes

It is possible for an application to bind more than one
attribute name to the same location. This is referred to as
aliasing. It is allowed in: OpenGL 2.0 (and above) and
OpenGL ES 2.0. This test varifies that aliasing can be used
successfully in a vertex shader.

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Add compiler tests for overlapping attribute locations

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
fbo-drawbuffers-none: Add the instructions for running subtests

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
fbo-drawbuffers-none: Move the drawing code to piglit_display()

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
fbo-drawbuffers-none: Refactor the code to add a sub test

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
fbo-drawbuffers-none: Add a sub test for fragment shader out variables

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
fbo-drawbuffers-none: Use util function streq() in place of strcmp()

Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
framework/exectest.py: Fix a few pylint warnings

This removes some unreachable code, adds an name definition to the Test
constructor, and comments a bare except. There are only a couple of
cases where a bare except is appropriate, most of the time it is a
problem, but we are using one the few cases where it is okay, we're
logging all exceptions to a file.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>