Allon Mureinik has uploaded a new change for review.

Change subject: tools: enable unit tests
......................................................................

tools: enable unit tests

The tools package contains some useful unit tests, which are
unfortunately disabled since they are intermixed with integration tests.

This patch replaces the all-out disabling of the package's tests with a
black list of component tests, allowing the unit tests to run in build
time.

Change-Id: Ic9205fd5d46fb8e44c0ad87cfbe4c3b5c7817dbe
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M backend/manager/tools/pom.xml
1 file changed, 16 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/71/14271/1

diff --git a/backend/manager/tools/pom.xml b/backend/manager/tools/pom.xml
index 9e97ce2..e231854 100644
--- a/backend/manager/tools/pom.xml
+++ b/backend/manager/tools/pom.xml
@@ -65,9 +65,16 @@
   <profiles>
     <profile>
       <id>enable-tools-itests</id>
-      <properties>
-        <test>**/*Test.java</test>
-      </properties>
+      <build>
+        <plugins>
+          <plugin>
+            <artifactId>maven-surefire-plugin</artifactId>
+            <configuration combine.self="override">
+              <excludes/>
+            </configuration>
+          </plugin>
+        </plugins>
+      </build>
     </profile>
   </profiles>
 
@@ -80,7 +87,12 @@
             
<additionalClasspathElement>${basedir}/**/src/test/java</additionalClasspathElement>
           </additionalClasspathElements>
           <excludes>
-            <exclude>**/*Test.java</exclude>
+            <exclude>**/EngineConfigLogicTest.java</exclude>
+            <exclude>**/EngineConfigTest.java</exclude>
+            <exclude>**/MailSenderTest.java</exclude>
+            <exclude>**/StandaloneDataSourceTest.java</exclude>
+            <exclude>**/NotificationServiceTest.java</exclude>
+            <exclude>**/EngineMonitorServiceTest.java</exclude>
           </excludes>
         </configuration>
       </plugin>


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic9205fd5d46fb8e44c0ad87cfbe4c3b5c7817dbe
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to