gl-3.2-depth-tex-sampling: Check return value of piglit_check_gl_error.

Fixes "Unchecked return value" defects reported by Coverity.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
framework/shader_test.py: Remove standalone capability

Shader_test has a bizarre capability to be run as a standalone script as
well as be used as a module. This was probably useful when developing
the original shader_test implementation, but at this point it's unused
code just waiting to bit-rot adding needless complexity
framework/shader_test.py: Replace ShaderTest class with function

This replaces the ShaderTest class with a shader_test function that
returns a PlainExecTest instance. The reason to make this transition is
simple, ShaderTest never actually modified any of the methods in
PlainExecTest, it never needed to be a class, making it a function
reduces the amount of code required, vastly reduces the complexity of
the code, and makes testing much simpler.

This also removes the dmesg_test for ShaderTest, since shader_test just
returns a PlainExecTest, there isn't any reason for dmesg to test it
specially.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
shader_tests_tests.py: Adds some tests for the shader_test module

This adds a couple of tests for the regex searches in
shader_test.shader_test(), and some simple tests for the
add_shader_test* functions.

Signed-off-by: Dylan Baker <baker.dylan.c@gmail.com>
Replace end_config with 'end config' in glsl parser tests

The example glsl parser test doesn't list end_config as acceptable, and
there are only 5 tests using end_config, so it makes sense to just fix
those tests rather than generating slower more complicated regex for
handling end_config.
glsl_parser_test.py: Remove standalone functionality.

This removes the standalone run functionality from glsl_parser_test.
Like shader_test this probably made sense as a development option, but
at this point it's just dead code.
glsl_parser_test.py: Replace GLSLParserTest with function

This is a rough 'port' of the class to a function. It deviates only to
make idiomatic changes and to remove references to self. With a few
exceptions. The class always returned a result when something went
wrong, this function raises an exception. Some of these cases can
probably be better handled, but returning a result seems like the wrong
decision, these should either be handled or excepted.
glsl_parser_test.py: Vastly simplify the way the config is parsed

This reduces the complexity of the code that searches for config section
and writes it into a StringIO. The new code just looks for a comment
followed by [config] to open the section and a comment followed by [end
config] to close it. It naively slurps up everything in between.
Adds some tests for glsl_parser_tests.py
exectest.py: Merge GleanTest into exectest

It doesn't really make sense to have a dozen test modules.
exectest.py: Code cleanup and refactoring

Cleans up and refactor some code, mostly to meet style guidelines or
remove deprecated functions like filter() and map(). There are also a
few cases or replacing small loops with list comprehensions.
exectest.py: Move Test baseclass from core

Since Test is only used as a base class by ExecTest it makes sense to
put both of them in the same module.
exectest.py: move testBinDir from core to exectest

Since this exists mainly to be used by test classes it should live in
their module.
Remove unused shell script

This is just some unused legacy code I found while greping through the
tree.
exectest.py merge shader_test

Shader tests are just a wrapper for PlainExecTest, so lets put them in
the same module.
exectest.py: merge glsl_parser_test

This is also just a wrapper for PlainExecTest, so just merge it into
exectest.
exectest.py: Refactor Test, ExecTest and PlainExecTest

Piglit has this strange three class inheritance scheme, where Test and
ExecTest serve purely as abstract base classes, however no test inherits
directly from Test, they all inherit from ExecTest.

What this patch does is merge Test and ExecTest into a single abstract
base class, Test, then renames PlainExecTest to PiglitTest (since it's
all about running native piglit tests), it then refactors other tests to
inherit from Test instead of ExecTest.
Replace testBinDir with pep8 compliant TEST_BIN_DIR
googletest.py: Refactor several test modules into one

Previously there was a module for gtest (short for googletest), and one
for opencv. Since there might be further use for googletest, but
probably not any additional children of opencv it makes sense to
refactor this to be about googletest.