February 2011 - Topia-commits - mailman.cloud.codelutin.com

r2212 - trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util
by tchemit＠users.nuiton.org 27 Feb '11

27 Feb '11

Author: tchemit Date: 2011-02-27 19:20:17 +0100 (Sun, 27 Feb 2011) New Revision: 2212 Url: http://nuiton.org/repositories/revision/topia/2212 Log: Evolution #1371: Introduce TopiaEntityMap Added: trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util/TopiaEntityMap.java Added: trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util/TopiaEntityMap.java =================================================================== --- trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util/TopiaEntityMap.java (rev 0) +++ trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util/TopiaEntityMap.java 2011-02-27 18:20:17 UTC (rev 2212) @@ -0,0 +1,108 @@ +/* + * #%L + * ToPIA :: Persistence + * + * $Id$ + * $HeadURL$ + * %% + * Copyright (C) 2004 - 2011 CodeLutin + * %% + * This program is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation, either version 3 of the + * License, or (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Lesser Public License for more details. + * + * You should have received a copy of the GNU General Lesser Public + * License along with this program. If not, see + * <http://www.gnu.org/licenses/lgpl-3.0.html>. + * #L% + */ +package org.nuiton.topia.persistence.util; + +import org.nuiton.topia.persistence.TopiaEntity; +import org.nuiton.topia.persistence.TopiaEntityEnum; + +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +/** + * A dictionnary of {@link TopiaEntity} associated to a {@link TopiaEntityEnum}. + * + * @author tchemit <chemit(a)codelutin.com> + * @since 2.5.3 + */ +public abstract class TopiaEntityMap<K extends TopiaEntityEnum, V extends TopiaEntity> extends HashMap<K, List<? extends V>> { + + private static final long serialVersionUID = 1L; + + public TopiaEntityMap(int initialCapacity, float loadFactor) { + super(initialCapacity, loadFactor); + } + + public TopiaEntityMap(int initialCapacity) { + super(initialCapacity); + } + + public TopiaEntityMap() { + } + + public TopiaEntityMap(Map<? extends K, ? extends List<? extends V>> m) { + super(m); + } + + /** + * Obtains from a entity his key. + * + * @param e the entity on which to find the key. + * @return the key of the given entity. + */ + protected abstract K getType(TopiaEntity e); + + + /** + * Adds the given entity to the dictonary only if it does not exists. + * + * Will return {@code true} if entity was added, {@code false} otherwise. + * + * @param entity the entity to add + * @param <T> the type of entity to add + * @return {@code true} if entity was added, {@code false} otherwise. + */ + public <T extends V> boolean addUniqueEntity(T entity) { + List<T> list = getList(entity); + if (!list.contains(entity)) { + list.add(entity); + return true; + } + return false; + } + + /** + * Adds the given entity to the dictonary (even if it does already exists). + * + * @param entity the entity to add + * @param <T> the type of entity to add + */ + public <T extends V> void addEntity(T entity) { + List<T> list = getList(entity); + list.add(entity); + } + + @SuppressWarnings({"unchecked"}) + protected <T extends V> List<T> getList(T entity) { + K type = getType(entity); + List<T> list = (List<T>) get(type); + if (list == null) { + list = new ArrayList<T>(); + put(type, list); + } + return list; + } +} Property changes on: trunk/topia-persistence/src/main/java/org/nuiton/topia/persistence/util/TopiaEntityMap.java ___________________________________________________________________ Added: svn:keywords + Author Date Id Revision HeadURL Added: svn:eol-style + native

1 0

r2211 - trunk
by tchemit＠users.nuiton.org 16 Feb '11

16 Feb '11

Author: tchemit Date: 2011-02-16 22:32:27 +0100 (Wed, 16 Feb 2011) New Revision: 2211 Url: http://nuiton.org/repositories/revision/topia/2211 Log: Update mavenpom4redmineAndCentral to 2.5.1. Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) +++ trunk/pom.xml 2011-02-16 21:32:27 UTC (rev 2211) @@ -33,7 +33,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmineAndCentral</artifactId> - <version>2.5</version> + <version>2.5.1</version> </parent> <artifactId>topia</artifactId>

1 0

r2210 - in trunk: . topia-persistence topia-service-migration topia-service-replication topia-service-security
by hudson＠users.nuiton.org 15 Feb '11

15 Feb '11

