Correct error message for matrix length (#9956)

Author: radarhere

Tests/test_image_convert.py

@@ -46,7 +46,7 @@ def convert(im: Image.Image, mode: str) -> None:
 
 def test_unsupported_conversion() -> None:
     im = hopper()
-    with pytest.raises(ValueError):
+    with pytest.raises(ValueError, match="image has wrong mode"):
         im.convert("INVALID")
 
 
@@ -305,26 +305,33 @@ def test_matrix_illegal_conversion() -> None:
     assert im.mode != "RGB"
 
     # Act / Assert
-    with pytest.raises(ValueError):
+    with pytest.raises(ValueError, match="illegal conversion"):
         im.convert(mode="CMYK", matrix=rgb2xyz_matrix)
 
 
 def test_matrix_wrong_mode() -> None:
     # Arrange
     im = hopper("L")
-    assert im.mode == "L"
 
     # Act / Assert
-    with pytest.raises(ValueError):
+    with pytest.raises(ValueError, match="image has wrong mode"):
         im.convert(mode="L", matrix=rgb2xyz_matrix)
 
 
+def test_matrix_truncated() -> None:
+    # Arrange
+    im = hopper()
+
+    # Act / Assert
+    with pytest.raises(TypeError, match="matrix must be tuple of length 4 or 12"):
+        im.convert(mode="L", matrix=(0,))
+
+
 @pytest.mark.parametrize("mode", ("RGB", "L"))
 def test_matrix_xyz(mode: str) -> None:
     # Arrange
     im = hopper("RGB")
     im.info["transparency"] = (255, 0, 0)
-    assert im.mode == "RGB"
 
     # Act
     # Convert an RGB image to the CIE XYZ colour space
@@ -350,7 +357,6 @@ def test_matrix_identity() -> None:
         0, 1, 0, 0,
         0, 0, 1, 0,
     )  # fmt: skip
-    assert im.mode == "RGB"
 
     # Act
     # Convert with an identity matrix

src/_imaging.c

@@ -1078,6 +1078,7 @@ _convert_matrix(ImagingObject *self, PyObject *args) {
                 m + 10,
                 m + 11
             )) {
+            PyErr_SetString(PyExc_TypeError, "matrix must be tuple of length 4 or 12");
             return NULL;
         }
     }