Chromium Code Reviews| Index: tools/cygprofile/patch_orderfile_unittest.py |
| diff --git a/tools/cygprofile/patch_orderfile_unittest.py b/tools/cygprofile/patch_orderfile_unittest.py |
| index b88dcafc38380069b47f44881500003a10dddb9b..8ee2df5c0060208254c867632b87003df4139a82 100755 |
| --- a/tools/cygprofile/patch_orderfile_unittest.py |
| +++ b/tools/cygprofile/patch_orderfile_unittest.py |
| @@ -101,9 +101,9 @@ class TestPatchOrderFile(unittest.TestCase): |
| def testPrintSymbolWithPrefixes(self): |
| class FakeOutputFile(object): |
| def __init__(self): |
| - self.output = [] |
| + self.output = '' |
| def write(self, s): |
| - self.output.append(s) |
| + self.output = self.output + s |
|
pasko
2015/01/22 17:37:14
did the old version not work? if so, why?
azarchs
2015/01/22 17:44:30
The old one resulted in extra '\n's between output
|
| test_symbol = "dummySymbol" |
| symbol_names = [test_symbol] |
| fake_output = FakeOutputFile() |
| @@ -113,7 +113,7 @@ class TestPatchOrderFile(unittest.TestCase): |
| .text.unlikely.dummySymbol |
| .text.dummySymbol |
| """ |
| - self.assertEquals("\n".join(fake_output.output), expected_output) |
| + self.assertEquals(fake_output.output, expected_output) |
| if __name__ == "__main__": |