In OverlayEffectTest, handle that RGB on zero alpha is undefined.

[movit] / overlay_effect_test.cpp

diff --git a/overlay_effect_test.cpp b/overlay_effect_test.cpp
index 865d392b73ea2b3db750e7132eb9ef53a5b061cb..d0867a13136e504a82bfa1a5dd3a0df9524186f1 100644 (file)
--- a/overlay_effect_test.cpp
+++ b/overlay_effect_test.cpp
@@ -42,7 +42,7 @@ TEST(OverlayEffectTest, BottomDominatesTopWhenTopIsTransparent) {
        expect_equal(data_a, out_data, 4, 1);
 }
 
-TEST(OverlayEffectTest, ZeroAlphaBecomesAllZero) {
+TEST(OverlayEffectTest, ZeroAlphaRemainsZeroAlpha) {
        float data_a[] = {
                0.0f, 0.25f, 0.5f, 0.0f
        };
@@ -60,7 +60,7 @@ TEST(OverlayEffectTest, ZeroAlphaBecomesAllZero) {
        tester.get_chain()->add_effect(new OverlayEffect(), input1, input2);
        tester.run(out_data, GL_BGRA, COLORSPACE_sRGB, GAMMA_LINEAR);
 
-       expect_equal(expected_data, out_data, 4, 1);
+       EXPECT_FLOAT_EQ(0.0f, expected_data[3]);
 }
 
 // This is tested against what Photoshop does: (255,0,128, 0.25) over (128,255,0, 0.5)