Skip to content

Fixed array matching when key does not exist in value array #15

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 3 commits into from
May 29, 2014

Conversation

pskt
Copy link
Contributor

@pskt pskt commented May 29, 2014

Fix for #13

@@ -131,7 +137,8 @@ public static function negativeMatchData()
array(array(1), array()),
array(array('key' => 'val'), array('key' => 'val2')),
array(array(1), array(2)),
array(array('foo', 1, 3), array('foo', 2, 3))
array(array('foo', 1, 3), array('foo', 2, 3)),
array(array(), array('foo' => 'bar'))
Copy link
Member

Choose a reason for hiding this comment

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

Could you add some tests for nested arrays?

@defrag
Copy link
Member

defrag commented May 29, 2014

@pskt can u add here twose two assertions from my previous pr in json matcher test to make sure its fine?

norberttech added a commit that referenced this pull request May 29, 2014
Fixed array matching when key does not exist in value array
@norberttech norberttech merged commit 3e320d6 into coduo:master May 29, 2014
@norberttech
Copy link
Member

🍺

@pskt pskt deleted the array-matching-fix branch May 29, 2014 10:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants