Bug 965915

Summary:	Add unit tests for the task library
Product:	[Retired] Beaker	Reporter:	Nick Coghlan <ncoghlan>
Component:	scheduler	Assignee:	Raymond Mancy <rmancy>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	develop	CC:	aigao, asaha, dcallagh, ebaak, llim, qwan, rmancy
Target Milestone:	0.15	Keywords:	FutureFeature
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	ImplementationQuality
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-10-03 02:27:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1002395

Description Nick Coghlan 2013-05-22 02:47:37 UTC

Correct manipulation of the task library is currently tested only indirectly, either through uploading and upgrading tasks through the web UI, or through the scheduling of jobs.

This has resulted in a number of regressions making it through to the development branch or even all the way to production.

With the new abstraction broken out as bkr.server.model.TaskLibrary, unit tests should be added that ensure correct operation (including filesystem manipulation and metadata generation) at the method level, independent of the rest of Beaker.

Comment 1 Nick Coghlan 2013-05-22 03:01:59 UTC

It would also be desirable to have RPM generation infrastructure in the test suite that supports creation of a valid RPM file for arbitrary tasks (see comments on http://gerrit.beaker-project.org/#/c/1958)

Comment 2 Raymond Mancy 2013-09-12 03:32:54 UTC

http://gerrit.beaker-project.org/#/c/2225/

Comment 3 Amit Saha 2013-09-17 08:59:32 UTC

Unit tests, so upgrading MODIFIED -> VERIFIED

Comment 4 Nick Coghlan 2013-10-03 02:27:54 UTC

Beaker 0.15 has been released.