Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

assertSame doesn't show diffs within arrays #1252

Closed
jacobweber opened this issue May 6, 2014 · 3 comments
Closed

assertSame doesn't show diffs within arrays #1252

jacobweber opened this issue May 6, 2014 · 3 comments

Comments

@jacobweber
Copy link

If I use assertEquals on two arrays, and one of their elements are different, it shows a helpful line-by-line diff:

$this->assertEquals(array('a' => false), array('b' => 0));
1) FooTest::testFoo
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
-    'a' => false
+    'b' => 0
 )

But if I use assertSame instead, it doesn't indicate where the differences are. This can make it difficult to see the error, if the arrays are big.

$this->assertSame(array('a' => false), array('b' => 0));
1) FooTest::testFoo
Failed asserting that Array &0 (
    'b' => 0
) is identical to Array &0 (
    'a' => false
).

It would be nice if there were a way to get the diff using assertSame.

@sun
Copy link
Contributor

sun commented Jul 21, 2014

Indeed. I sometimes had to switch from assertSame() to assertEquals() in such cases, even though it weakens the strict expectation. Getting a diff specifically for arrays from assertSame() upon failure would be great.

@jacobweber
Copy link
Author

Tested on PHPUnit 4.4.5 -- same result.

@sebastianbergmann
Copy link
Owner

Dear contributor,

let me start by apologizing for not commenting and/or working on the issue you have reported or merging the pull request you have sent sooner.

PHPUnit 5.0 was released today. And today I am closing all open bug reports and pull requests for PHPUnit and its dependencies that I maintain. Please do not interpret the closing of this ticket as an insult or a lack of interest in your problem. I am sorry for any inconvenience this may cause.

If the topic of this ticket is still relevant then please open a new ticket or send a new pull request. If your ticket or pull request is about a defect then please check whether the issue still exists in PHPUnit 4.8 (which will received bug fixes until August 2016). If your ticket or pull request is about a new feature then please port your patch PHPUnit 5.0 before sending a new pull request.

I hope that today's extreme backlog grooming will allow me to respond to bug reports and pull requests in a more timely manner in the future.

Thank you for your understanding,
Sebastian

Repository owner locked and limited conversation to collaborators Oct 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants