[Piglit] [PATCH] [v3] Add test case on error check on glGetTexImage with type

[jian.j.zhao at intel.com]

From: Jian Zhao <jian.j.zhao at intel.com>

As glGetTexImage in OpenGL 4.2 Reference Pages:
 "GL_INVALID_OPERATION is returned if type is one of GL_UNSIGNED_BYTE_3_3_2,
GL_UNSIGNED_BYTE_2_3_3_REV, GL_UNSIGNED_SHORT_5_6_5,
GL_UNSIGNED_SHORT_5_6_5_REV, or GL_UNSIGNED_INT_10F_11F_11F_REV and format is
not GL_RGB".

v2: change the test name to be more specific and change some code style to be
easier to read (per Brian's suggest).

v3: move the case to the spec directory and rename it to getteximage_error_check.c (per Brian's suggest)

Signed-off-by: Jian Zhao<jian.j.zhao at intel.com>
---
 tests/spec/ext_packed_float/CMakeLists.gl.txt      |    1 +
 .../ext_packed_float/getteximage-error-check.c     |   84 ++++++++++++++++++++
 2 files changed, 85 insertions(+), 0 deletions(-)
 create mode 100644 tests/spec/ext_packed_float/getteximage-error-check.c

diff --git a/tests/spec/ext_packed_float/CMakeLists.gl.txt b/tests/spec/ext_packed_float/CMakeLists.gl.txt
index 78bbcd1..b6f8356 100644
--- a/tests/spec/ext_packed_float/CMakeLists.gl.txt
+++ b/tests/spec/ext_packed_float/CMakeLists.gl.txt
@@ -13,5 +13,6 @@ link_libraries (
 )
 
 add_executable (ext_packed_float-pack pack.c)
+add_executable (getteximage-error-check getteximage-error-check.c)
 
 # vim: ft=cmake:
diff --git a/tests/spec/ext_packed_float/getteximage-error-check.c b/tests/spec/ext_packed_float/getteximage-error-check.c
new file mode 100644
index 0000000..50608fa
--- /dev/null
+++ b/tests/spec/ext_packed_float/getteximage-error-check.c
@@ -0,0 +1,84 @@
+/*
+ * Copyright © 2009 Intel Corporation
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice (including the next
+ * paragraph) shall be included in all copies or substantial portions of the
+ * Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+ * IN THE SOFTWARE.
+ *
+ *
+ */
+
+/** @file getteximage-error-check.c
+ *
+ * Tests on error check on glGetTexImage with type 
+ * GL_UNSIGNED_INT_10F_11F_11F_REV should be GL_INVALID_OPERATION.
+ */
+
+#include "piglit-util.h"
+
+int piglit_width = 100, piglit_height = 100;
+int piglit_window_mode = GLUT_RGBA | GLUT_DOUBLE;
+
+const GLenum formatTypes[] = {
+    GL_RGBA,
+    GL_RGB,
+    GL_RED,
+    GL_GREEN,
+    GL_BLUE,
+    GL_ALPHA,
+    GL_LUMINANCE,
+    GL_LUMINANCE_ALPHA,
+};
+
+
+enum piglit_result
+piglit_display(void)
+{
+	long rcvError, expError = GL_NO_ERROR;
+	GLfloat *pxBuffer;
+	int i;
+
+	pxBuffer = (GLfloat *)malloc(10 * 10 * 10 * 4 * sizeof(GLfloat));
+	if (!pxBuffer)
+		return PIGLIT_FAIL;
+
+	for(i=0; i< ARRAY_SIZE(formatTypes); i++)
+	{
+		glGetTexImage(GL_TEXTURE_2D, 0, formatTypes[i], 
+			    GL_UNSIGNED_INT_10F_11F_11F_REV_EXT, 
+			    pxBuffer);
+		rcvError = glGetError();
+		if ( formatTypes[i] == GL_RGB )
+			expError = GL_NO_ERROR;
+		else 
+			expError = GL_INVALID_OPERATION;
+
+		if (rcvError != expError)
+			return PIGLIT_FAIL;
+	}
+
+	return PIGLIT_PASS;
+}
+
+
+void
+piglit_init(int argc, char **argv)
+{
+	piglit_require_extension("GL_EXT_packed_float");
+	glewInit();
+}
-- 
1.7.3.1