Author: hudson Date: 2011-02-15 20:49:38 +0100 (Tue, 15 Feb 2011) New Revision: 2210 Url: http://nuiton.org/repositories/revision/topia/2210 Log: [maven-release-plugin] prepare for next development iteration Modified: trunk/pom.xml trunk/topia-persistence/pom.xml trunk/topia-service-migration/pom.xml trunk/topia-service-replication/pom.xml trunk/topia-service-security/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-15 19:49:23 UTC (rev 2209) +++ trunk/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) @@ -37,7 +37,7 @@ </parent> <artifactId>topia</artifactId> - <version>2.5.2</version> + <version>2.5.3-SNAPSHOT</version> <modules> <module>topia-persistence</module> @@ -344,9 +344,9 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/topia/tags/topia-2.5.2</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/topia/tags/topia-2.5.2</developerConnection> - <url>http://www.nuiton.org/repositories/browse/topia/tags/topia-2.5.2</url> + <connection>scm:svn:http://svn.nuiton.org/svn/topia/trunk</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/topia/trunk</developerConnection> + <url>http://www.nuiton.org/repositories/browse/topia/trunk</url> </scm> </project> Modified: trunk/topia-persistence/pom.xml =================================================================== --- trunk/topia-persistence/pom.xml 2011-02-15 19:49:23 UTC (rev 2209) +++ trunk/topia-persistence/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2</version> + <version>2.5.3-SNAPSHOT</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-migration/pom.xml =================================================================== --- trunk/topia-service-migration/pom.xml 2011-02-15 19:49:23 UTC (rev 2209) +++ trunk/topia-service-migration/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2</version> + <version>2.5.3-SNAPSHOT</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-replication/pom.xml =================================================================== --- trunk/topia-service-replication/pom.xml 2011-02-15 19:49:23 UTC (rev 2209) +++ trunk/topia-service-replication/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2</version> + <version>2.5.3-SNAPSHOT</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-security/pom.xml =================================================================== --- trunk/topia-service-security/pom.xml 2011-02-15 19:49:23 UTC (rev 2209) +++ trunk/topia-service-security/pom.xml 2011-02-15 19:49:38 UTC (rev 2210) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2</version> + <version>2.5.3-SNAPSHOT</version> </parent> <groupId>org.nuiton.topia</groupId>

1 0

r2209 - in tags: . topia-2.5.2
by hudson＠users.nuiton.org 15 Feb '11

15 Feb '11

Author: hudson Date: 2011-02-15 20:49:23 +0100 (Tue, 15 Feb 2011) New Revision: 2209 Url: http://nuiton.org/repositories/revision/topia/2209 Log: [maven-release-plugin] copy for tag topia-2.5.2 Added: tags/topia-2.5.2/ Property changes on: tags/topia-2.5.2 ___________________________________________________________________ Added: svn:ignore + *.ipr *.iws .classpath *.iml .project target .settings Added: svn:mergeinfo + /branches/from2.2.2-eugene2-beta:1662-1714

1 0

r2208 - in trunk: . topia-persistence topia-service-migration topia-service-replication topia-service-security
by hudson＠users.nuiton.org 15 Feb '11

15 Feb '11

Author: hudson Date: 2011-02-15 20:49:21 +0100 (Tue, 15 Feb 2011) New Revision: 2208 Url: http://nuiton.org/repositories/revision/topia/2208 Log: [maven-release-plugin] prepare release topia-2.5.2 Modified: trunk/pom.xml trunk/topia-persistence/pom.xml trunk/topia-service-migration/pom.xml trunk/topia-service-replication/pom.xml trunk/topia-service-security/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-15 19:40:19 UTC (rev 2207) +++ trunk/pom.xml 2011-02-15 19:49:21 UTC (rev 2208) @@ -37,7 +37,7 @@ </parent> <artifactId>topia</artifactId> - <version>2.5.2-SNAPSHOT</version> + <version>2.5.2</version> <modules> <module>topia-persistence</module> @@ -344,9 +344,9 @@  <scm> - <connection>scm:svn:http://svn.nuiton.org/svn/topia/trunk</connection> - <developerConnection>scm:svn:http://svn.nuiton.org/svn/topia/trunk</developerConnection> - <url>http://www.nuiton.org/repositories/browse/topia/trunk</url> + <connection>scm:svn:http://svn.nuiton.org/svn/topia/tags/topia-2.5.2</connection> + <developerConnection>scm:svn:http://svn.nuiton.org/svn/topia/tags/topia-2.5.2</developerConnection> + <url>http://www.nuiton.org/repositories/browse/topia/tags/topia-2.5.2</url> </scm> </project> Modified: trunk/topia-persistence/pom.xml =================================================================== --- trunk/topia-persistence/pom.xml 2011-02-15 19:40:19 UTC (rev 2207) +++ trunk/topia-persistence/pom.xml 2011-02-15 19:49:21 UTC (rev 2208) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2-SNAPSHOT</version> + <version>2.5.2</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-migration/pom.xml =================================================================== --- trunk/topia-service-migration/pom.xml 2011-02-15 19:40:19 UTC (rev 2207) +++ trunk/topia-service-migration/pom.xml 2011-02-15 19:49:21 UTC (rev 2208) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2-SNAPSHOT</version> + <version>2.5.2</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-replication/pom.xml =================================================================== --- trunk/topia-service-replication/pom.xml 2011-02-15 19:40:19 UTC (rev 2207) +++ trunk/topia-service-replication/pom.xml 2011-02-15 19:49:21 UTC (rev 2208) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2-SNAPSHOT</version> + <version>2.5.2</version> </parent> <groupId>org.nuiton.topia</groupId> Modified: trunk/topia-service-security/pom.xml =================================================================== --- trunk/topia-service-security/pom.xml 2011-02-15 19:40:19 UTC (rev 2207) +++ trunk/topia-service-security/pom.xml 2011-02-15 19:49:21 UTC (rev 2208) @@ -35,7 +35,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>topia</artifactId> - <version>2.5.2-SNAPSHOT</version> + <version>2.5.2</version> </parent> <groupId>org.nuiton.topia</groupId>

