@cansarigol, can you please specify what do you expect and how it works?

The PR changes the meaning of assertCountEqual. https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertCountEqual says: Equivalent to: assertEqual(Counter(list(first)), Counter(list(second))) ...

At the very least, the documentation would need to be updated. But since this could be a breaking change, I don't think it will be accepted.

@sobolevn I was expecting the following test case to be covered.

self.assertRaises(self.failureException, self.assertCountEqual,
                          {'a': 1}, {'a': 2})

what I understand from the doc of the assertion is that elements have to be the same without regard to order. The doc can be extended for dict.

In [10]: d = {"key": "value"}

In [11]: collections.Counter(list(d))
Out[11]: Counter({'key': 1})

In [12]: collections.Counter(d.items())
Out[12]: Counter({('key', 'value'): 1})

I agree with Eric that this should not be changed.  It is working as documented and intended.

In its original incarenation, assertCountEqual was documented as being equivalent to ``assertEqual(sorted(expected), sorted(actual))``.  Either way, the goal was to count the values produced by iterating.

To compare two dicts or two Counters including both keys and values, just use assertEqual.

Concur with Eric and Raymond.