testDetectsAdditionsToUnionType

This commit is contained in:
Ben Roberts 2017-11-21 12:18:28 -05:00
parent e0a63ec792
commit 533b8b8b5f
2 changed files with 63 additions and 3 deletions

View File

@ -433,12 +433,13 @@ class FindBreakingChanges
Schema $oldSchema, Schema $newSchema Schema $oldSchema, Schema $newSchema
) )
{ {
$oldTypeMap = $oldSchema->getTypeMap();
$newTypeMap = $newSchema->getTypeMap(); $newTypeMap = $newSchema->getTypeMap();
$typesAddedToUnion = []; $typesAddedToUnion = [];
foreach ($newTypeMap as $typeName => $oldType) { foreach ($newTypeMap as $typeName => $newType) {
$newType = isset($newTypeMap[$typeName]) ? $newTypeMap[$typeName] : null; $oldType = isset($oldTypeMap[$typeName]) ? $oldTypeMap[$typeName] : null;
if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) { if (!($oldType instanceof UnionType) || !($newType instanceof UnionType)) {
continue; continue;
} }
@ -450,7 +451,7 @@ class FindBreakingChanges
foreach ($newType->getTypes() as $type) { foreach ($newType->getTypes() as $type) {
if (!isset($typeNamesInOldUnion[$type->name])) { if (!isset($typeNamesInOldUnion[$type->name])) {
$addedTypeName = $type->name; $addedTypeName = $type->name;
$typesRemovedFromUnion[] = ['type' => self::DANGEROUS_CHANGE_TYPE_ADDED_TO_UNION, 'description' => "${addedTypeName} was removed to union type ${typeName}"]; $typesAddedToUnion[] = ['type' => self::DANGEROUS_CHANGE_TYPE_ADDED_TO_UNION, 'description' => "${addedTypeName} was added to union type ${typeName}"];
} }
} }
} }

View File

@ -1326,4 +1326,63 @@ class FindBreakingChangesTest extends \PHPUnit_Framework_TestCase
FindBreakingChanges::findValuesAddedToEnums($oldSchema, $newSchema)[0] FindBreakingChanges::findValuesAddedToEnums($oldSchema, $newSchema)[0]
); );
} }
public function testDetectsAdditionsToUnionType() {
$type1 = new ObjectType([
'name' => 'Type1',
'fields' => [
'field1' => Type::string()
]
]);
$type1a = new ObjectType([
'name' => 'Type1',
'fields' => [
'field1' => Type::string()
]
]);
$type2 = new ObjectType([
'name' => 'Type2',
'fields' => [
'field1' => Type::string()
]
]);
$oldUnionType = new UnionType([
'name' => 'UnionType1',
'types' => [$type1],
'resolveType' => function () {
}
]);
$newUnionType = new UnionType([
'name' => 'UnionType1',
'types' => [$type1a, $type2],
'resolveType' => function () {
}
]);
$oldSchema = new Schema([
'query' => $this->queryType,
'types' => [
$oldUnionType
]
]);
$newSchema = new Schema([
'query' => $this->queryType,
'types' => [
$newUnionType
]
]);
$this->assertEquals(
[
'type' => FindBreakingChanges::DANGEROUS_CHANGE_TYPE_ADDED_TO_UNION,
'description' => 'Type2 was added to union type UnionType1'
],
FindBreakingChanges::findTypesAddedToUnions($oldSchema, $newSchema)[0]
);
}
} }