1 0

r2207 - trunk/topia-service-security/src/test/java/org/nuiton/topia/security
by hudson＠users.nuiton.org 15 Feb '11

15 Feb '11

Author: hudson Date: 2011-02-15 20:40:19 +0100 (Tue, 15 Feb 2011) New Revision: 2207 Url: http://nuiton.org/repositories/revision/topia/2207 Log: fix test Modified: trunk/topia-service-security/src/test/java/org/nuiton/topia/security/TopiaSecurityTest.java Modified: trunk/topia-service-security/src/test/java/org/nuiton/topia/security/TopiaSecurityTest.java =================================================================== --- trunk/topia-service-security/src/test/java/org/nuiton/topia/security/TopiaSecurityTest.java 2011-02-15 18:53:34 UTC (rev 2206) +++ trunk/topia-service-security/src/test/java/org/nuiton/topia/security/TopiaSecurityTest.java 2011-02-15 19:40:19 UTC (rev 2207) @@ -102,7 +102,7 @@ @BeforeClass public static void init() throws Exception { - I18n.init(Locale.FRANCE); + I18n.init(null, Locale.FRANCE); tesDir = TestHelper.getTestBasedir(TopiaSecurityTest.class); }

1 0

r2206 - trunk
by tchemit＠users.nuiton.org 15 Feb '11

15 Feb '11

Author: tchemit Date: 2011-02-15 19:53:34 +0100 (Tue, 15 Feb 2011) New Revision: 2206 Url: http://nuiton.org/repositories/revision/topia/2206 Log: Evolution #1321: Updates to i18n 2.3.1 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-12 11:23:31 UTC (rev 2205) +++ trunk/pom.xml 2011-02-15 18:53:34 UTC (rev 2206) @@ -257,7 +257,7 @@ <eugeneVersion>2.3.2</eugeneVersion> <nuitonUtilsVersion>2.0</nuitonUtilsVersion> <processorPluginVersion>1.2.1</processorPluginVersion> - <nuitonI18nVersion>2.3</nuitonI18nVersion> + <nuitonI18nVersion>2.3.1</nuitonI18nVersion> <xmlrpcVersion>3.1.2</xmlrpcVersion> <hibernateVersion>3.3.2.GA</hibernateVersion>

1 0

r2205 - in trunk/topia-persistence/src: main/java/org/nuiton/topia/generator test/java/org/nuiton/topia/framework
by tchemit＠users.nuiton.org 12 Feb '11

12 Feb '11

