From 46558d3d7f709de508c6432cec5d93fdfeeb385b Mon Sep 17 00:00:00 2001 From: David Montag Date: Mon, 28 Mar 2011 18:49:36 -0700 Subject: [PATCH] Renamed NodeTypeStrategy to TypeRepresentationStrategy --- .../src/test/resources/log4j.properties | 2 +- .../jpa/Neo4jEntityManagerTest-context.xml | 4 +- ...y.java => TypeRepresentationStrategy.java} | 3 +- .../neo4j/config/Neo4jConfiguration.java | 2 +- .../graph/neo4j/finder/AbstractFinder.java | 2 +- .../neo4j/support/GraphDatabaseContext.java | 36 +++++++++--------- ...> IndexingTypeRepresentationStrategy.java} | 6 +-- .../support/NodeTypeStrategyFactoryBean.java | 38 +++++++++---------- ...va => NoopTypeRepresentationStrategy.java} | 10 ++--- ...bReferenceTypeRepresentationStrategy.java} | 10 ++--- .../support/IndexingNodeTypeStrategyTest.java | 3 +- .../support/NoopNodeTypeStrategyTest.java | 2 +- .../SubReferenceNodeTypeStrategyTest.java | 12 +++--- .../src/test/resources/log4j.properties | 2 +- .../Neo4jGraphRecommendationTest-context.xml | 4 +- ...dexingNodeTypeStrategyOverride-context.xml | 2 +- .../Neo4jGraphPersistenceTest-context.xml | 4 +- .../NoopNodeTypeStrategyOverride-context.xml | 2 +- ...erenceNodeTypeStrategyOverride-context.xml | 2 +- 19 files changed, 71 insertions(+), 75 deletions(-) rename spring-data-neo4j/src/main/java/org/springframework/data/graph/core/{NodeTypeStrategy.java => TypeRepresentationStrategy.java} (96%) rename spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/{IndexingNodeTypeStrategy.java => IndexingTypeRepresentationStrategy.java} (93%) rename spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/{NoopNodeTypeStrategy.java => NoopTypeRepresentationStrategy.java} (75%) rename spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/{SubReferenceNodeTypeStrategy.java => SubReferenceTypeRepresentationStrategy.java} (95%) diff --git a/spring-data-neo4j-rest/src/test/resources/log4j.properties b/spring-data-neo4j-rest/src/test/resources/log4j.properties index e69fbde1b..9b295b169 100644 --- a/spring-data-neo4j-rest/src/test/resources/log4j.properties +++ b/spring-data-neo4j-rest/src/test/resources/log4j.properties @@ -18,7 +18,7 @@ log4j.appender.R.layout.ConversionPattern=%p %t %c - %m%n log4j.category.org.springframework=WARN -#log4j.category.org.springframework.data.graph.neo4j.support.SubReferenceNodeTypeStrategy=DEBUG +#log4j.category.org.springframework.data.graph.neo4j.support.SubReferenceTypeRepresentationStrategy=DEBUG #log4j.category.org.springframework.data.graph.neo4j.fieldaccess=DEBUG #log4j.category.org.springframework.data=TRACE #log4j.category.org.springframework.data.support=TRACE diff --git a/spring-data-neo4j-roo/src/test/resources/org/springframework/data/graph/neo4j/jpa/Neo4jEntityManagerTest-context.xml b/spring-data-neo4j-roo/src/test/resources/org/springframework/data/graph/neo4j/jpa/Neo4jEntityManagerTest-context.xml index dddb3438c..f37b31c94 100644 --- a/spring-data-neo4j-roo/src/test/resources/org/springframework/data/graph/neo4j/jpa/Neo4jEntityManagerTest-context.xml +++ b/spring-data-neo4j-roo/src/test/resources/org/springframework/data/graph/neo4j/jpa/Neo4jEntityManagerTest-context.xml @@ -97,10 +97,10 @@ - + - + diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/core/NodeTypeStrategy.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/core/TypeRepresentationStrategy.java similarity index 96% rename from spring-data-neo4j/src/main/java/org/springframework/data/graph/core/NodeTypeStrategy.java rename to spring-data-neo4j/src/main/java/org/springframework/data/graph/core/TypeRepresentationStrategy.java index 1b8485f6a..af3be4df0 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/core/NodeTypeStrategy.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/core/TypeRepresentationStrategy.java @@ -17,7 +17,6 @@ package org.springframework.data.graph.core; import org.neo4j.graphdb.Node; -import org.springframework.data.graph.core.NodeBacked; /** * Strategy to handle representation of java types in the graph. Possible implementation are type/class nodes @@ -30,7 +29,7 @@ import org.springframework.data.graph.core.NodeBacked; * @author Michael Hunger * @since 13.09.2010 */ -public interface NodeTypeStrategy { +public interface TypeRepresentationStrategy { /** * callback on entity creation for setting up type representation * @param entity diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/config/Neo4jConfiguration.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/config/Neo4jConfiguration.java index deabfbf34..bba485461 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/config/Neo4jConfiguration.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/config/Neo4jConfiguration.java @@ -95,7 +95,7 @@ public class Neo4jConfiguration { gdc.setGraphEntityInstantiator(graphEntityInstantiator); gdc.setConversionService(conversionService()); NodeTypeStrategyFactoryBean nodeTypeStrategyFactoryBean = new NodeTypeStrategyFactoryBean(graphDatabaseService, graphEntityInstantiator); - gdc.setNodeTypeStrategy(nodeTypeStrategyFactoryBean.getObject()); + gdc.setTypeRepresentationStrategy(nodeTypeStrategyFactoryBean.getObject()); if (validator!=null) { gdc.setValidator(validator); } diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/finder/AbstractFinder.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/finder/AbstractFinder.java index 90bd5fe9d..5a1499cfb 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/finder/AbstractFinder.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/finder/AbstractFinder.java @@ -12,7 +12,7 @@ import org.springframework.data.graph.neo4j.support.GraphDatabaseContext; import java.util.Collections; /** - * Repository like finder for Node and Relationship-Entities. Provides finder methods for direct access, access via {@link org.springframework.data.graph.core.NodeTypeStrategy} + * Repository like finder for Node and Relationship-Entities. Provides finder methods for direct access, access via {@link org.springframework.data.graph.core.TypeRepresentationStrategy} * and indexing. * * @param GraphBacked target of this finder, enables the finder methods to return this concrete type diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/GraphDatabaseContext.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/GraphDatabaseContext.java index ac24b6925..751f80e09 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/GraphDatabaseContext.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/GraphDatabaseContext.java @@ -27,7 +27,7 @@ import org.springframework.core.convert.ConversionService; import org.springframework.data.annotation.Indexed; import org.springframework.data.graph.core.GraphBacked; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; +import org.springframework.data.graph.core.TypeRepresentationStrategy; import org.springframework.data.graph.core.RelationshipBacked; import org.springframework.data.persistence.EntityInstantiator; @@ -39,7 +39,7 @@ import java.util.Map; /** * Mediator class for the graph related services like the {@link GraphDatabaseService}, the used - * {@link NodeTypeStrategy}, entity instantiators for nodes and relationships as well as a spring conversion service. + * {@link org.springframework.data.graph.core.TypeRepresentationStrategy}, entity instantiators for nodes and relationships as well as a spring conversion service. * * It delegates the appropriate methods to those services. The services are not intended to be accessible from outside. * @@ -59,7 +59,7 @@ public class GraphDatabaseContext { private ConversionService conversionService; - private NodeTypeStrategy nodeTypeStrategy; + private TypeRepresentationStrategy typeRepresentationStrategy; private Validator validator; @@ -99,12 +99,12 @@ public class GraphDatabaseContext { this.conversionService = conversionService; } - public NodeTypeStrategy getNodeTypeStrategy() { - return nodeTypeStrategy; + public TypeRepresentationStrategy getTypeRepresentationStrategy() { + return typeRepresentationStrategy; } - public void setNodeTypeStrategy(NodeTypeStrategy nodeTypeStrategy) { - this.nodeTypeStrategy = nodeTypeStrategy; + public void setTypeRepresentationStrategy(TypeRepresentationStrategy typeRepresentationStrategy) { + this.typeRepresentationStrategy = typeRepresentationStrategy; } public Node createNode() { @@ -139,7 +139,7 @@ public class GraphDatabaseContext { public void removeNodeEntity(NodeBacked entity) { Node node = entity.getPersistentState(); if (node==null) return; - this.nodeTypeStrategy.preEntityRemoval(entity); + this.typeRepresentationStrategy.preEntityRemoval(entity); for (Relationship relationship : node.getRelationships()) { removeRelationship(relationship); } @@ -171,7 +171,7 @@ public class GraphDatabaseContext { public T createEntityFromState(final S state, final Class type) { if (state==null) throw new IllegalArgumentException("state has to be either a Node or Relationship, not null"); if (state instanceof Node) - return (T) graphEntityInstantiator.createEntityFromState((Node) state, nodeTypeStrategy.confirmType((Node)state, (Class)type)); + return (T) graphEntityInstantiator.createEntityFromState((Node) state, typeRepresentationStrategy.confirmType((Node)state, (Class)type)); else return (T) relationshipEntityInstantiator.createEntityFromState((Relationship) state, (Class) type); } @@ -214,15 +214,15 @@ public class GraphDatabaseContext { } /** - * delegates to the configured @{link NodeTypeStrategy} for after entity creation operations + * delegates to the configured @{link TypeRepresentationStrategy} for after entity creation operations * @param entity */ public void postEntityCreation(final NodeBacked entity) { - nodeTypeStrategy.postEntityCreation(entity); + typeRepresentationStrategy.postEntityCreation(entity); } /** - * delegates to the configured @{link NodeTypeStrategy} to iterate over all instances of this type + * delegates to the configured @{link TypeRepresentationStrategy} to iterate over all instances of this type * @param clazz type of entity * @param * @return @@ -230,7 +230,7 @@ public class GraphDatabaseContext { */ public Iterable findAll(final Class clazz) { if (!checkIsNodeBacked(clazz)) throw new UnsupportedOperationException("No support for relationships"); - return (Iterable) nodeTypeStrategy.findAll((Class)clazz); + return (Iterable) typeRepresentationStrategy.findAll((Class)clazz); } /** @@ -241,24 +241,24 @@ public class GraphDatabaseContext { } /** - * delegates to the configured @{link NodeTypeStrategy} for a count of all instances of this type + * delegates to the configured @{link TypeRepresentationStrategy} for a count of all instances of this type * @param entityClass * @return count of all instances */ public long count(final Class entityClass) { if (!checkIsNodeBacked(entityClass)) throw new UnsupportedOperationException("No support for relationships"); - return nodeTypeStrategy.count((Class)entityClass); + return typeRepresentationStrategy.count((Class)entityClass); } /** - * delegates to the configured @{link NodeTypeStrategy} to lookup the type information for the given node + * delegates to the configured @{link TypeRepresentationStrategy} to lookup the type information for the given node * @param node * @param * @return entity type of the node * @throws IllegalStateException for nodes that are not instance backing nodes of a known type */ public Class getJavaType(final Node node) { - return nodeTypeStrategy.getJavaType(node); + return typeRepresentationStrategy.getJavaType(node); } /** @@ -327,7 +327,7 @@ public class GraphDatabaseContext { } public T createEntityFromStoredType(Node node) { - return (T)graphEntityInstantiator.createEntityFromState(node,nodeTypeStrategy.getJavaType(node)); + return (T)graphEntityInstantiator.createEntityFromState(node, typeRepresentationStrategy.getJavaType(node)); } } diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategy.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingTypeRepresentationStrategy.java similarity index 93% rename from spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategy.java rename to spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingTypeRepresentationStrategy.java index f5470c0b0..ff29cf854 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategy.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/IndexingTypeRepresentationStrategy.java @@ -10,13 +10,13 @@ import org.neo4j.helpers.collection.FilteringIterable; import org.neo4j.helpers.collection.IterableWrapper; import org.springframework.data.graph.annotation.NodeEntity; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; +import org.springframework.data.graph.core.TypeRepresentationStrategy; import org.springframework.data.persistence.EntityInstantiator; import java.util.HashMap; import java.util.Map; -public class IndexingNodeTypeStrategy implements NodeTypeStrategy { +public class IndexingTypeRepresentationStrategy implements TypeRepresentationStrategy { public static final String NODE_INDEX_NAME = "__types__"; public static final String TYPE_PROPERTY_NAME = "__type__"; @@ -25,7 +25,7 @@ public class IndexingNodeTypeStrategy implements NodeTypeStrategy { private GraphDatabaseService graphDb; private final Map> cache=new HashMap>(); - public IndexingNodeTypeStrategy(GraphDatabaseService graphDb, EntityInstantiator graphEntityInstantiator) { + public IndexingTypeRepresentationStrategy(GraphDatabaseService graphDb, EntityInstantiator graphEntityInstantiator) { this.graphDb = graphDb; this.graphEntityInstantiator = graphEntityInstantiator; } diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NodeTypeStrategyFactoryBean.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NodeTypeStrategyFactoryBean.java index e3d97b7b0..694abf51f 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NodeTypeStrategyFactoryBean.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NodeTypeStrategyFactoryBean.java @@ -5,10 +5,10 @@ import org.neo4j.graphdb.Node; import org.neo4j.graphdb.Relationship; import org.springframework.beans.factory.FactoryBean; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; +import org.springframework.data.graph.core.TypeRepresentationStrategy; import org.springframework.data.persistence.EntityInstantiator; -public class NodeTypeStrategyFactoryBean implements FactoryBean { +public class NodeTypeStrategyFactoryBean implements FactoryBean { private GraphDatabaseService graphDatabaseService; private EntityInstantiator graphEntityInstantiator; private Strategy strategy; @@ -26,12 +26,12 @@ public class NodeTypeStrategyFactoryBean implements FactoryBean graphEntityInstantiator) { - return new SubReferenceNodeTypeStrategy(graphDatabaseService, graphEntityInstantiator); + TypeRepresentationStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator) { + return new SubReferenceTypeRepresentationStrategy(graphDatabaseService, graphEntityInstantiator); } @Override - Class getObjectType() { - return SubReferenceNodeTypeStrategy.class; + Class getObjectType() { + return SubReferenceTypeRepresentationStrategy.class; } }, Indexed { @Override - NodeTypeStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator) { - return new IndexingNodeTypeStrategy(graphDatabaseService, graphEntityInstantiator); + TypeRepresentationStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator) { + return new IndexingTypeRepresentationStrategy(graphDatabaseService, graphEntityInstantiator); } @Override - Class getObjectType() { - return IndexingNodeTypeStrategy.class; + Class getObjectType() { + return IndexingTypeRepresentationStrategy.class; } }, Noop { @Override - NodeTypeStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator) { - return new NoopNodeTypeStrategy(); + TypeRepresentationStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator) { + return new NoopTypeRepresentationStrategy(); } @Override - Class getObjectType() { - return NoopNodeTypeStrategy.class; + Class getObjectType() { + return NoopTypeRepresentationStrategy.class; } }; - abstract NodeTypeStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator); - abstract Class getObjectType(); + abstract TypeRepresentationStrategy getObject(GraphDatabaseService graphDatabaseService, EntityInstantiator graphEntityInstantiator); + abstract Class getObjectType(); } } diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategy.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopTypeRepresentationStrategy.java similarity index 75% rename from spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategy.java rename to spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopTypeRepresentationStrategy.java index 76c0822a7..387e0c71c 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategy.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/NoopTypeRepresentationStrategy.java @@ -2,26 +2,26 @@ package org.springframework.data.graph.neo4j.support; import org.neo4j.graphdb.Node; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; +import org.springframework.data.graph.core.TypeRepresentationStrategy; -public class NoopNodeTypeStrategy implements NodeTypeStrategy { +public class NoopTypeRepresentationStrategy implements TypeRepresentationStrategy { @Override public void postEntityCreation(NodeBacked entity) { } @Override public Iterable findAll(Class clazz) { - throw new UnsupportedOperationException("findAll not supported by NoopNodeTypeStrategy."); + throw new UnsupportedOperationException("findAll not supported by NoopTypeRepresentationStrategy."); } @Override public long count(Class entityClass) { - throw new UnsupportedOperationException("count not supported by NoopNodeTypeStrategy."); + throw new UnsupportedOperationException("count not supported by NoopTypeRepresentationStrategy."); } @Override public Class getJavaType(Node node) { - throw new UnsupportedOperationException("getJavaType not supported NoopNodeTypeStrategy."); + throw new UnsupportedOperationException("getJavaType not supported NoopTypeRepresentationStrategy."); } @Override diff --git a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategy.java b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceTypeRepresentationStrategy.java similarity index 95% rename from spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategy.java rename to spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceTypeRepresentationStrategy.java index 9cfc3728b..8d7027474 100644 --- a/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategy.java +++ b/spring-data-neo4j/src/main/java/org/springframework/data/graph/neo4j/support/SubReferenceTypeRepresentationStrategy.java @@ -24,13 +24,13 @@ import org.neo4j.helpers.collection.CombiningIterable; import org.neo4j.helpers.collection.IterableWrapper; import org.neo4j.kernel.impl.traversal.TraversalDescriptionImpl; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; +import org.springframework.data.graph.core.TypeRepresentationStrategy; import org.springframework.data.persistence.EntityInstantiator; import java.util.*; /** - * A {@link NodeTypeStrategy} that uses a hierarchy of reference nodes to represent the java type of the entity in the + * A {@link org.springframework.data.graph.core.TypeRepresentationStrategy} that uses a hierarchy of reference nodes to represent the java type of the entity in the * graph database. Entity nodes are related to their concrete type via an INSTANCE_OF relationship, the type hierarchy is * related to supertypes via SUBCLASS_OF relationships. Each concrete subreference node keeps a count property with the number of * instances of this class in the graph. @@ -38,8 +38,8 @@ import java.util.*; * @author Michael Hunger * @since 13.09.2010 */ -public class SubReferenceNodeTypeStrategy implements NodeTypeStrategy { - private final static Log log = LogFactory.getLog(SubReferenceNodeTypeStrategy.class); +public class SubReferenceTypeRepresentationStrategy implements TypeRepresentationStrategy { + private final static Log log = LogFactory.getLog(SubReferenceTypeRepresentationStrategy.class); public final static RelationshipType INSTANCE_OF_RELATIONSHIP_TYPE = DynamicRelationshipType.withName("INSTANCE_OF"); public final static RelationshipType SUBCLASS_OF_RELATIONSHIP_TYPE = DynamicRelationshipType.withName("SUBCLASS_OF"); @@ -51,7 +51,7 @@ public class SubReferenceNodeTypeStrategy implements NodeTypeStrategy { private GraphDatabaseService graphDatabaseService; private EntityInstantiator entityInstantiator; - public SubReferenceNodeTypeStrategy(GraphDatabaseService graphDatabaseService, EntityInstantiator entityInstantiator) { + public SubReferenceTypeRepresentationStrategy(GraphDatabaseService graphDatabaseService, EntityInstantiator entityInstantiator) { this.graphDatabaseService = graphDatabaseService; this.entityInstantiator = entityInstantiator; } diff --git a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyTest.java b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyTest.java index 2076af873..018ad6413 100644 --- a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyTest.java +++ b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyTest.java @@ -1,7 +1,6 @@ package org.springframework.data.graph.neo4j.support; import org.junit.Before; -import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; import org.neo4j.graphdb.GraphDatabaseService; @@ -39,7 +38,7 @@ public class IndexingNodeTypeStrategyTest { @Autowired private GraphDatabaseService graphDatabaseService; @Autowired - private IndexingNodeTypeStrategy nodeTypeStrategy; + private IndexingTypeRepresentationStrategy nodeTypeStrategy; private Thing thing; private SubThing subThing; diff --git a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyTest.java b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyTest.java index e5a7c83ae..08a46db89 100644 --- a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyTest.java +++ b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyTest.java @@ -25,7 +25,7 @@ public class NoopNodeTypeStrategyTest { @Autowired private GraphDatabaseContext graphDatabaseContext; @Autowired - private NoopNodeTypeStrategy nodeTypeStrategy; + private NoopTypeRepresentationStrategy nodeTypeStrategy; private Thing thing; diff --git a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyTest.java b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyTest.java index ba1871523..f75bf0760 100644 --- a/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyTest.java +++ b/spring-data-neo4j/src/test/java/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyTest.java @@ -14,7 +14,6 @@ import org.neo4j.helpers.collection.IteratorUtil; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.data.graph.annotation.NodeEntity; import org.springframework.data.graph.core.NodeBacked; -import org.springframework.data.graph.core.NodeTypeStrategy; import org.springframework.data.graph.neo4j.Car; import org.springframework.data.graph.neo4j.Person; import static org.springframework.data.graph.neo4j.Person.persistedPerson; @@ -24,7 +23,6 @@ import org.springframework.data.graph.neo4j.finder.FinderFactory; import org.springframework.data.graph.neo4j.finder.NodeFinder; import org.springframework.data.graph.neo4j.support.node.Neo4jHelper; -import org.springframework.test.annotation.DirtiesContext; import org.springframework.test.context.CleanContextCacheTestExecutionListener; import org.springframework.test.context.ContextConfiguration; import org.springframework.test.context.TestExecutionListeners; @@ -55,7 +53,7 @@ public class SubReferenceNodeTypeStrategyTest { @Autowired private FinderFactory finderFactory; @Autowired - private SubReferenceNodeTypeStrategy nodeTypeStrategy; + private SubReferenceTypeRepresentationStrategy nodeTypeStrategy; private Node thingNode; private Thing thing; @@ -75,8 +73,8 @@ public class SubReferenceNodeTypeStrategyTest { public void testPostEntityCreation() throws Exception { Node typeNode = getInstanceofRelationship().getOtherNode(thingNode); Assert.assertNotNull("type node for thing exists", typeNode); - Assert.assertEquals("type node has property of type Thing.class", Thing.class.getName(), typeNode.getProperty(SubReferenceNodeTypeStrategy.SUBREF_CLASS_KEY)); - Assert.assertEquals("one thing has been created", 1, typeNode.getProperty(SubReferenceNodeTypeStrategy.SUBREFERENCE_NODE_COUNTER_KEY)); + Assert.assertEquals("type node has property of type Thing.class", Thing.class.getName(), typeNode.getProperty(SubReferenceTypeRepresentationStrategy.SUBREF_CLASS_KEY)); + Assert.assertEquals("one thing has been created", 1, typeNode.getProperty(SubReferenceTypeRepresentationStrategy.SUBREFERENCE_NODE_COUNTER_KEY)); } @Test(expected = IllegalArgumentException.class) public void gettingTypeFromNonTypeNodeShouldThrowAnDescriptiveException() throws Exception { @@ -115,13 +113,13 @@ public class SubReferenceNodeTypeStrategyTest { Node typeNode = getInstanceofRelationship().getOtherNode(thingNode); nodeTypeStrategy.preEntityRemoval(thing); Assert.assertNull("instanceof relationship was removed", getInstanceofRelationship()); - Assert.assertEquals("no things left after removal", 0, typeNode.getProperty(SubReferenceNodeTypeStrategy.SUBREFERENCE_NODE_COUNTER_KEY)); + Assert.assertEquals("no things left after removal", 0, typeNode.getProperty(SubReferenceTypeRepresentationStrategy.SUBREFERENCE_NODE_COUNTER_KEY)); } @Transactional private Relationship getInstanceofRelationship() { - return thingNode.getSingleRelationship(SubReferenceNodeTypeStrategy.INSTANCE_OF_RELATIONSHIP_TYPE, Direction.OUTGOING); + return thingNode.getSingleRelationship(SubReferenceTypeRepresentationStrategy.INSTANCE_OF_RELATIONSHIP_TYPE, Direction.OUTGOING); } @Test diff --git a/spring-data-neo4j/src/test/resources/log4j.properties b/spring-data-neo4j/src/test/resources/log4j.properties index e69fbde1b..c33528705 100644 --- a/spring-data-neo4j/src/test/resources/log4j.properties +++ b/spring-data-neo4j/src/test/resources/log4j.properties @@ -18,7 +18,7 @@ log4j.appender.R.layout.ConversionPattern=%p %t %c - %m%n log4j.category.org.springframework=WARN -#log4j.category.org.springframework.data.graph.neo4j.support.SubReferenceNodeTypeStrategy=DEBUG +#log4j.category.org.springframework.data.graph.neo4j.support.SubReferenceTypeRepresentationStrategyonStrategy=DEBUG #log4j.category.org.springframework.data.graph.neo4j.fieldaccess=DEBUG #log4j.category.org.springframework.data=TRACE #log4j.category.org.springframework.data.support=TRACE diff --git a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/partial/Neo4jGraphRecommendationTest-context.xml b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/partial/Neo4jGraphRecommendationTest-context.xml index 35745814d..43b2288e9 100644 --- a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/partial/Neo4jGraphRecommendationTest-context.xml +++ b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/partial/Neo4jGraphRecommendationTest-context.xml @@ -100,10 +100,10 @@ - + - + diff --git a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyOverride-context.xml b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyOverride-context.xml index 8c0941652..6b5470375 100644 --- a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyOverride-context.xml +++ b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/IndexingNodeTypeStrategyOverride-context.xml @@ -14,7 +14,7 @@ http://www.springframework.org/schema/tx http://www.springframework.org/schema/tx/spring-tx-3.0.xsd http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context-3.0.xsd"> - + diff --git a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/Neo4jGraphPersistenceTest-context.xml b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/Neo4jGraphPersistenceTest-context.xml index 4c20f924d..b8c13e80a 100644 --- a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/Neo4jGraphPersistenceTest-context.xml +++ b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/Neo4jGraphPersistenceTest-context.xml @@ -91,7 +91,7 @@ - + @@ -100,7 +100,7 @@ - + diff --git a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyOverride-context.xml b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyOverride-context.xml index b535caf13..052d9b45e 100644 --- a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyOverride-context.xml +++ b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/NoopNodeTypeStrategyOverride-context.xml @@ -14,5 +14,5 @@ http://www.springframework.org/schema/tx http://www.springframework.org/schema/tx/spring-tx-3.0.xsd http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context-3.0.xsd"> - + \ No newline at end of file diff --git a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyOverride-context.xml b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyOverride-context.xml index e2d8e83f7..31fb50433 100644 --- a/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyOverride-context.xml +++ b/spring-data-neo4j/src/test/resources/org/springframework/data/graph/neo4j/support/SubReferenceNodeTypeStrategyOverride-context.xml @@ -14,7 +14,7 @@ http://www.springframework.org/schema/tx http://www.springframework.org/schema/tx/spring-tx-3.0.xsd http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context-3.0.xsd"> - +