Drop incorrect framebuffer texture subtests.
authorKenneth Graunke <kenneth.w.graunke@intel.com>
Fri, 6 Jan 2017 08:25:28 +0000 (00:25 -0800)
committerPyry Haulos <phaulos@google.com>
Tue, 17 Jan 2017 17:36:12 +0000 (17:36 +0000)
This patch removes two broken subcases of:
dEQP-GLES31.functional.debug.negative_coverage.log.buffer.framebuffer_texture_layer

These were fixed in dEQP master by commit f28187809176f5eeb1ee25ccfa70d3,
but we cannot backport that to release branches because it changes the
existing expectations, which would break previously passing
implementations.  See buganizer 30287182 and 28090252.

The first subtest incorrectly checked layer (array slice) when it should
have checked miplevel.  The subtest description says:

"GL_INVALID_VALUE is generated if texture is a 2D multisample array
 texture and layer not 0."

which is wrong - array textures can certainly have multiple slices.
The ES 3.1 specification says that:

"An INVALID_VALUE error is generated if texture is not zero and level is
 not a supported texture level for texture, as described above."

and miplevel 0 is the only valid miplevel for a multisample texture.

The second subtest is no longer valid according to the resolution of
https://cvs.khronos.org/bugzilla/show_bug.cgi?id=15968.

Bug: 34281627
Merged-In: I500074c1777d88291c0b395498bb0f7d5613d27d

modules/gles31/functional/es31fNegativeBufferApiTests.cpp

index d2bb14f..8ca4a9b 100644 (file)
@@ -1658,18 +1658,6 @@ void framebuffer_texture_layer (NegativeTestContext& ctx)
                ctx.glBindTexture                               (GL_TEXTURE_BUFFER, texBuffer);
                ctx.expectError                                 (GL_NO_ERROR);
 
-               ctx.beginSection("GL_INVALID_VALUE is generated if texture is a 2D multisample array texture and layer not 0.");
-               ctx.glFramebufferTextureLayer   (GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, tex2DMSArray, 0, -1);
-               ctx.expectError                                 (GL_INVALID_VALUE);
-               ctx.glFramebufferTextureLayer   (GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, tex2DMSArray, 0, 1);
-               ctx.expectError                                 (GL_INVALID_VALUE);
-               ctx.endSection();
-
-               ctx.beginSection("GL_INVALID_VALUE is generated if texture is a cube map array texture and layer is larger than the value of MAX_CUBE_MAP_TEXTURE_SIZE-1 minus one.");
-               ctx.glFramebufferTextureLayer   (GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, texCube, 0, maxCubeTexSize);
-               ctx.expectError                                 (GL_INVALID_VALUE);
-               ctx.endSection();
-
                ctx.beginSection("GL_INVALID_OPERATION is generated if texture is the name of a buffer texture.");
                ctx.glFramebufferTextureLayer   (GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, texBuffer, 0, 0);
                ctx.expectError                                 (GL_INVALID_OPERATION);