Refine exception and messages

Author: cuzfrog

processor/src/main/java/org/sharedtype/processor/AnnotationProcessorImpl.java

@@ -66,7 +66,7 @@ public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment
             return ANNOTATION_CONSUMED;
         }
         if (annotations.size() > 1) {
-            throw new SharedTypeInternalError(String.format("Only '%s' is expected.", ANNOTATION_QUALIFIED_NAME));
+            throw new SharedTypeInternalError(String.format("Only annotation %s is expected.", ANNOTATION_QUALIFIED_NAME));
         }
         TypeElement annotation = annotations.iterator().next();
         checkArgument(annotation.getQualifiedName().contentEquals(ANNOTATION_QUALIFIED_NAME), "Wrong anno: %s", annotation);
@@ -88,7 +88,7 @@ void doProcess(Set<? extends Element> elements) {
                     ctx.warning("Type '%s' is ignored or invalid, but annotated with '%s'.", typeElement.getQualifiedName().toString(), ANNOTATION_QUALIFIED_NAME);
                 }
             } else {
-                throw new UnsupportedOperationException("Unsupported element: " + element);
+                throw new SharedTypeInternalError(String.format("Unsupported element: %s of kind %s", element, element.getKind()));
             }
         }
         List<TypeDef> resolvedDefs = resolver.resolve(discoveredDefs);

processor/src/main/java/org/sharedtype/processor/parser/EnumTypeDefParser.java

@@ -92,7 +92,8 @@ private List<EnumValueInfo> parseEnumConstants(TypeElement enumTypeElement, List
                     " Is this class from a dependency jar/compiled class file? Please refer to the documentation for more information.",
                     enumConstant, enumTypeElement);
             } else {
-                throw new SharedTypeInternalError(String.format("Unsupported tree, kind: %s, tree: %s, element: %s", tree.getKind(), tree, enumConstant));
+                throw new SharedTypeInternalError(String.format(
+                    "Unsupported tree during parsing enum %s, kind: %s, tree: %s, element: %s", enumTypeElement, tree.getKind(), tree, enumConstant));
             }
         }
         return res;
@@ -114,10 +115,10 @@ private Object resolveValue(TypeElement enumTypeElement, VariableTree tree, int
                 }
             } catch (IndexOutOfBoundsException e) {
                 throw new SharedTypeInternalError(String.format(
-                    "Initializer has incorrect number of arguments: %s in tree: %s, argIndex: %s", init, tree, ctorArgIdx));
+                    "Initializer in enum %s has incorrect number of arguments: %s in tree: %s, argIndex: %s", enumTypeElement, init, tree, ctorArgIdx));
             }
         }
-        throw new SharedTypeInternalError(String.format("Unsupported initializer: %s in tree: %s", init, tree));
+        throw new SharedTypeInternalError(String.format("Unsupported initializer in enum %s: %s in tree: %s", enumTypeElement, init, tree));
     }
 
     @RequiredArgsConstructor

processor/src/main/java/org/sharedtype/processor/parser/type/TypeInfoParserImpl.java

@@ -46,7 +46,7 @@ public TypeInfo parse(TypeMirror typeMirror) {
         } else if (typeKind == TypeKind.EXECUTABLE) {
             return parse(((ExecutableType) typeMirror).getReturnType());
         }
-        throw new SharedTypeInternalError(String.format("Unsupported field type, element: %s, typeKind: %s", typeMirror, typeKind)); // TODO: context info
+        throw new SharedTypeInternalError(String.format("Unsupported type: %s, typeKind: %s", typeMirror, typeKind));
     }
 
     private TypeInfo parseDeclared(DeclaredType declaredType) {

processor/src/main/java/org/sharedtype/processor/resolver/LoopTypeResolver.java

@@ -78,7 +78,7 @@ public List<TypeDef> resolve(List<TypeDef> typeDefs) {
                     }
                 }
             } else {
-                throw new SharedTypeInternalError("Unsupported type: " + typeDef.getClass());
+                throw new SharedTypeInternalError("Unsupported TypeDef type: " + typeDef.getClass());
             }
 
             resolveTypeInfo(processingDefStack, processingInfoStack);
@@ -113,11 +113,10 @@ private void resolveTypeInfo(Deque<TypeDef> processingDefStack, Deque<TypeInfo>
                 }
             } else if (typeInfo instanceof TypeVariableInfo) {
                 TypeVariableInfo typeVariableInfo = (TypeVariableInfo) typeInfo;
-                throw new UnsupportedOperationException("Type variable not supported yet: " + typeVariableInfo);
+                throw new SharedTypeInternalError("TypeVariableInfo is not supported yet: " + typeVariableInfo);
             } else {
-                throw new SharedTypeInternalError(
-                    String.format("Only ConcreteTypeInfo needs to be resolved, but got typeInfo: %s with %s", typeInfo, typeInfo.getClass())
-                );
+                throw new SharedTypeInternalError(String.format(
+                    "Only ConcreteTypeInfo needs to be resolved, but got: %s of %s", typeInfo, typeInfo.getClass()));
             }
         }
     }