Merge pull request #292 from SpineEventEngine/get-part-state

Rename `getPart()` to `getPartState()`

Author: alexander-yevsyukov

build.gradle

@@ -33,7 +33,7 @@ buildscript {
         guavaVersion = '20.0'
         protobufGradlePluginVerison = '0.8.0'
 
-        spineVersion = '0.7.23-SNAPSHOT'
+        spineVersion = '0.7.24-SNAPSHOT'
         //TODO:2016-12-12:alexander.yevsyukov: Advance the plug-in version together with all the components and change
         // the version of the dependency below to `spineVersion` defined above.
         spineProtobufPluginVersion = '0.6.6-SNAPSHOT'

server/src/main/java/org/spine3/server/aggregate/AggregateRoot.java

@@ -82,7 +82,7 @@ protected BoundedContext getBoundedContext() {
      * @param <S> the type of the part state
      * @return the state of the part or a default state if the state was not found
      */
-    protected <S extends Message, A extends AggregatePart<I, S, ?>> S getPart(Class<S> partStateClass) {
+    protected <S extends Message, A extends AggregatePart<I, S, ?>> S getPartState(Class<S> partStateClass) {
         final AggregatePartRepository<I, A> repo = getRepository(partStateClass);
         final AggregatePart<I, S, ?> aggregatePart = repo.loadOrCreate(getId());
         final S partState = aggregatePart.getState();
@@ -92,8 +92,9 @@ protected BoundedContext getBoundedContext() {
     /**
      * Obtains a repository for the passed state class.
      *
-     * @throws IllegalStateException if a repository was not found
-     *                               or the repository does not match expectations of this {@code AggregateRoot}
+     * @throws IllegalStateException
+     *              if a repository was not found,
+     *              or the repository does not match expectations of this {@code AggregateRoot}
      */
     private <S extends Message, A extends AggregatePart<I, S, ?>>
             AggregatePartRepository<I, A> getRepository(Class<S> stateClass) {

server/src/test/java/org/spine3/server/aggregate/AggregateRootShould.java

@@ -56,7 +56,7 @@ public void setUp() {
 
     @Test
     public void return_part_state_by_class() {
-        final Message part = aggregateRoot.getPart(Project.class);
+        final Message part = aggregateRoot.getPartState(Project.class);
 
         assertNotNull(part);
     }
@@ -66,8 +66,8 @@ public void cache_repositories() {
         final AggregateRoot rootSpy = spy(aggregateRoot);
         final Class<Project> partClass = Project.class;
 
-        rootSpy.getPart(partClass);
-        rootSpy.getPart(partClass);
+        rootSpy.getPartState(partClass);
+        rootSpy.getPartState(partClass);
 
         // It may be called once in another test. So here we check for atMost().
         verify(rootSpy, atMost(1)).lookup(partClass);