Mercurial > bitten > bitten-test
comparison bitten/tests/queue.py @ 893:ed77e3e1c43f
BuildConfig may be `None` when checking to see if build should be deleted. Fix + test, closes #666.
author | osimons |
---|---|
date | Tue, 08 Mar 2011 03:19:17 +0000 |
parents | 7c80375d4817 |
children | 0cf576cea845 |
comparison
equal
deleted
inserted
replaced
891:60d0807334ec | 893:ed77e3e1c43f |
---|---|
350 queue = BuildQueue(self.env, build_all=True) | 350 queue = BuildQueue(self.env, build_all=True) |
351 | 351 |
352 self.assertEqual(True, queue.should_delete_build(build, self.repos)) | 352 self.assertEqual(True, queue.should_delete_build(build, self.repos)) |
353 self.assert_("configuration is deactivated" in messages[0]) | 353 self.assert_("configuration is deactivated" in messages[0]) |
354 | 354 |
355 def test_should_delete_build_config_none(self): | |
356 out = [] | |
357 self.env.log = Mock( | |
358 info=lambda msg, *args: out.extend([msg] + list(args))) | |
359 platform = TargetPlatform(self.env, config='test', name='stuff') | |
360 platform.insert() | |
361 build = Build(self.env, config='does_not_exist', rev=42, | |
362 platform=platform.id, rev_time=123456) | |
363 build.insert() | |
364 queue = BuildQueue(self.env, build_all=True) | |
365 | |
366 self.assertEqual(True, queue.should_delete_build(build, self.repos)) | |
367 self.assertTrue("configuration is deactivated" in out[0]) | |
368 self.assertEquals('unknown config "does_not_exist"', out[1]) | |
369 | |
355 def test_should_delete_build_outside_revision_range(self): | 370 def test_should_delete_build_outside_revision_range(self): |
356 messages = [] | 371 messages = [] |
357 self.env.log = Mock(info=lambda msg, *args: messages.append(msg)) | 372 self.env.log = Mock(info=lambda msg, *args: messages.append(msg)) |
358 self.repos.rev_older_than = lambda rev1, rev2: rev1 < rev2 | 373 self.repos.rev_older_than = lambda rev1, rev2: rev1 < rev2 |
359 config = BuildConfig(self.env, 'test', active=True, min_rev=120, | 374 config = BuildConfig(self.env, 'test', active=True, min_rev=120, |