Change in ovirt-engine[master]: engine: Streamline testing references in pom.xmls

lhornyak at redhat.com lhornyak at redhat.com
Wed Aug 29 12:50:54 UTC 2012


Laszlo Hornyak has submitted this change and it was merged.

Change subject: engine: Streamline testing references in pom.xmls
......................................................................


engine: Streamline testing references in pom.xmls

After a major cleanup to PowerMock and EasyMock usages, it is now
possible to streamline the references to the various testing libraries
oVirt Engine uses in the pom files:

1. Removed redundant declarations of JUnit's dependency - there should
   be only one, in the root pom.xml. In order to achieve this, the
   dependency was moved from the <dependencies> element to the
   <dependencyManagement> element.

2. Streamlined JUnit's versioning to use JUnit 4.8.1, instead of some
   subprojects using 4.7 and some 4.8.1.

3. Removed redundant declarations of Mockito's dependency - there should
   be only one, in the root pom.xml.

4. Moved EasyMock dependency from the root pom.xml to the restapi
   pom.xml since this is the only place EasyMock is allowed in the
   project.

5. Moved PowerMock dependency and repository definition from the root
   pom.xml to the restapi pom.xml since this is the only place
   PowerMock is allowed in the project.

6. Removed any references to PowerMockito, as it is no longer used
   anywhere in the project.

Change-Id: Ieaae801516e68f7e7abfb43f732d1558200e1f25
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/dal/pom.xml
M backend/manager/modules/restapi/interface/common/jaxrs/pom.xml
M backend/manager/modules/restapi/pom.xml
M frontend/webadmin/modules/pom.xml
M frontend/webadmin/modules/userportal-gwtp/pom.xml
M frontend/webadmin/modules/webadmin/pom.xml
M pom.xml
7 files changed, 20 insertions(+), 62 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/7546
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ieaae801516e68f7e7abfb43f732d1558200e1f25
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list