-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34333/
-----------------------------------------------------------
(Updated May 18, 2015, 5:56 p.m.)
Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Myroslav
Papirkovskyy, and Nate Cole.
Changes
-------
Published a patch that fixes all tests except
AlertDispatchDAOTest#testDeleteTargetWithNotices() . This test causes some
complicated issues with entity mappings, need to take a deeper look.
Tests in error:
testDeleteTargetWithNotices(org.apache.ambari.server.orm.dao.AlertDispatchDAOTest):
Exception [EclipseLink-4002] (Eclipse Persistence Services -
2.5.2.v20140319-9ad6abd):
org.eclipse.persistence.exceptions.DatabaseException(..)
Tests run: 2989, Failures: 0, Errors: 1, Skipped: 20
The latter test is still ignored (opened a separate jira for it). Other tests
are fixed.
Bugs: AMBARI-11200
https://issues.apache.org/jira/browse/AMBARI-11200
Repository: ambari
Description
-------
Fix unit tests that were disabled when committing blocker AMBARI-10818
Diffs (updated)
-----
ambari-server/pom.xml 4c34e6a
ambari-server/src/main/java/org/apache/ambari/server/orm/entities/AlertTargetEntity.java
ce29921
ambari-server/src/test/java/org/apache/ambari/server/events/listeners/upgrade/HostVersionOutOfSyncListenerTest.java
7f5cbfb
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDefinitionDAOTest.java
547edf2
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDispatchDAOTest.java
4138774
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java
c774474
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/UpgradeActionTest.java
1bc6436
ambari-server/src/test/java/org/apache/ambari/server/state/alerts/InitialAlertEventTest.java
31b707a
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/AlertDataManagerTest.java
aa6f1df
ambari-server/src/test/java/org/apache/ambari/server/state/cluster/ClusterTest.java
677ee1a
Diff: https://reviews.apache.org/r/34333/diff/
Testing
-------
Running full mvn test (in progress)
Thanks,
Dmitro Lisnichenko
