Bug 847974

Summary: [Codechange]: Remove broken test MonitoringStrategyFactoryTest
Product: Red Hat Enterprise Virtualization Manager Reporter: Allon Mureinik <amureini>
Component: Build SubsystemAssignee: Allon Mureinik <amureini>
Status: CLOSED CURRENTRELEASE QA Contact: Yaniv Kaul <ykaul>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.1.0CC: dyasny, ecohen, iheim, jkt, ofrenkel, Rhev-m-bugs, sgrinber, ykaul
Target Milestone: ---Keywords: CodeChange
Target Release: ---   
Hardware: All   
OS: All   
Whiteboard: infra
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-27 11:10:18 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Allon Mureinik 2012-08-14 09:01:51 UTC
Description of problem:
This test is written wrong - it assumes a certain execution order by JUnit (which cannot be assumed) in order to run properly.
Currently, it consistently fails.

Since even if it worked properly it doesn't test any meaningful flow, and uses PowerMock which we're attempting to remove from the project, we wish to remove it, as was done in oVirt.

Version-Release number of selected component (if applicable):
The test breaks in Change-Id: Ifaa25bbaddad4c6894abfa3b9e83f205ee9678a1.

How reproducible:
100%

Steps to Reproduce:
1. Check out project
2. mvn clean install
  
Actual results:
MonitoringStrategyFactoryTest fails the build

Expected results:
The build should succeed.

Additional info:

Comment 1 Allon Mureinik 2012-08-14 09:06:12 UTC
Solution in oVirt was merged, pending acks to merge downstream:
http://gerrit.ovirt.org/#/c/7123/