Noissue - fix brittle tests of update_issues.py

sitescripts/hg/test/update_issues.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2016 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 45 matching lines @@
     def _create_mock_milestone_multicall(self):
         ret = []
         multicall = mock.Mock(return_value=ret)
         multicall.ticket.milestone.get = lambda i: ret.append(MILESTONES[i])
         return multicall
 
     def _mock_trac(self):
         trac = mock.Mock()
         trac.ticket.get = lambda i: [i, mock.ANY, mock.ANY, ISSUES[i]['attrs']]
         trac.ticket.getActions = lambda i: ISSUES[i]['actions']
-        trac.ticket.milestone.getAll = lambda: MILESTONES.keys()
+        trac.ticket.milestone.getAll = lambda: sorted(MILESTONES.keys())
         self.trac_proxy_mock = trac
         self._patchWith('xmlrpclib.ServerProxy', trac)
         self._patchWith('xmlrpclib.MultiCall',
                         self._create_mock_milestone_multicall())
 
     def _mock_config(self):
         config = ConfigParser.ConfigParser()
         config.add_section('hg')
         config.set('hg', 'trac_xmlrpc_url', 'foo')
         config.set('hg', 'issue_url_template', '#{id}')
@@ skipping 130 matching lines @@
 
     def test_fix_nonexistent(self):
         self._run_hook(['Fixes 7331 - Foo'], warning_count=1)
 
     def test_fix_closed_with_assigned_milestone(self):
         self._run_hook(['fixes 3559 - Foo'])  # No updates.
 
 
 if __name__ == '__main__':
     unittest.main()