Allon Mureinik has uploaded a new change for review.
Change subject: core: Fix the enable-dao-tests
......................................................................
core: Fix the enable-dao-tests
Change Ia736419e added a database test to DbConfigUtilsTest, which was
not a database test originally.
This causes the regular test suite to fail since no database is
present.
The following patch amends the dal module's pom not to run this test by
default, but only when -Penable-dao-tests is specified.
This is done by adding this test to the default ignore list, and
overriding the surefire-plugin configuration in the profile.
Change-Id: Icbc55a014a54c584a476e66612e54b8b926d02c4
Signed-off-by: Allon Mureinik <amure...@redhat.com>
---
M backend/manager/modules/dal/pom.xml
1 file changed, 11 insertions(+), 3 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/04/14004/1
diff --git a/backend/manager/modules/dal/pom.xml
b/backend/manager/modules/dal/pom.xml
index e9f59d9..a0ed073 100644
--- a/backend/manager/modules/dal/pom.xml
+++ b/backend/manager/modules/dal/pom.xml
@@ -174,6 +174,7 @@
<configuration>
<excludes>
<exclude>**/*D??Test*.java</exclude>
+ <exclude>**/DBConfigUtilsTest.java</exclude>
</excludes>
</configuration>
</plugin>
@@ -189,9 +190,16 @@
<profiles>
<profile>
<id>enable-dao-tests</id>
- <properties>
- <test>*D??Test.java</test>
- </properties>
+ <build>
+ <plugins>
+ <plugin>
+ <artifactId>maven-surefire-plugin</artifactId>
+ <configuration combine.self="override">
+ <excludes/>
+ </configuration>
+ </plugin>
+ </plugins>
+ </build>
</profile>
</profiles>
</project>
--
To view, visit http://gerrit.ovirt.org/14004
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icbc55a014a54c584a476e66612e54b8b926d02c4
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
