Bug 851650

Summary:	Create unit tests for inventory sync
Product:	[Other] RHQ Project	Reporter:	John Sanda <jsanda>
Component:	Core Server, Plugin Container	Assignee:	Nobody <nobody>
Status:	NEW ---	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	4.4	CC:	hrupp
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		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:	851634

Description John Sanda 2012-08-24 15:25:02 UTC

Description of problem:
These tests will involve a combination of state and behavioral verification. The state verification should be pretty similar to the verification done in the integration tests created for bug 851640 - making sure the local inventory matches expected values. The behavioral verification will cover server-side calls. For example, when the agent is merging a unknown resource into its inventory (as would be the case when starting with --purgedata), we want to verify that a call is made to MeasurementServerService to obtain the metric collection schedules for that resource.

On the agent side, we want to add tests for InventoryManager.synchronizeInventory and methods called from there within. Tests should also include coverage for DriftSyncManager, and the relevant parts of MeasurementManager and DriftManager. It may make sense to create separate bugs for some of these different parts as they cover a good bit of code.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info: