Skip to content

Commit

Permalink
Merge pull request #124 from ClusterHQ/failing-test-123
Browse files Browse the repository at this point in the history
Don't rely on dictionary order.

Fixes #123.
  • Loading branch information
itamarst committed Oct 22, 2014
2 parents 5686f26 + 9ad15dc commit ed767b6
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions eliot/tests/test_output.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
from unittest import TestCase, skipIf
from io import BytesIO

from six import PY3, text_type as unicode
from six import PY3

from zope.interface.verify import verifyClass

Expand Down Expand Up @@ -412,9 +412,9 @@ def __repr__(self):
dictionary = {badobject(): 123,
123: badobject()}
badMessage = "eliot: unknown, unicode() raised exception"
self.assertEqual(Logger()._safeUnicodeDictionary(dictionary),
unicode({badMessage: "123",
"123": badMessage}))
self.assertEqual(eval(Logger()._safeUnicodeDictionary(dictionary)),
{badMessage: "123",
"123": badMessage})


def test_safeUnicodeDictionaryFallback(self):
Expand Down

0 comments on commit ed767b6

Please sign in to comment.