Author: tchemit Date: 2011-02-12 12:23:31 +0100 (Sat, 12 Feb 2011) New Revision: 2205 Url: http://nuiton.org/repositories/revision/topia/2205 Log: Anomalie #1317: Hibernate mapping generation is wrong! Use new i18n api Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/generator/EntityHibernateMappingGenerator.java trunk/topia-persistence/src/test/java/org/nuiton/topia/framework/TopiaContextImplTest.java Modified: trunk/topia-persistence/src/main/java/org/nuiton/topia/generator/EntityHibernateMappingGenerator.java =================================================================== --- trunk/topia-persistence/src/main/java/org/nuiton/topia/generator/EntityHibernateMappingGenerator.java 2011-02-10 10:16:59 UTC (rev 2204) +++ trunk/topia-persistence/src/main/java/org/nuiton/topia/generator/EntityHibernateMappingGenerator.java 2011-02-12 11:23:31 UTC (rev 2205) @@ -93,6 +93,12 @@ public static final String HIBERNATE_ATTRIBUTE_NOT_NULL = "not-null"; + public static final String HIBERNATE_ATTRIBUTE_SCHEMA = "schema"; + + public static final String HIBERNATE_ATTRIBUTE_INDEX = "index"; + + public static final String HIBERNATE_ATTRIBUTE_UNIQUE = "unique"; + public static final String HIBERNATE_ATTRIBUTE_LENGTH = "length"; public static final String HIBERNATE_ATTRIBUTE_ORDER_BY = "order-by"; @@ -344,12 +350,14 @@ String optionalAttributes = ""; String schema = TopiaGeneratorUtil.getDbSchemaNameTagValue(attr, model); if (schema != null) { - optionalAttributes += "schema=\"" + schema + "\" "; + optionalAttributes += generateFromTagValue(HIBERNATE_ATTRIBUTE_SCHEMA, schema); +// optionalAttributes += "schema=\"" + schema + "\" "; } if (JavaGeneratorUtil.hasIndexedStereotype(attr)) { String indexName = tableName + "_idx"; - optionalAttributes += "index=\"" + indexName + "\" "; + optionalAttributes += generateFromTagValue(HIBERNATE_ATTRIBUTE_INDEX, indexName); +// optionalAttributes += "index=\"" + indexName + "\" "; } /*{<%=prefix%> <primitive-array name="<%=attrName%>" table="<%=tableName%>" access="<%=accessField%>" <%=optionalAttributes%>> @@ -362,12 +370,14 @@ String optionalAttributes = ""; if (JavaGeneratorUtil.hasIndexedStereotype(attr)) { String indexName = tableName + "_idx"; - optionalAttributes += "index=\"" + indexName + "\""; + optionalAttributes += generateFromTagValue(HIBERNATE_ATTRIBUTE_INDEX, indexName); +// optionalAttributes += "index=\"" + indexName + "\""; } if (TopiaGeneratorUtil.hasUniqueStereotype(attr)) { // the trim method is called on optionalAttributes after this set to suppress unusual space if no index is set on this attribute - optionalAttributes += " unique=\"true\""; + optionalAttributes += generateFromTagValue(HIBERNATE_ATTRIBUTE_UNIQUE, "true"); +// optionalAttributes += " unique=\"true\""; } optionalAttributes += generateFromTagValue(HIBERNATE_ATTRIBUTE_NOT_NULL, TopiaGeneratorUtil.getNotNullTagValue(attr)); /*{<%=prefix%> <property name="<%=attrName%>" type="<%=attrType%>" access="<%=accessField%>"}*/ @@ -397,7 +407,7 @@ } optionalAttributes = optionalAttributes.trim(); - if (!optionalAttributes.isEmpty()) { + if (StringUtils.isNotEmpty(optionalAttributes)) { optionalAttributes = " " + optionalAttributes; } @@ -588,7 +598,7 @@ // vérifier si le tag lazy est defini par defaut dans le fichier de proprietes String lazy = generateFromTagValue(HIBERNATE_ATTRIBUTE_LAZY, TopiaGeneratorUtil.getLazyTagValue(attr)); /*{<%=lazy%>}*/ - String notNull = " " + generateFromTagValue(HIBERNATE_ATTRIBUTE_NOT_NULL, TopiaGeneratorUtil.getNotNullTagValue(attr)); + String notNull = generateFromTagValue(HIBERNATE_ATTRIBUTE_NOT_NULL, TopiaGeneratorUtil.getNotNullTagValue(attr)); /*{<%=notNull%>}*/ if (isUnique) { /*{unique="true" }*/ Modified: trunk/topia-persistence/src/test/java/org/nuiton/topia/framework/TopiaContextImplTest.java =================================================================== --- trunk/topia-persistence/src/test/java/org/nuiton/topia/framework/TopiaContextImplTest.java 2011-02-10 10:16:59 UTC (rev 2204) +++ trunk/topia-persistence/src/test/java/org/nuiton/topia/framework/TopiaContextImplTest.java 2011-02-12 11:23:31 UTC (rev 2205) @@ -62,7 +62,7 @@ @BeforeClass public static void setUpClass() throws Exception { - I18n.init(Locale.FRENCH); + I18n.init(null, Locale.FRENCH); } @Before

1 0

r2204 - trunk
by tchemit＠users.nuiton.org 10 Feb '11

10 Feb '11

Author: tchemit Date: 2011-02-10 11:16:59 +0100 (Thu, 10 Feb 2011) New Revision: 2204 Url: http://nuiton.org/repositories/revision/topia/2204 Log: Evolution #1305: Updates to mavenpom 2.5 Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-02-09 11:12:15 UTC (rev 2203) +++ trunk/pom.xml 2011-02-10 10:16:59 UTC (rev 2204) @@ -33,7 +33,7 @@ <parent> <groupId>org.nuiton</groupId> <artifactId>mavenpom4redmineAndCentral</artifactId> - <version>2.4.4</version> + <version>2.5</version> </parent> <artifactId>topia</artifactId>

1 0

r2203 - in trunk/topia-service-replication/src: main/java/org/nuiton/topia/replication main/java/org/nuiton/topia/replication/model main/java/org/nuiton/topia/replication/operation test/java/org/nuiton/topia/replication
by tchemit＠users.nuiton.org 09 Feb '11

09 Feb '11

