Skip to content

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jan 9, 2025

Q A
Branch? 7.3
Bug fix? no
New feature? no
Deprecations? no
Issues
License MIT

@@ -119,9 +119,6 @@ public function testAccepts()

$this->assertTrue($type->accepts(new \ArrayObject([1 => true])));
$this->assertFalse($type->accepts(new \ArrayObject(['foo' => true])));

$type = new CollectionType(Type::generic(Type::builtin(TypeIdentifier::ITERABLE), Type::int(), Type::bool()));
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was a bit confused when reading this test added in #59291 as I thought this would be a new test case but it actually seems to be the same as defined on line 118 /cc @mtarld

Copy link
Contributor

@mtarld mtarld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed nice catch, thanks @xabbuh 🙂

@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit f51c2d8 into symfony:7.3 Jan 9, 2025
11 checks passed
@xabbuh xabbuh deleted the pr-59291 branch January 9, 2025 11:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants