#58 - Assert annotation is legal when adding to target

hibernate · Apr 4, 2024 · 7492b52 · 7492b52
1 parent 7d55df9
commit 7492b52
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/src/main/java/org/hibernate/models/internal/dynamic/AbstractAnnotationTarget.java b/src/main/java/org/hibernate/models/internal/dynamic/AbstractAnnotationTarget.java
@@ -46,6 +46,7 @@ public void clearAnnotationUsages() {
 	 * @todo It is undefined currently what happens if the annotation type is already applied on this target.
 	 */
 	public <X extends Annotation> void addAnnotationUsage(AnnotationUsage<X> annotationUsage) {
+		assert annotationUsage.getAnnotationDescriptor().getAllowableTargets().contains( getKind() );
 		final AnnotationUsage<?> previous = usageMap.put( annotationUsage.getAnnotationType(), annotationUsage );
 
 		if ( previous != null ) {

diff --git a/src/main/java/org/hibernate/models/internal/jandex/AbstractAnnotationTarget.java b/src/main/java/org/hibernate/models/internal/jandex/AbstractAnnotationTarget.java
@@ -46,6 +46,7 @@ public void clearAnnotationUsages() {
 
 	@Override
 	public <X extends Annotation> void addAnnotationUsage(AnnotationUsage<X> annotationUsage) {
+		assert annotationUsage.getAnnotationDescriptor().getAllowableTargets().contains( getKind() );
 		getUsageMap().put( annotationUsage.getAnnotationType(), annotationUsage );
 	}
 

diff --git a/src/main/java/org/hibernate/models/internal/jdk/AbstractAnnotationTarget.java b/src/main/java/org/hibernate/models/internal/jdk/AbstractAnnotationTarget.java
@@ -65,6 +65,7 @@ public void clearAnnotationUsages() {
 
 	@Override
 	public <X extends Annotation> void addAnnotationUsage(AnnotationUsage<X> annotationUsage) {
+		assert annotationUsage.getAnnotationDescriptor().getAllowableTargets().contains( getKind() );
 		getUsageMap().put( annotationUsage.getAnnotationType(), annotationUsage );
 	}
 }