Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 7c88bdd9 authored by Nolan Scobie's avatar Nolan Scobie
Browse files

Re-enable 3 output dataspace RE tests on Graphite 

These were all fixed with Skia change
I455b95ad3150a28ec92c9a9d236890f15782ba3d ("[graphite] Implement
RuntimeEffect toLinearSrgb/fromLinearSrgb handling")

Bug: b/331445583
Bug: b/331447131
Bug: b/331446495
Change-Id: If5d9a119ec18eb4e744543e37b89f3c69489cdd8
Test: librenderengine_test
parent efd55b1a

libs/renderengine/tests/RenderEngineTest.cpp

+0 −15
Original line number Diff line number Diff line
@@ -1687,11 +1687,6 @@ TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransform_sourceDataspace) { 
}



TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransform_outputDataspace) {

    // TODO: b/331445583 - Fix in Graphite and re-enable.

    if (GetParam()->skiaBackend() == renderengine::RenderEngine::SkiaBackend::GRAPHITE) {

        GTEST_SKIP();

    }



    const auto& renderEngineFactory = GetParam();

    // skip for non color management

    if (!renderEngineFactory->apiSupported()) {
@@ -1842,11 +1837,6 @@ TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransformAndSourceDataspace_o 
}



TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransformAndOutputDataspace_opaqueBufferSource) {

    // TODO: b/331447131 - Fix in Graphite and re-enable.

    if (GetParam()->skiaBackend() == renderengine::RenderEngine::SkiaBackend::GRAPHITE) {

        GTEST_SKIP();

    }



    const auto& renderEngineFactory = GetParam();

    // skip for non color management

    if (!renderEngineFactory->apiSupported()) {
@@ -1997,11 +1987,6 @@ TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransformAndSourceDataspace_b 
}



TEST_P(RenderEngineTest, drawLayers_fillBufferColorTransformAndOutputDataspace_bufferSource) {

    // TODO: b/331446495 - Fix in Graphite and re-enable.

    if (GetParam()->skiaBackend() == renderengine::RenderEngine::SkiaBackend::GRAPHITE) {

        GTEST_SKIP();

    }



    const auto& renderEngineFactory = GetParam();

    // skip for non color management

    if (!renderEngineFactory->apiSupported()) {