Merge branch 'master' into 0.7-and-artifact-id-change

Author: Oleksii Tymchenko

.idea/inspectionProfiles/Project_Default.xml

@@ -30,7 +30,7 @@
 import org.spine3.client.CommandFactory;
 import org.spine3.protobuf.AnyPacker;
 import org.spine3.protobuf.Timestamps;
-import org.spine3.protobuf.TypeUrl;
+import org.spine3.protobuf.TypeName;
 import org.spine3.time.ZoneOffset;
 import org.spine3.users.TenantId;
 import org.spine3.users.UserId;
@@ -229,8 +229,7 @@ public static String formatMessageTypeAndId(String format, Message commandMessag
         checkNotNull(format);
         checkNotEmptyOrBlank(format, "format string");
 
-        final String cmdType = TypeUrl.of(commandMessage)
-                                      .getTypeName();
+        final String cmdType = TypeName.of(commandMessage);
         final String id = idToString(commandId);
         final String result = String.format(format, cmdType, id);
         return result;

client/src/main/java/org/spine3/base/Commands.java

@@ -27,7 +27,7 @@
 import com.google.protobuf.Timestamp;
 import org.spine3.protobuf.AnyPacker;
 import org.spine3.protobuf.Timestamps;
-import org.spine3.protobuf.TypeUrl;
+import org.spine3.protobuf.TypeName;
 import org.spine3.users.UserId;
 
 import javax.annotation.Nullable;
@@ -275,8 +275,7 @@ public static <E extends Message> Optional<E> getEnrichment(Class<E> enrichmentC
             return Optional.absent();
         }
         final Enrichments enrichments = value.get();
-        final String typeName = TypeUrl.of(enrichmentClass)
-                                       .getTypeName();
+        final String typeName = TypeName.of(enrichmentClass);
         final Any any = enrichments.getMapMap()
                                    .get(typeName);
         if (any == null) {

client/src/main/java/org/spine3/base/Events.java

@@ -40,6 +40,7 @@ public abstract class FailureThrowable extends Throwable {
     private final Timestamp timestamp;
 
     protected FailureThrowable(GeneratedMessageV3 failure) {
+        super();
         this.failure = failure;
         this.timestamp = getCurrentTime();
     }

client/src/main/java/org/spine3/base/FailureThrowable.java

@@ -29,6 +29,7 @@
 import org.spine3.client.Target;
 import org.spine3.protobuf.AnyPacker;
 import org.spine3.protobuf.KnownTypes;
+import org.spine3.protobuf.TypeName;
 import org.spine3.protobuf.TypeUrl;
 
 import javax.annotation.Nullable;
@@ -191,11 +192,9 @@ public static Target allOf(Class<? extends Message> entityClass) {
             return result;
         }
 
-        /* package */
-        static Target composeTarget(Class<? extends Message> entityClass, @Nullable Set<? extends Message> ids) {
-            final TypeUrl type = TypeUrl.of(entityClass);
-
-            final boolean includeAll = ids == null;
+        /* package */ static Target composeTarget(Class<? extends Message> entityClass,
+                                                  @Nullable Set<? extends Message> ids) {
+            final boolean includeAll = (ids == null);
 
             final EntityIdFilter.Builder idFilterBuilder = EntityIdFilter.newBuilder();
 
@@ -213,7 +212,7 @@ static Target composeTarget(Class<? extends Message> entityClass, @Nullable Set<
                                                        .setIdFilter(idFilter)
                                                        .build();
             final Target.Builder builder = Target.newBuilder()
-                                                 .setType(type.getTypeName());
+                                                 .setType(TypeName.of(entityClass));
             if (includeAll) {
                 builder.setIncludeAll(true);
             } else {

client/src/main/java/org/spine3/base/Queries.java

@@ -30,13 +30,15 @@
  */
 /* package */ class Mismatches {
 
+    private static final String ERR_CANNOT_BE_EQUAL = "`expected` and `actual` cannot be equal in ValueMismatch";
+
     private Mismatches() {
         // Prevent instantiations.
     }
 
     /* package */ static void checkNotNullOrEqual(Object expected, Object actual) {
         checkNotNull(expected);
         checkNotNull(actual);
-        checkArgument(!expected.equals(actual), ErrorMessage.EXPECTED_AND_ACTUAL_CANNOT_BE_EQUAL);
+        checkArgument(!expected.equals(actual), ERR_CANNOT_BE_EQUAL);
     }
 }

client/src/main/java/org/spine3/change/Mismatches.java

@@ -25,9 +25,14 @@
  *
  * @author Alexander Yevsyukov
  */
-public interface ConnectionConstants {
+public class ConnectionConstants {
 
-    /** The default port number on which a client server runs. */
-    int DEFAULT_CLIENT_SERVICE_PORT = 50051;
+    /**
+     * The default port number on which a client server runs.
+     */
+    public static final int DEFAULT_CLIENT_SERVICE_PORT = 50051;
 
+    private ConnectionConstants() {
+        // Prevent instantiation.
+    }
 }

client/src/main/java/org/spine3/client/ConnectionConstants.java

@@ -45,7 +45,7 @@ private AnyPacker() {}
      * @return the instance of {@link Any} object that wraps given message
      */
     public static Any pack(Message message) {
-        final TypeUrl typeUrl = TypeUrl.of(message.getDescriptorForType());
+        final TypeUrl typeUrl = TypeUrl.from(message.getDescriptorForType());
         final String typeUrlPrefix = typeUrl.getPrefix();
         final Any result = Any.pack(message, typeUrlPrefix);
         return result;

client/src/main/java/org/spine3/protobuf/AnyPacker.java

@@ -309,7 +309,7 @@ private void put(Class<? extends GeneratedMessageV3> clazz) {
         }
 
         private void putEnum(EnumDescriptor desc, Class<? extends EnumLite> enumClass) {
-            final TypeUrl typeUrl = TypeUrl.of(desc);
+            final TypeUrl typeUrl = TypeUrl.from(desc);
             final ClassName className = ClassName.of(enumClass);
             put(typeUrl, className);
         }

client/src/main/java/org/spine3/protobuf/KnownTypes.java

@@ -177,7 +177,7 @@ public static Class<?> getFieldClass(FieldDescriptor field) {
                 final Class<? extends Message> enumClass = toMessageClass(TypeUrl.of(enumTypeName));
                 return enumClass;
             case MESSAGE:
-                final TypeUrl typeUrl = TypeUrl.of(field.getMessageType());
+                final TypeUrl typeUrl = TypeUrl.from(field.getMessageType());
                 final Class<? extends Message> msgClass = toMessageClass(typeUrl);
                 return msgClass;
             default: