Skip to content

Commit

Permalink
Suppress flaky Win/AMD/GL end2end tests.
Browse files Browse the repository at this point in the history
These tests became flaky when google test retries are turned off.

BUG=chromium:669196
BUG=angleproject:1643
BUG=angleproject:1541

Change-Id: I7cc10f532662928e5436a4e88a42fc109c00bfe8
Reviewed-on: https://chromium-review.googlesource.com/414216
Reviewed-by: Jamie Madill <[email protected]>
Commit-Queue: Jamie Madill <[email protected]>
  • Loading branch information
null77 authored and Commit Bot committed Nov 29, 2016
1 parent 0107425 commit 51f75c5
Show file tree
Hide file tree
Showing 2 changed files with 35 additions and 2 deletions.
32 changes: 32 additions & 0 deletions src/tests/gl_tests/PointSpritesTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,14 @@ TEST_P(PointSpritesTest, PointCoordAndPointSizeCompliance)
return;
}

// TODO(jmadill): Investigate potential AMD driver bug.
// http://anglebug.com/1643
if (IsAMD() && IsDesktopOpenGL() && IsWindows())
{
std::cout << "Test skipped on desktop GL AMD Windows." << std::endl;
return;
}

const std::string fs = SHADER_SOURCE
(
precision mediump float;
Expand Down Expand Up @@ -154,6 +162,14 @@ TEST_P(PointSpritesTest, PointCoordAndPointSizeCompliance)
// https://www.khronos.org/registry/webgl/sdk/tests/conformance/rendering/point-no-attributes.html
TEST_P(PointSpritesTest, PointWithoutAttributesCompliance)
{
// TODO(jmadill): Investigate potential AMD driver bug.
// http://anglebug.com/1643
if (IsAMD() && IsDesktopOpenGL() && IsWindows())
{
std::cout << "Test skipped on desktop GL AMD Windows." << std::endl;
return;
}

// clang-format off
const std::string fs = SHADER_SOURCE
(
Expand Down Expand Up @@ -191,6 +207,14 @@ TEST_P(PointSpritesTest, PointWithoutAttributesCompliance)
// https://www.khronos.org/registry/webgl/sdk/tests/conformance/rendering/point-with-gl-pointcoord-in-fragment-shader.html
TEST_P(PointSpritesTest, PointCoordRegressionTest)
{
// TODO(jmadill): Investigate potential AMD driver bug.
// http://anglebug.com/1643
if (IsAMD() && IsDesktopOpenGL() && IsWindows())
{
std::cout << "Test skipped on desktop GL AMD Windows." << std::endl;
return;
}

const std::string fs = SHADER_SOURCE
(
precision mediump float;
Expand Down Expand Up @@ -276,6 +300,14 @@ TEST_P(PointSpritesTest, PointCoordRegressionTest)
// https://www.khronos.org/registry/webgl/sdk/tests/conformance/rendering/point-size.html
TEST_P(PointSpritesTest, PointSizeEnabledCompliance)
{
// TODO(jmadill): Investigate potential AMD driver bug.
// http://anglebug.com/1643
if (IsAMD() && IsDesktopOpenGL() && IsWindows())
{
std::cout << "Test skipped on desktop GL AMD Windows." << std::endl;
return;
}

const std::string fs = SHADER_SOURCE
(
precision mediump float;
Expand Down
5 changes: 3 additions & 2 deletions src/tests/gl_tests/TimerQueriesTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -311,9 +311,10 @@ TEST_P(TimerQueriesTest, TimeElapsedValidationTest)
// Tests timer queries operating under multiple EGL contexts with mid-query switching
TEST_P(TimerQueriesTest, TimeElapsedMulticontextTest)
{
if (IsAMD() && IsOpenGL() && IsWindows() && IsDebug())
if (IsAMD() && IsOpenGL() && IsWindows())
{
// TODO(jmadill): Figure out why this test is flaky on Win/AMD/OpenGL/Debug.
// TODO(jmadill): Figure out why this test is flaky on Win/AMD/OpenGL.
// http://anglebug.com/1541
std::cout << "Test skipped on Windows AMD OpenGL Debug." << std::endl;
return;
}
Expand Down

0 comments on commit 51f75c5

Please sign in to comment.