fix deserialization of incorrect types into enums

composer.json

@@ -11,7 +11,7 @@
 	],
 	"require": {
 		"php": "~7.2",
-		"consistence/consistence": "~1.0|~2.0",
+		"consistence/consistence": "^2.0.2",
 		"jms/serializer": "~2.0|~3.0"
 	},
 	"require-dev": {

src/Enum/EnumSerializerHandler.php

@@ -176,6 +176,10 @@ private function deserializeEnumValue(DeserializationVisitorInterface $visitor,
 			return $enumClass::getMultiByEnums($singleEnums);
 		}
 
+		if ($this->isMultiEnumClass($enumClass) && Type::getType($data) !== 'int') {
+			throw new \Consistence\Enum\InvalidEnumValueException($data, $enumClass);
+		}
+
 		return $enumClass::get($this->deserializationVisitType($visitor, $data, $type));
 	}
 
@@ -274,9 +278,14 @@ private function findParameter(array $type, Closure $callback): ?array
 		return ArrayType::findValueByCallback($type['params'], $callback);
 	}
 
+	private function isMultiEnumClass(string $enumClass): bool
+	{
+		return is_a($enumClass, MultiEnum::class, true);
+	}
+
 	private function checkMultiEnum(string $enumClass): void
 	{
-		if (!is_a($enumClass, MultiEnum::class, true)) {
+		if (!$this->isMultiEnumClass($enumClass)) {
 			throw new \Consistence\JmsSerializer\Enum\NotMultiEnumException($enumClass);
 		}
 	}

tests/Enum/EnumSerializerHandlerTest.php

@@ -362,6 +362,63 @@ public function testDeserializeEnumInvalidValueEmbededObject(): void
 		}
 	}
 
+	public function testDeserializeEnumWhenValueIsArray(): void
+	{
+		$serializer = $this->getSerializer();
+
+		try {
+			$serializer->deserialize('{
+				"single_enum": [1, 2, 3]
+			}', User::class, 'json');
+
+			$this->fail('Exception expected.');
+
+		} catch (\Consistence\JmsSerializer\Enum\DeserializationInvalidValueException $e) {
+			$this->assertEquals('single_enum', $e->getFieldPath());
+			$previous = $e->getPrevious();
+			$this->assertInstanceOf(\Consistence\Enum\InvalidEnumValueException::class, $previous);
+			$this->assertEquals([1, 2, 3], $previous->getValue());
+		}
+	}
+
+	public function testDeserializeEnumWhenValueIsObject(): void
+	{
+		$serializer = $this->getSerializer();
+
+		try {
+			$serializer->deserialize('{
+				"single_enum": {"foo": "bar"}
+			}', User::class, 'json');
+
+			$this->fail('Exception expected.');
+
+		} catch (\Consistence\JmsSerializer\Enum\DeserializationInvalidValueException $e) {
+			$this->assertEquals('single_enum', $e->getFieldPath());
+			$previous = $e->getPrevious();
+			$this->assertInstanceOf(\Consistence\Enum\InvalidEnumValueException::class, $previous);
+			$this->assertEquals(['foo' => 'bar'], $previous->getValue());
+		}
+	}
+
+	public function testDeserializeMultiEnumWithInvalidValueType(): void
+	{
+		$serializer = $this->getSerializer();
+
+		try {
+			$serializer->deserialize(sprintf('{
+				"multi_enum": "%s"
+			}', 'foo'), User::class, 'json');
+
+			$this->fail('Exception expected.');
+
+		} catch (\Consistence\JmsSerializer\Enum\DeserializationInvalidValueException $e) {
+			$this->assertEquals('multi_enum', $e->getFieldPath());
+			$previous = $e->getPrevious();
+			$this->assertInstanceOf(\Consistence\Enum\InvalidEnumValueException::class, $previous);
+			$this->assertSame('foo', $previous->getValue());
+		}
+	}
+
 	public function testSerializeEnumAsSingleEnumsArrayNotMappedSingleEnum(): void
 	{
 		$user = new User();