zuul
/
zuul
Code Issues Proposed changes

Merge "Fix race in test_queue_rate_limiting_dependent" into feature/zuulv3

This commit is contained in:
Jenkins 2017-07-11 03:18:12 +00:00 committed by Gerrit Code Review
parent 5434c40c41 df25ddc818
commit 91fad2609f
2 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
tests/base.py
View File

@ -2342,6 +2342,11 @@ class ZuulTestCase(BaseTestCase):
if count is not None and i >= count:
break
def getSortedBuilds(self):
"Return the list of currently running builds sorted by name"
return sorted(self.builds, key=lambda x: x.name)
def release(self, job):
if isinstance(job, FakeBuild):
job.release()

5
tests/unit/test_scheduler.py
View File

@ -3367,8 +3367,9 @@ class TestScheduler(ZuulTestCase):
# Only C's test jobs are queued because window is still 1.
self.assertEqual(len(self.builds), 2)
self.assertEqual(self.builds[0].name, 'project-test1')
self.assertEqual(self.builds[1].name, 'project-test2')
builds = self.getSortedBuilds()
self.assertEqual(builds[0].name, 'project-test1')
self.assertEqual(builds[1].name, 'project-test2')
self.executor_server.release('project-.*')
self.waitUntilSettled()