Author: tchemit Date: 2011-02-09 12:12:15 +0100 (Wed, 09 Feb 2011) New Revision: 2203 Url: http://nuiton.org/repositories/revision/topia/2203 Log: Anomalie #1306: Some association are not reattached while replication Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationContext.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationOperation.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationService.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationServiceImpl.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationModel.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationNode.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachAssociation.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachLink.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/DettachAssociation.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/Duplicate.java trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/LoadLink.java trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/AbstractTopiaReplicationServiceTest.java trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/TopiaReplicationOperationTest.java Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationContext.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationContext.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationContext.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -172,6 +172,12 @@ List<String> nodeEntityIds = getEntityIds(node); + if (nodeEntityIds == null) { + + // no ids found for this node + return Collections.emptyList(); + } + if (log.isDebugEnabled()) { log.debug("Will load " + nodeEntityIds.size() + " entities"); } Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationOperation.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationOperation.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationOperation.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -61,7 +61,7 @@ * Creer et enregister une operation utilisateur apres la creation du * modele via la methode * - * {@link TopiaReplicationService#prepare(TopiaEntityEnum[],boolean, String...)}. + * {@link TopiaReplicationService#prepare(TopiaEntityEnum[], boolean, String...)}. * * @param model le modele de replication * @param ownerNode le noeud proprietaire de l'operation @@ -71,11 +71,13 @@ * (says when operation is only internal) * @see ReplicationModel * @see ReplicationOperationPhase + * @deprecated since 2.5.2, there is no difference between a user operation and an internal one. This method will be + * removed in version 2.6 and never replaced */ - void register(ReplicationModel model, - ReplicationNode ownerNode, - ReplicationOperationPhase phase, - Object... parameters) throws UnsupportedOperationException; + @Deprecated + void register(ReplicationModel model, ReplicationNode ownerNode, ReplicationOperationPhase phase, + Object... parameters) + throws UnsupportedOperationException; /** * Execute l'operation avec le parametrage donnee. @@ -89,10 +91,8 @@ * @param entities la liste des entités à traiter * @throws Exception pour toute erreur */ - void run(TopiaReplicationContext replicationContext, - ReplicationOperationDef operationDef, - TopiaContextImplementor srcCtxt, - TopiaContextImplementor dstCtxt, - List<? extends TopiaEntity> entities) throws Exception; + void run(TopiaReplicationContext replicationContext, ReplicationOperationDef operationDef, + TopiaContextImplementor srcCtxt, TopiaContextImplementor dstCtxt, List<? extends TopiaEntity> entities) + throws Exception; } Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationService.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationService.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationService.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -39,7 +39,7 @@ * * The replication is always done in two phases : * - * <li> - prepare the replication model - ({@link #prepare(TopiaEntityEnum[],boolean, String...)} - ({@link #prepareForAll(TopiaEntityEnum[])} + * <li> - prepare the replication model - ({@link #prepare(TopiaEntityEnum[], boolean, String...)} - ({@link #prepareForAll(TopiaEntityEnum[])} * * <li> - lanch replication {@link #doReplicate(ReplicationModel, * TopiaContext)} @@ -49,10 +49,14 @@ */ public interface TopiaReplicationService extends TopiaService { - /** Nom du service */ + /** + * Nom du service + */ String SERVICE_NAME = "replication"; - /** Nom du service topia */ + /** + * Nom du service topia + */ String TOPIA_SERVICE_NAME = "topia.service.replication"; /** @@ -80,23 +84,6 @@ String... topiaIds) throws TopiaException; /** - * Prepare le modele de replication pour les entites dans les topiaIds sont - * donnes en utilisant l'ordre des {@code contracts} donné pour l'ordre de - * replication. - * - * @param contracts les contrats a repliquer - * l'ordre des entités à repliquer, sinon on utilise - * l'ordre induit par les {@code contracts}. - * @param topiaIds la liste des ids d'entites a repliquer - * @return le model de replication initialise - * @throws TopiaException pour toute erreur recontree - * @deprecated since 2.4.3, prefer use the method {@link #prepare(TopiaEntityEnum[], boolean, String...)} with flag setted to {@code false}. - */ - @Deprecated - ReplicationModel prepareWithComputedOrder(TopiaEntityEnum[] contracts, - String... topiaIds) throws TopiaException; - - /** * Prepare le modele de replication pour toutes les entites des types * donnes. * @@ -114,40 +101,10 @@ ReplicationModel prepareForAll(TopiaEntityEnum[] contracts) throws TopiaException; /** - * Ajouter une nouvelle operation pre-replication, sur un type de donnee. - * - * @param model le modele de replication - * @param type le type du noeud de replication - * @param operationClass l'implantation de l'operation - * @param parameters les parametres supplementaires pour l'operation - * @deprecated since 2.4.3, prefer use the method {@link TopiaReplicationModelBuilder#addBeforeOperation(ReplicationModel, TopiaEntityEnum, Class, Object...)} - */ - @Deprecated - void addBeforeOperation(ReplicationModel model, - TopiaEntityEnum type, - Class<? extends TopiaReplicationOperation> operationClass, - Object... parameters); - - /** - * Ajouter une nouvelle operation post-replication, sur un type de donnee. - * - * @param model le modele de replication - * @param type le type du noeud de replication - * @param operationClass l'implantation de l'operation - * @param parameters les parametres supplementaires pour l'operation - * @deprecated since 2.4.3, prefer use the method {@link TopiaReplicationModelBuilder#addAfterOperation(ReplicationModel, TopiaEntityEnum, Class, Object...)} - */ - @Deprecated - void addAfterOperation(ReplicationModel model, - TopiaEntityEnum type, - Class<? extends TopiaReplicationOperation> operationClass, - Object... parameters); - - /** * Lance l'operation de replication a partir du context source vers le * context de destination sur les entites donnees et en utilisant le modele * de replication precedemment construit via la methode {@link - * #prepare(TopiaEntityEnum[],boolean, String...)} ou {@link + * #prepare(TopiaEntityEnum[], boolean, String...)} ou {@link * #prepareForAll(TopiaEntityEnum[])}. * * @param model le modele de replication Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationServiceImpl.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationServiceImpl.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/TopiaReplicationServiceImpl.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -110,44 +110,13 @@ } @Override - public ReplicationModel prepareWithComputedOrder(TopiaEntityEnum[] contracts, - String... topiaIds) throws TopiaException { - return prepare(contracts, false, topiaIds); - } - - @Override public ReplicationModel prepareForAll(TopiaEntityEnum[] contracts) throws TopiaException { ReplicationModel model = getModelBuilder().prepareForAll(contracts); return model; } - @Override - public void addBeforeOperation(ReplicationModel model, - TopiaEntityEnum type, - Class<? extends TopiaReplicationOperation> operationClass, - Object... parameters) { - getModelBuilder().addBeforeOperation(model, - type, - operationClass, - parameters - ); - } - @Override - public void addAfterOperation(ReplicationModel model, - TopiaEntityEnum type, - Class<? extends TopiaReplicationOperation> operationClass, - Object... parameters) { - - getModelBuilder().addAfterOperation(model, - type, - operationClass, - parameters - ); - } - - @Override public TopiaReplicationModelBuilder getModelBuilder() { if (modelBuilder == null) { modelBuilder = new TopiaReplicationModelBuilder(); @@ -172,10 +141,13 @@ // init replication context replicationContext.init(); + ReplicationNode currentNode = null; try { for (ReplicationNode node : model.getOrder()) { + currentNode = node; + // start replication of current node doReplicateNode(replicationContext, node); @@ -184,6 +156,9 @@ } } catch (Exception e) { + if (log.isErrorEnabled()) { + log.error("Could not replicate node " + currentNode, e); + } // an error occurs, rollback all sucessfull replicated nodes. doRollback(replicationContext); @@ -273,6 +248,13 @@ TopiaReplicationOperation operation = replicationContext.getOperation(def); + if (!node.equals(def.getNode())) { + + // when a node operation is reattached to a later node... + entities = + replicationContext.getEntities(srcCtxt, def.getNode()); + } + operation.run(replicationContext, def, srcCtxt, Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationModel.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationModel.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationModel.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -504,7 +504,8 @@ if (!tmpLinks.isEmpty()) { // on a des associations a charger avant replication for (ReplicationLink link : tmpLinks) { - addPreOperation(node, node, LoadLink.class, link); +// addPreOperation(node, node, LoadLink.class, link); + addPreOperation(node, link.getSource(), LoadLink.class, link); } linksToLoad.removeAll(links); tmpLinks.clear(); @@ -525,7 +526,8 @@ if (!tmpLinks.isEmpty()) { // on a trouve des liens a reattacher for (ReplicationLink link : tmpLinks) { - addPostOperation(node, node, AttachLink.class, link); +// addPostOperation(node, node, AttachLink.class, link); + addPostOperation(node, link.getTarget(), AttachLink.class, link); } // ces liens ne sont plus a traiter links.removeAll(tmpLinks); @@ -533,6 +535,7 @@ } } + // List<ReplicationOperationDef> realOperations = new ArrayList<ReplicationOperationDef>(); public void adjustOperations(TopiaEntityIdsMap data) { for (TopiaEntityEnum e : getContracts()) { List<String> ids = data.get(e.getContract()); @@ -541,17 +544,50 @@ // le noeud n'est pas connu (ce n'est pas normal!) continue; } + List<ReplicationOperationDef> realOperations = + new ArrayList<ReplicationOperationDef>(); if (CollectionUtils.isEmpty(ids)) { + + if (log.isInfoEnabled()) { + log.info("skip operations on node " + node + + " (no data associated)"); + } + + // must only keep operation that are not directly linked with the node + // at the moment only attachLink is in that case ReplicationOperationDef[] operations = node.getOperations(); - log.info("skip operations on node " + node + - " (no data associated)"); for (ReplicationOperationDef op : operations) { - log.info(" skip " + op); + + if (!node.equals(op.getNode())) { + + // keep this operation + realOperations.add(op); + if (log.isInfoEnabled()) { + log.info(" keep " + op); + } + } else { + if (log.isInfoEnabled()) { + log.info(" skip " + op); + } + } } - node.clearOperations(); - } else { - node.sortOperations(); + node.setOperations(realOperations); } + + // always sort operations on the phase + node.sortOperations(); + +// if (CollectionUtils.isEmpty(ids)) { +// ReplicationOperationDef[] operations = node.getOperations(); +// log.info("skip operations on node " + node + +// " (no data associated)"); +// for (ReplicationOperationDef op : operations) { +// log.info(" skip " + op); +// } +// node.clearOperations(); +// } else { +// node.sortOperations(); +// } } } Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationNode.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationNode.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/model/ReplicationNode.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -117,6 +117,11 @@ } } + public void setOperations(List<ReplicationOperationDef> operations) { + clearOperations(); + this.operations.addAll(operations); + } + public ReplicationOperationDef[] getOperations() { return operations.toArray( new ReplicationOperationDef[operations.size()]); Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachAssociation.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachAssociation.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachAssociation.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -77,7 +77,10 @@ * * @author tchemit <chemit(a)codelutin.com> * @since 2.2.0 + * @deprecated since 2.5.2, this operation will be removed in version 2.6 and + * will not be replaced : prefer use the {@link AttachLink} instead. */ +@Deprecated public class AttachAssociation implements TopiaReplicationOperation { /** Logger */ @@ -89,8 +92,8 @@ ReplicationOperationPhase phase, Object... parameters) { - throw new UnsupportedOperationException( - _("topia.replication.error.operation.uncreatable", getClass())); +// throw new UnsupportedOperationException( +// _("topia.replication.error.operation.uncreatable", getClass())); } @Override Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachLink.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachLink.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/AttachLink.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -80,8 +80,8 @@ ReplicationOperationPhase phase, Object... parameters) { - throw new UnsupportedOperationException( - _("topia.replication.error.operation.uncreatable", getClass())); +// throw new UnsupportedOperationException( +// _("topia.replication.error.operation.uncreatable", getClass())); } @Override Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/DettachAssociation.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/DettachAssociation.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/DettachAssociation.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -40,8 +40,6 @@ import java.util.List; -import static org.nuiton.i18n.I18n._; - /** * Pour dettacher une association. * @@ -54,7 +52,9 @@ */ public class DettachAssociation implements TopiaReplicationOperation { - /** Logger */ + /** + * Logger + */ private static final Log log = LogFactory.getLog(DettachAssociation.class); @Override @@ -63,8 +63,8 @@ ReplicationOperationPhase phase, Object... parameters) { - throw new UnsupportedOperationException( - _("topia.replication.error.operation.uncreatable", getClass())); +// throw new UnsupportedOperationException( +// _("topia.replication.error.operation.uncreatable", getClass())); } @Override @@ -87,7 +87,7 @@ if (size > 0) { if (log.isDebugEnabled()) { log.debug("will dettach " + size + " association(s) '" + - name + "' from " + e); + name + "' from " + e); } operator.setNull(name, e); } Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/Duplicate.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/Duplicate.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/Duplicate.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -41,8 +41,6 @@ import java.util.List; -import static org.nuiton.i18n.I18n._; - /** * L'operation de duplication d'un noeud de replication. * @@ -56,7 +54,9 @@ public class Duplicate implements TopiaReplicationOperationUndoable { - /** Logger */ + /** + * Logger + */ private static final Log log = LogFactory.getLog(Duplicate.class); @@ -65,8 +65,8 @@ ReplicationNode ownerNode, ReplicationOperationPhase phase, Object... parameters) throws UnsupportedOperationException { - throw new UnsupportedOperationException( - _("topia.replication.error.operation.uncreatable", getClass())); +// throw new UnsupportedOperationException( +// _("topia.replication.error.operation.uncreatable", getClass())); } @Override Modified: trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/LoadLink.java =================================================================== --- trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/LoadLink.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/main/java/org/nuiton/topia/replication/operation/LoadLink.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -64,8 +64,8 @@ ReplicationOperationPhase phase, Object... parameters) { - throw new UnsupportedOperationException( - _("topia.replication.error.operation.uncreatable", getClass())); +// throw new UnsupportedOperationException( +// _("topia.replication.error.operation.uncreatable", getClass())); } @Override Modified: trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/AbstractTopiaReplicationServiceTest.java =================================================================== --- trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/AbstractTopiaReplicationServiceTest.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/AbstractTopiaReplicationServiceTest.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -594,6 +594,7 @@ } } + @Deprecated protected void createUnsupportedBeforeOperation(TopiaEntityEnum contract, TopiaEntity entity, Class<? extends TopiaReplicationOperation> operationClass, @@ -607,6 +608,7 @@ fail(); } + @Deprecated protected void createUnsupportedAfterOperation( TopiaEntityEnum contract, TopiaEntity entity, @@ -622,6 +624,34 @@ fail(); } + protected void createSupportedBeforeOperation(TopiaEntityEnum contract, + TopiaEntity entity, + Class<? extends TopiaReplicationOperation> operationClass, + Object... parameters) throws Exception { + + getLog().info("entity " + entity.getTopiaId()); + prepareModel(entity.getTopiaId()); + + getModelBuilder().addBeforeOperation(model, contract, operationClass, parameters); + // on doit avoir le droit de creer cette operation + Assert.assertTrue(true); + } + + protected void createSupportedAfterOperation( + TopiaEntityEnum contract, + TopiaEntity entity, + Class<? extends TopiaReplicationOperation> operationClass, + Object... parameters) throws Exception { + + getLog().info("entity " + entity.getTopiaId()); + prepareModel(entity.getTopiaId()); +// model = service.createModel(getContracts()); +// model.detectDirectDependencies(); + getModelBuilder().addAfterOperation(model, contract, operationClass, parameters); + // on doit avoir le droit de creer cette operation + Assert.assertTrue(true); + } + protected Long getTestsTimeStamp() { if (testsTimeStamp == null) { testsTimeStamp = System.currentTimeMillis(); Modified: trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/TopiaReplicationOperationTest.java =================================================================== --- trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/TopiaReplicationOperationTest.java 2011-02-09 11:00:14 UTC (rev 2202) +++ trunk/topia-service-replication/src/test/java/org/nuiton/topia/replication/TopiaReplicationOperationTest.java 2011-02-09 11:12:15 UTC (rev 2203) @@ -27,11 +27,7 @@ import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import org.junit.After; -import org.junit.AfterClass; -import org.junit.Before; -import org.junit.BeforeClass; -import org.junit.Test; +import org.junit.*; import org.nuiton.topia.TestHelper; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaContextFactory; @@ -39,18 +35,8 @@ import org.nuiton.topia.framework.TopiaContextImplementor; import org.nuiton.topia.persistence.TopiaEntityEnum; import org.nuiton.topia.replication.model.ReplicationOperationPhase; -import org.nuiton.topia.replication.operation.AttachAssociation; -import org.nuiton.topia.replication.operation.DettachAssociation; -import org.nuiton.topia.replication.operation.Duplicate; -import org.nuiton.topia.replication.operation.FakeOperation; -import org.nuiton.topia.replication.operation.UncreatableOperation; -import org.nuiton.topia.replication.operation.UnregistredOperation; -import org.nuiton.topia.test.entities.Person; -import org.nuiton.topia.test.entities.PersonImpl; -import org.nuiton.topia.test.entities.Pet; -import org.nuiton.topia.test.entities.PetImpl; -import org.nuiton.topia.test.entities.Race; -import org.nuiton.topia.test.entities.RaceImpl; +import org.nuiton.topia.replication.operation.*; +import org.nuiton.topia.test.entities.*; import java.io.File; import java.io.IOException; @@ -140,7 +126,7 @@ getOperation(UncreatableOperation.class, true); getOperation(FakeOperation.class, true); getOperation(Duplicate.class, true); - getOperation(AttachAssociation.class, true); +// getOperation(AttachAssociation.class, true); getOperation(DettachAssociation.class, true); } @@ -186,45 +172,49 @@ getModelBuilder().addAfterOperation(model, TopiaTestEntityEnum.Pet, UnregistredOperation.class); } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedBeforeOperation_Duplicate() throws Exception { - createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, Duplicate.class); + @Test +// @Test(expected = UnsupportedOperationException.class) + public void testCreateSupportedBeforeOperation_Duplicate() throws Exception { + createSupportedBeforeOperation(TopiaTestEntityEnum.Person, person, Duplicate.class); } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedBeforeOperation_AttachAssociation() throws Exception { - createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, AttachAssociation.class); - } +// @Test(expected = UnsupportedOperationException.class) +// public void testCreateUnsupportedBeforeOperation_AttachAssociation() throws Exception { +// createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, AttachAssociation.class); +// } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedBeforeOperation_DetachAssociation() throws Exception { - createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, DettachAssociation.class); + @Test +// @Test(expected = UnsupportedOperationException.class) + public void testCreateSupportedBeforeOperation_DetachAssociation() throws Exception { + createSupportedBeforeOperation(TopiaTestEntityEnum.Person, person, DettachAssociation.class); } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedAfterOperation_Duplicate() throws Exception { - createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, Duplicate.class); + @Test +// @Test(expected = UnsupportedOperationException.class) + public void testCreateSupportedAfterOperation_Duplicate() throws Exception { + createSupportedAfterOperation(TopiaTestEntityEnum.Person, person, Duplicate.class); } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedAfterOperation_AttachAssociation() throws Exception { - createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, AttachAssociation.class); - } +// @Test(expected = UnsupportedOperationException.class) +// public void testCreateSupportedAfterOperation_AttachAssociation() throws Exception { +// createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, AttachAssociation.class); +// } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedAfterOperation_DetachAssociation() throws Exception { - createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, DettachAssociation.class); + @Test +// @Test(expected = UnsupportedOperationException.class) + public void testCreateSupportedAfterOperation_DetachAssociation() throws Exception { + createSupportedAfterOperation(TopiaTestEntityEnum.Person, person, DettachAssociation.class); } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedBeforeOperation_UncreatableOperation() throws Exception { - createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, UncreatableOperation.class); - } +// @Test(expected = UnsupportedOperationException.class) +// public void testCreateUnsupportedBeforeOperation_UncreatableOperation() throws Exception { +// createUnsupportedBeforeOperation(TopiaTestEntityEnum.Person, person, UncreatableOperation.class); +// } - @Test(expected = UnsupportedOperationException.class) - public void testCreateUnsupportedAfterOperation_UncreatableOperation() throws Exception { - createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, UncreatableOperation.class); - } +// @Test(expected = UnsupportedOperationException.class) +// public void testCreateUnsupportedAfterOperation_UncreatableOperation() throws Exception { +// createUnsupportedAfterOperation(TopiaTestEntityEnum.Person, person, UncreatableOperation.class); +// } @Test(expected = IllegalArgumentException.class) public void testCreateOperation_wrongParameterNumber() throws Exception {

1 0